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

Feature Request: App Mesh Gateway add Support for passing Multiple Port List Items #455

Closed
colinbjohnson opened this issue Feb 16, 2021 · 1 comment · May be fixed by #1154
Closed

Feature Request: App Mesh Gateway add Support for passing Multiple Port List Items #455

colinbjohnson opened this issue Feb 16, 2021 · 1 comment · May be fixed by #1154
Labels
enhancement New feature or request

Comments

@colinbjohnson
Copy link

Is your feature request related to a problem?

I believe the appmesh-gateway service should support multiple "port" configurations in the eks-charts/stable/appmesh-gateway/templates/service.yaml file so that I can accept traffic on both ports 80 and 443.

Is your feature request related to a specific Helm chart, if yes mention name of the chart?

appmesh-gateway

Describe the solution you'd like

I would like the appmesh-gateway to support multiple "port" list items. The current configuration is:

  ports:
  - port: {{ .Values.service.port }}
    targetPort: http
    name: http
    protocol: TCP

But I believe a list of "ports" should be able to be passed in.

Describe alternatives you've considered

Options I've considered:

  • submitting a PR with a fix
  • modifying the load balancer after creating the App Mesh helm chart.
@colinbjohnson
Copy link
Author

Will close this issue due to #1154.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
1 participant