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

Docs feedback: Helm chart alloy.extraPorts.hostPort should be port instead #1997

Open
jkruke opened this issue Oct 29, 2024 · 0 comments · May be fixed by #1998
Open

Docs feedback: Helm chart alloy.extraPorts.hostPort should be port instead #1997

jkruke opened this issue Oct 29, 2024 · 0 comments · May be fixed by #1998
Labels
type/docs Docs Squad label across all Grafana Labs repos

Comments

@jkruke
Copy link

jkruke commented Oct 29, 2024

URL

https://artifacthub.io/packages/helm/grafana/alloy#alloy-extraports

Feedback

If I configure alloy.extraPorts according to the documentation as follows:

alloy:
  extraPorts:
    - targetPort: 4317
      hostPort: 4317
      name: otlp-grpc

I get this error from kubernetes after helm upgrade:

Error: UPGRADE FAILED: cannot patch "alloy" with kind Service: Service "alloy" is invalid: spec.ports[1].port: Invalid value: 0: must be between 1 and 65535, inclusive

This is because the generated manifest looks like this:

...
    ports:
      - name: http-metrics
         port: 12345
         targetPort: 12345
         protocol: "TCP"
      - name: otlp-grpc
        port: 
        targetPort: 4317
        protocol: TCP
.... 

So I changed configuration to the following:

alloy:
  extraPorts:
    - targetPort: 4317
      port: 4317
      name: otlp-grpc

...and it worked well!

This should be reflected to the documentation.

@jkruke jkruke added the type/docs Docs Squad label across all Grafana Labs repos label Oct 29, 2024
@jkruke jkruke linked a pull request Oct 29, 2024 that will close this issue
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/docs Docs Squad label across all Grafana Labs repos
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant