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-node-exporter] Unable to define Service port seperately from container port. #4414

Open
arahja opened this issue Apr 4, 2024 · 0 comments · May be fixed by #4415
Open

[prometheus-node-exporter] Unable to define Service port seperately from container port. #4414

arahja opened this issue Apr 4, 2024 · 0 comments · May be fixed by #4415
Labels
bug Something isn't working

Comments

@arahja
Copy link

arahja commented Apr 4, 2024

Describe the bug a clear and concise description of what the bug is.

Unable to define the port for just the Kubernetes service. If you change .Values.service.port you change the container port defined in the daemonset and the Kubernetes service. We need a way to define just the Kubernetes service port.

What's your helm version?

3.14.2

What's your kubectl version?

1.28.8

Which chart?

prometheus-node-exporter

What's the chart version?

4.32.0

What happened?

Node exporter pods failed to start.

What you expected to happen?

Node exporter pods to start up and the kubernetes service to respond to port 80 forwarding to the node exporter default port of 9100.

How to reproduce it?

Set the .Values.service.port to 80 without having to change the security context.

Enter the changed values of values.yaml?

service:
port: 80

Enter the command that you execute and failing/misfunctioning.

helm install --namespace testing --generate-name prometheus-community/prometheus-node-exporter --set service.port=80

Anything else we need to know?

No response

@arahja arahja added the bug Something isn't working label Apr 4, 2024
arahja added a commit to arahja/prometheus-helm-charts that referenced this issue Apr 4, 2024
…e Kubernetes service port seperately from the container port. fixes prometheus-community#4414
arahja added a commit to arahja/prometheus-helm-charts that referenced this issue Apr 4, 2024
…e Kubernetes service port seperately from the container port. fixes prometheus-community#4414

Signed-off-by: Adam Rahja <adam@resolve.com>
arahja added a commit to arahja/prometheus-helm-charts that referenced this issue Apr 4, 2024
…e Kubernetes service port seperately from the container port. fixes prometheus-community#4414

Signed-off-by: Adam Rahja <arahja@gmail.com>
arahja added a commit to arahja/prometheus-helm-charts that referenced this issue Apr 4, 2024
…e Kubernetes service port seperately from the container port. fixes prometheus-community#4414

Signed-off-by: Adam Rahja <arahja@gmail.com>
@arahja arahja linked a pull request Apr 4, 2024 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant