-
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: move prometheus install into make file #1093
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 #1093 +/- ##
=======================================
Coverage 58.91% 58.92%
=======================================
Files 126 126
Lines 10007 10007
=======================================
+ Hits 5896 5897 +1
- Misses 3893 3895 +2
+ Partials 218 215 -3 see 3 files with indirect coverage changes
Flags with carried forward coverage won't be shown. Click here to find out more. |
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.
LGTM!
…n tests Signed-off-by: realanna <[email protected]>
Signed-off-by: realanna <[email protected]>
a94814d
to
6e2df99
Compare
Kudos, SonarCloud Quality Gate passed! |
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.
good work! lgtm
Signed-off-by: realanna <[email protected]>
closes #1085
kind create cluster
helm upgrade --install klt klt/klt -n keptn-lifecycle-toolkit-system --create-namespace
make integration-test-local
now Prometheus is installed before running so all tests pass locally