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

[WILDFLY]: misconfigured mgmtIngress annotations #29787

Open
emilianosantucci opened this issue Oct 6, 2024 · 0 comments
Open

[WILDFLY]: misconfigured mgmtIngress annotations #29787

emilianosantucci opened this issue Oct 6, 2024 · 0 comments
Assignees
Labels
tech-issues The user has a technical issue about an application triage Triage is needed

Comments

@emilianosantucci
Copy link

Name and Version

bitnami/wildfly 15.2.1

What architecture are you using?

arm64

What steps will reproduce the bug?

  1. helm install wildfly bitnami/wildfly --version 15.2.1 -f values.yaml

Are you using any custom parameters or values?

exposeManagementConsole: true
ingress:
  enabled: true
  hostname: wildfly.192.168.70.2.nip.io
  annotations:
    nginx.ingress.kubernetes.io/proxy-body-size: 10m
mgmtIngress:
  enabled: true
  hostname: wildfly-management.192.168.70.2.nip.io
  annotations:
    nginx.ingress.kubernetes.io/proxy-body-size: 100m

What is the expected behavior?

  • ingress accept upload with max size 10m
  • mgmtIngress accept upload with max size 100m

What do you see instead?

Created ingress not inherit annotations added to values.yaml.

Additional information

Workaround used:

Use commonAnnotations into values.yaml as below:

commonAnnotations:
  nginx.ingress.kubernetes.io/proxy-body-size: 100m

I know it is not the best choice (it set this annotations on all components created) but allow me to load WAR with a size greater than 1Mb.

Hint for the resolutions:

I'm not a Helm master but I think the issue is on this template at row 19:

  • Is:
    {{- $annotations := include "common.tplvalues.merge" ( dict "values" ( list .Values.persistence.annotations .Values.commonAnnotations ) "context" . ) }}
  • Should be:
    {{- $annotations := include "common.tplvalues.merge" ( dict "values" ( list .Values.mgmtIngress.annotations .Values.commonAnnotations ) "context" . ) }}
@emilianosantucci emilianosantucci added the tech-issues The user has a technical issue about an application label Oct 6, 2024
@github-actions github-actions bot added the triage Triage is needed label Oct 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech-issues The user has a technical issue about an application triage Triage is needed
Projects
None yet
Development

No branches or pull requests

2 participants