-
Notifications
You must be signed in to change notification settings - Fork 126
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): improve error handling in metrics providers #1466
Conversation
✅ Deploy Preview for keptn-lifecycle-toolkit ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #1466 +/- ##
==========================================
+ Coverage 61.62% 63.06% +1.44%
==========================================
Files 137 139 +2
Lines 10456 10834 +378
==========================================
+ Hits 6444 6833 +389
+ Misses 3762 3745 -17
- Partials 250 256 +6
... and 15 files with indirect coverage changes
Flags with carried forward coverage won't be shown. Click here to find out more. |
0bd2151
to
ac8e92d
Compare
Signed-off-by: realanna <[email protected]> testerrorr Signed-off-by: realanna <[email protected]> testerrorr Signed-off-by: realanna <[email protected]> testerrorr Signed-off-by: realanna <[email protected]> testerrorr Signed-off-by: realanna <[email protected]> testerrorr Signed-off-by: realanna <[email protected]> testerrorr Signed-off-by: realanna <[email protected]> testerrorr Signed-off-by: realanna <[email protected]> testerrorr Signed-off-by: realanna <[email protected]> test params Signed-off-by: realanna <[email protected]> test params Signed-off-by: realanna <[email protected]> test params Signed-off-by: realanna <[email protected]>
ffcf9a4
to
3ab717b
Compare
Signed-off-by: realanna <[email protected]>
metrics-operator/controllers/common/providers/datadog/datadog.go
Outdated
Show resolved
Hide resolved
metrics-operator/controllers/common/providers/dynatrace/dynatrace.go
Outdated
Show resolved
Hide resolved
metrics-operator/controllers/common/providers/dynatrace/dynatrace_dql.go
Outdated
Show resolved
Hide resolved
Co-authored-by: Giovanni Liva <[email protected]> Signed-off-by: RealAnna <[email protected]>
…ace_dql.go Co-authored-by: Giovanni Liva <[email protected]> Signed-off-by: RealAnna <[email protected]>
…ace.go Co-authored-by: Giovanni Liva <[email protected]> Signed-off-by: RealAnna <[email protected]>
metrics-operator/controllers/common/providers/dynatrace/dynatrace.go
Outdated
Show resolved
Hide resolved
metrics-operator/controllers/common/providers/dynatrace/common.go
Outdated
Show resolved
Hide resolved
metrics-operator/controllers/common/providers/dynatrace/dynatrace.go
Outdated
Show resolved
Hide resolved
metrics-operator/controllers/common/providers/dynatrace/dynatrace_dql.go
Outdated
Show resolved
Hide resolved
Co-authored-by: odubajDT <[email protected]> Signed-off-by: RealAnna <[email protected]>
Signed-off-by: realanna <[email protected]>
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
This pr:
How to test:
Status: Err Msg: provider api response: Token Authentication failed