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

[prometheus] Add CI test cases #4477

Merged

Conversation

zeritti
Copy link
Member

@zeritti zeritti commented Apr 20, 2024

What this PR does / why we need it

The chart does not currently have any specific test cases to run in CI tasks and as a result, many fields and scenarios remain untested. This PR adds a few test cases setting fields that are otherwise unset or commented out.

Which issue this PR fixes

None

Special notes for your reviewer

Checklist

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [prometheus-couchdb-exporter])

Signed-off-by: zeritti <47476160+zeritti@users.noreply.github.com>
Signed-off-by: zeritti <47476160+zeritti@users.noreply.github.com>
@zeritti zeritti marked this pull request as ready for review April 20, 2024 22:04
@zeritti
Copy link
Member Author

zeritti commented Apr 22, 2024

cc @zanhsieh

@zeritti zeritti merged commit fe80f00 into prometheus-community:main Apr 25, 2024
4 checks passed
wilfriedroset pushed a commit to wilfriedroset/helm-charts that referenced this pull request May 8, 2024
* [prometheus] Add CI values files

Signed-off-by: zeritti <47476160+zeritti@users.noreply.github.com>

* Use helper template setting labels in clusterrole

Signed-off-by: zeritti <47476160+zeritti@users.noreply.github.com>

---------

Signed-off-by: zeritti <47476160+zeritti@users.noreply.github.com>
Co-authored-by: MH <zanhsieh@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants