-
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
poc: integrate KeptnMetrics into Flagger analysis #3371
Conversation
Signed-off-by: Florian Bacher <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3371 +/- ##
==========================================
- Coverage 88.04% 85.31% -2.74%
==========================================
Files 176 167 -9
Lines 9604 7415 -2189
==========================================
- Hits 8456 6326 -2130
+ Misses 854 800 -54
+ Partials 294 289 -5 see 48 files with indirect coverage changes
Flags with carried forward coverage won't be shown. Click here to find out more. |
Signed-off-by: Florian Bacher <[email protected]>
In this example, we are making use of the Prometheus endpoint provided | ||
by Keptn (i.e. the metrics-operator), which serves the values of all `KeptnMetrics`. | ||
|
||
This way, we are able to use a Flagger `MetricTemplate` of type `prometheus`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This way, we are able to use a Flagger `MetricTemplate` of type `prometheus`, | |
This enables us to use a Flagger `MetricTemplate` of type `prometheus`, |
by Keptn (i.e. the metrics-operator), which serves the values of all `KeptnMetrics`. | ||
|
||
This way, we are able to use a Flagger `MetricTemplate` of type `prometheus`, | ||
which retrieves the value from a Prometheus instance that has access to the `KeptnMetrics`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
which retrieves the value from a Prometheus instance that has access to the `KeptnMetrics`. | |
which retrieves the value from a Prometheus instance that has access to the `KeptnMetrics` resource. |
The example is based on the [Istio Canary Deployments tutorial](https://docs.flagger.app/tutorials/istio-progressive-delivery) | ||
provided in the Flagger docs. | ||
|
||
The difference to the tutorial is that instead of using the `request-duration` duration |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The difference to the tutorial is that instead of using the `request-duration` duration | |
The difference from the tutorial is that, instead of using the `request-duration` duration |
Signed-off-by: Florian Bacher <[email protected]>
Quality Gate passedIssues Measures |
This PR has been automatically marked as stale because it has not had recent activity. It will be |
since this is an old PoC i believe we can close it, please re-open if needed |
Draft PR for a PoC - DO NOT MERGE
Part of #3050