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

dynatrace_aws_credentials resource creating massive diffs, incorrectly reporting changes #220

Closed
FinUD opened this issue Mar 24, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@FinUD
Copy link

FinUD commented Mar 24, 2023

Describe the bug
For the "dynatrace_aws_credentials" resource when adding new supporting services to an already existing account with already monitored services, the order returned by the API is drastically different. For example we have services ABCDE initially applied. When applied they were ordered ABCDE. Now if we add F and still present them in the initial order adding F at the end like ABCDEF, they get returned jumbled by the API, for example as CBAFDE. This causes a massive diff to be created making it difficult to ensure that ABCDEF services were correctly applied.

To Reproduce

  1. Create an initial "dynatrace_aws_credentials" resource and apply it, including several monitored services and their metrics.
  2. Add a new monitored service and new metrics to the configuration.
  3. Apply the new configuration, observe the massive diff as it thinks more has changed than just the new monitored service.

Expected behavior
Only the new monitored services to be reported in the diff.

Additional context
Reordering of services by the API has actually happened in the past where it used to be alphabetical and now its unpredictable causing these huge diffs to be created even when there was no change. If this could be changed to be order-agnostic this would be massively helpful for configurations including hundreds of metrics spanning 50+ services.

Cheers

@FinUD FinUD added the bug Something isn't working label Mar 24, 2023
@Dynatrace-Reinhard-Pilz
Copy link
Member

Hello @FinUD , this issue will be taken care of with the upcoming release.

1 similar comment
@Dynatrace-Reinhard-Pilz
Copy link
Member

Hello @FinUD , this issue will be taken care of with the upcoming release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants