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.

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

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

Open
jkruke opened this issue Oct 29, 2024 · 1 comment · 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 · 1 comment · May be fixed by #1998
Labels
needs-attention 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
Open
2 tasks
@github-actions GitHub Actions
Copy link
Contributor

This issue has not had any activity in the past 30 days, so the needs-attention label has been added to it.
If the opened issue is a bug, check to see if a newer release fixed your issue. If it is no longer relevant, please feel free to close this issue.
The needs-attention label signals to maintainers that something has fallen through the cracks. No action is needed by you; your issue will be kept open and you do not have to respond to this comment. The label will be removed the next time this job runs if there is new activity.
Thank you for your contributions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
needs-attention type/docs Docs Squad label across all Grafana Labs repos
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix documentation for helm extraPorts jkruke/alloy
1 participant
@jkruke