opensearch-project / opensearch-project/data-prepper

[BUG] HELM template service.yaml missing open 4900 port

Open
#5,724 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
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

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.