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(metrics-operator): update controller logic to support multiple metric values #2190

Merged
Merged
Changes from 1 commit
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
e7ef0c3
feat: update controller logic to support multiple metric values
rakshitgondwal Sep 27, 2023
458fbb2
Merge branch 'main' into feat/controller-update-for-multiple-metrics
rakshitgondwal Oct 6, 2023
f1a3204
modify logic
rakshitgondwal Oct 6, 2023
e318b9f
remove storedResults as a pointer
rakshitgondwal Oct 6, 2023
5c8fbcf
fix after review
rakshitgondwal Nov 1, 2023
89455c1
Merge branch 'main' into feat/controller-update-for-multiple-metrics
rakshitgondwal Nov 1, 2023
a92b566
Merge branch 'main' into feat/controller-update-for-multiple-metrics
rakshitgondwal Jan 7, 2024
49b5088
add range check
rakshitgondwal Jan 7, 2024
e474a02
fix range check
rakshitgondwal Jan 8, 2024
f75ebab
add and fix tests
rakshitgondwal Jan 10, 2024
0d9dbe8
add more tests
rakshitgondwal Jan 10, 2024
a3efcf4
fix errors
rakshitgondwal Jan 10, 2024
89538b8
add integration test
rakshitgondwal Jan 10, 2024
d67210a
fix yaml lint err
rakshitgondwal Jan 10, 2024
593680b
Merge branch 'main' into feat/controller-update-for-multiple-metrics
rakshitgondwal Jan 10, 2024
73cad1f
fix test
rakshitgondwal Jan 10, 2024
b03a80a
fix after review
rakshitgondwal Jan 11, 2024
0621045
Merge branch 'main' into feat/controller-update-for-multiple-metrics
rakshitgondwal Jan 12, 2024
7e3e753
Merge branch 'main' into feat/controller-update-for-multiple-metrics
rakshitgondwal Jan 16, 2024
6ae2f6f
fix lint err
rakshitgondwal Jan 16, 2024
9a5c33b
fix after review
rakshitgondwal Jan 17, 2024
5ad411e
Merge branch 'main' into feat/controller-update-for-multiple-metrics
bacherfl Jan 18, 2024
291b111
Merge branch 'main' into feat/controller-update-for-multiple-metrics
bacherfl Jan 18, 2024
20ceab9
adapt unit tests
rakshitgondwal Jan 19, 2024
7ceb7a8
Merge branch 'main' into feat/controller-update-for-multiple-metrics
rakshitgondwal Jan 19, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'main' into feat/controller-update-for-multiple-metrics
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
bacherfl authored Jan 18, 2024
commit 5ad411ecd305738da5ff484d6b24f72c39cdfd0c
8 changes: 8 additions & 0 deletions test/testmetrics/metrics/02-teststep-assert.yaml
Original file line number Diff line number Diff line change
@@ -11,3 +11,11 @@ assert: # this checks that kubectl get resource succeeds
- goodmetric4.yaml
- goodmetric5.yaml
- goodmetric6.yaml
collectors:
- type: pod
selector: app=test
- command: |
kubectl logs -l app=metrics-operator -n keptn-system
kubectl describe keptnmetric podtatometric -n $NAMESPACE
kubectl describe keptnmetricsprovider dynatrace -n $NAMESPACE
kubectl describe pods -l app=podtato-head -n $NAMESPACE
You are viewing a condensed version of this merge commit. You can view the full changes here.