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

[1.12.x] Duplicated "VolumeMounts" Keys in 1-gloo-deployment.yaml #6977

Closed
vmarlier opened this issue Aug 17, 2022 · 2 comments
Closed

[1.12.x] Duplicated "VolumeMounts" Keys in 1-gloo-deployment.yaml #6977

vmarlier opened this issue Aug 17, 2022 · 2 comments
Assignees
Labels
Type: Bug Something isn't working

Comments

@vmarlier
Copy link

Gloo Edge Version

1.12.x (latest stable)

Kubernetes Version

1.22.x

Describe the bug

Hello,
During our attempt to update Gloo to 1.12.6 with our Fluxv2 stack, we noticed an issue outputted by the Helm-Controller.
Here is the error:

Helm upgrade failed: error while running post render on files: map[string]interface {}(nil): yaml: unmarshal errors:
        line 101: mapping key "volumeMounts" already defined at line 75

Basically Helm is complaining about a duplicated volumeMounts key.

The issue is only firing when we want to install through the flux stack since they added a check for duplated keys. When using the helm cli, it seems helm do not complain but take randomly one of the VolumeMounts key definition. So it is still an issue.

When taking a look at the file for 1.12.x I can see a duplication here and here.

It is possible to fix this please ?

Steps to reproduce the bug

  1. Install Gloo via Fluxv2 stack with a version prior to 1.12.x
  2. Update to 1.12.x
  3. Should see an issue with the fluxv2 helm-controller.

Expected Behavior

Being able to install it through fluxv2 without any blocker.

Additional Context

No response

@vmarlier vmarlier added the Type: Bug Something isn't working label Aug 17, 2022
@kevin-shelaga
Copy link
Contributor

looks to be an issue when gateway.validation.enabled and gateway.enabled are true

@sam-heilbron
Copy link
Contributor

This has been resolved in affected versions (1.12, 1.3) of Gloo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants