-
Notifications
You must be signed in to change notification settings - Fork 8.9k
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
[bitnami/prometheus] Network policy review #25902
Conversation
ec5bad1
to
c6ebca5
Compare
Apply the same changes made in PR bitnami#25519 to the Prometheus chart Signed-off-by: Ben Foster <bpfoster@gmail.com>
c6ebca5
to
374aaec
Compare
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.
Thank you so much for taking care of these charts! I appreciate it a lot @bpfoster!
There are some conflicts coming from a previous PR Could you give it a glance?
Signed-off-by: Ben Foster <bpfoster@gmail.com>
Co-authored-by: Fran Mulero <francisco-jose.mulero@broadcom.com> Signed-off-by: Ben Foster <bpfoster@gmail.com>
Thanks @fmulero , merged in the latest |
Signed-off-by: Ben Foster <bpfoster@gmail.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Signed-off-by: Fran Mulero <fmulero@vmware.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
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
* [bitnami/prometheus] Network policy review Apply the same changes made in PR bitnami#25519 to the Prometheus chart Signed-off-by: Ben Foster <bpfoster@gmail.com> * Apply suggestions from code review Co-authored-by: Fran Mulero <francisco-jose.mulero@broadcom.com> Signed-off-by: Ben Foster <bpfoster@gmail.com> * Update CHANGELOG.md Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com> * Fix Changelog Signed-off-by: Fran Mulero <fmulero@vmware.com> * Update CHANGELOG.md Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com> * Update CHANGELOG.md Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com> --------- Signed-off-by: Ben Foster <bpfoster@gmail.com> Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com> Signed-off-by: Fran Mulero <fmulero@vmware.com> Co-authored-by: Fran Mulero <francisco-jose.mulero@broadcom.com> Co-authored-by: Bitnami Containers <bitnami-bot@vmware.com> Co-authored-by: Fran Mulero <fmulero@vmware.com> Signed-off-by: Fran Mulero <fmulero@vmware.com>
Description of the change
Apply the same changes made in PR #25519 to the Prometheus chart
Review network policy features to allow users cover their needs without forcing specific labels.
Benefits
Users can restrict access to their deployments in a cleaner way, without leaving open doors to specific labeled pods.
Possible drawbacks
None
Applicable issues
Additional information
Checklist
Chart.yaml
according to semver. This is not necessary when the changes only affect README.md files.README.md
using readme-generator-for-helm