Skip to content

helm: support prometheus port #4666

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.

Sign up for GitHub

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

Jump to bottom
Closed
Tracked by #4628
, #4748
...
bpmct opened this issue Oct 20, 2022 · 4 comments · Fixed by #5213
Closed
Tracked by #4628
, #4748
...

helm: support prometheus port #4666

bpmct opened this issue Oct 20, 2022 · 4 comments · Fixed by #5213
Assignees
@ericpaulsen @mtojek

Comments

@bpmct
Copy link
Member

bpmct commented Oct 20, 2022

I have set the environment variables CODER_PROMETHEUS_ENABLE and CODER_PROMETHEUS_ADDRESS in my helm chart, but I suppose I can't use a PodMonitoring to ingest into Google Cloud Monitoring until the port is exposed in the pod spec?

I might be missing something though!
@bpmct bpmct mentioned this issue Oct 20, 2022
Closed
13 tasks
@bpmct bpmct changed the title helm: support prometheus endpoint helm: support prometheus port Oct 20, 2022
@coadler
Copy link
Contributor

coadler commented Oct 20, 2022

This is correct!

@ericpaulsen
Copy link
Member

@coadler what's the port we are exposing? I can take this.

@ericpaulsen ericpaulsen self-assigned this Oct 24, 2022
@coadler
Copy link
Contributor

coadler commented Oct 24, 2022

By default it's 2112 but you can set it to whatever you want. You'll probably also need to change it to listen on 0.0.0.0.

@bpmct bpmct mentioned this issue Oct 25, 2022
Closed
21 tasks
@bpmct bpmct mentioned this issue Nov 28, 2022
Merged
@mtojek
Copy link
Member

mtojek commented Dec 1, 2022 •
edited
Loading

It looks like the issue is stale, so I hope you don't mind if I take over it as I haven't looked into the helm chart yet (ramp-up).

@mtojek mtojek self-assigned this Dec 1, 2022
@mtojek mtojek mentioned this issue Dec 1, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ericpaulsen ericpaulsen

@mtojek mtojek

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: Define Prometheus port in the pod spec mtojek/coder
4 participants
@coadler @ericpaulsen @mtojek @bpmct