-
Notifications
You must be signed in to change notification settings - Fork 118
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
fix(metrics-operator): use context with timeout for fetching analysis values #2213
Conversation
… values Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #2213 +/- ##
=======================================
Coverage 84.61% 84.61%
=======================================
Files 154 154
Lines 9957 9957
=======================================
Hits 8425 8425
Misses 1245 1245
Partials 287 287
Flags with carried forward coverage won't be shown. Click here to find out more. |
✅ Deploy Preview for keptn-lifecycle-toolkit ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for keptn-lifecycle-toolkit ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
… fix/dql-provider-debugging
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Kudos, SonarCloud Quality Gate passed!
|
… values (keptn#2213) Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
The analysis controller did not pass the correct context through to the workers for fetching the analysis values, and thus no timeout was applied. This lead to the controller stalling if one of the workers did not finish