Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add support for tags to the newrelic_entity data source #2682

Merged
merged 2 commits into from
Jun 6, 2024

Conversation

abasha1234
Copy link
Member

data_source_newrelic_entity

Fixes # issue-2666

In the GitHub issue linked above, it has been reported as feedback from a customer that the newrelic_entity data source in the New Relic Terraform Provider currently does not return the tags of the entity fetched. This PR will add support for tags to the data_source_newrelic_entity

@abasha1234 abasha1234 marked this pull request as ready for review June 4, 2024 02:32
@pranav-new-relic pranav-new-relic changed the title feat: Add Support for Tags to the newrelic_entity Data Source feat: add support for tags to the newrelic_entity data source Jun 5, 2024
@pranav-new-relic
Copy link
Member

pranav-new-relic commented Jun 5, 2024

All other failures in integration tests seem to be expected because of latency, except

  • TestAccNewRelicEntityData_RetrieveSubAccountEntity
  • TestAccNewRelicEntityData_EntityInSubAccount

and these aren't working because the entity they're using in the referenced account no longer exists. I've just replenished the entity, and both of these tests shouldn't be failing from the next run.

@pranav-new-relic
Copy link
Member

Replenished the dead entity again; rerunning integration tests

@abasha1234 abasha1234 merged commit bff769e into main Jun 6, 2024
13 of 14 checks passed
@abasha1234 abasha1234 deleted the NR-273644 branch June 6, 2024 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants