opensearch-project / opensearch-project/data-prepper
[BUG] HELM template service.yaml missing open 4900 port
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 374
- Forks
- 354
- Avg merge
- 3d 18h
- Merged PRs (30d)
- 8
Description
Describe the bug
A clear and concise description of what the bug is.
in the HELM chart, deployment.yaml template, it opens 4900 port by default. this is OK.
- name: server
containerPort: {{ (.Values.config).serverPort | default 4900 }}
protocol: TCP
however, the HELM's service.yaml template doesn't have 4900 port open.
so, if not changing the devault values.yaml file, then the 4900 port is missing from the clusterIP service.
If adding the 4900 port in the .Values.ports manually, then the deployment will repot 4900 being open 2 times.
W0528 11:33:20.601128 17742 warnings.go:70] spec.template.spec.containers[0].ports[5]: duplicate port definition with spec.template.spec.containers[0].ports[4]
see the deployment.yaml template:
ports:
{{- range .Values.ports }}
- name: {{ .name }}
containerPort: {{ .port }}
protocol: TCP
{{- end }}
- name: server
containerPort: {{ (.Values.config).serverPort | default 4900 }}
protocol: TCP
To Reproduce
Steps to reproduce the behavior:
see above
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):
HELM version: 0.3.1
Additional context
Add any other context about the problem here.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Compare the Helm chart's deployment.yaml and service.yaml templates, starting with how .Values.config.serverPort and .Values.ports are rendered. Verify the default server port is exposed by the ClusterIP service without creating a duplicate deployment port, then reproduce the chart installation using the default values.yaml.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- helm
- Domain
- devops
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 55/100