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] Adding additionalPorts and servicePort #3779

Merged
merged 5 commits into from
Sep 13, 2023

Conversation

mortenbirkelund
Copy link
Contributor

@mortenbirkelund mortenbirkelund commented Sep 11, 2023

What this PR does / why we need it

Similar to #2163 for the kube-prometheus-stack, this PR will make it possible to add an oauth proxy as a sidecar container, and change the ingress to point to an additional port in your service, that then goes to your oauth proxy. Nice and simple way of adding oauth to Prometheus.

Which issue this PR fixes

No issues opened for this chart, but this have been solved for the kube-prometheus-stack chart, so why not also make it possible in the prometheus chart.

Special notes for your reviewer

I have checked the helm chart with my changed, and it seems to work fine, but let me know if any changes is required.

Checklist

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

zanhsieh
zanhsieh previously approved these changes Sep 12, 2023
charts/prometheus/Chart.yaml Outdated Show resolved Hide resolved
@zeritti zeritti merged commit 183caee into prometheus-community:main Sep 13, 2023
4 checks passed
@mortenbirkelund mortenbirkelund deleted the oauth branch September 24, 2023 20:17
Matiasmct pushed a commit to giffgaff/prometheus-charts-backup that referenced this pull request Mar 20, 2024
…nity#3779)

* Adding additionalPorts and servicePort

Signed-off-by: Morten Birkelund <[email protected]>

* Change version bump from patch to minor

Co-authored-by: zeritti <[email protected]>
Signed-off-by: Morten Birkelund <[email protected]>

---------

Signed-off-by: Morten Birkelund <[email protected]>
Signed-off-by: MH <[email protected]>
Signed-off-by: Morten Birkelund <[email protected]>
Co-authored-by: MH <[email protected]>
Co-authored-by: zeritti <[email protected]>
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.

3 participants