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

Server side apply does not properly render volumeClaimTemplates #981

Open
thomaspetit opened this issue Nov 27, 2023 · 0 comments
Open

Server side apply does not properly render volumeClaimTemplates #981

thomaspetit opened this issue Nov 27, 2023 · 0 comments
Labels
bug Something isn't working

Comments

@thomaspetit
Copy link

thomaspetit commented Nov 27, 2023

Describe the bug
Server side apply does not properly render volumeClaimTemplates. When rendering the Helm chart with server side apply we can see that the volumeClaimTemplates object (with datastorage enabled) renders the following:

  volumeClaimTemplates:
    - metadata:
        ...
      spec:
        accessModes:
          ...

While it should actually be the following

  volumeClaimTemplates:
    - apiVersion: v1
      kind: PersistentVolumeClaim
      metadata:
        ...
      spec:
        accessModes:
          ...

To Reproduce

  1. Setup ArgoCD
  2. Add Vault helm chart as an application
  3. Enable datastorage on the Helm chart

Expected behavior
Make the volumeClaimTemplate more explicit/declarative and thus eliminate possible errors when applying

Environment

  • Kubernetes version: 1.27
  • vault-helm version: 1.15.2

Chart values:

vault:
  controller:
    imagePullSecrets: []
  ui:
    enabled: true
  injector:
    enabled: false
  server:
    ingress:
      enabled: true
      hosts:
        - host: vault.foo.bar

Additional context

Issue was introduced with the following change: kubernetes/kubernetes#87706

Good to know is that this issue is not new: argoproj/argo-cd#11143
Other downstream applications nicely fixed this: vernemq/docker-vernemq#347

@thomaspetit thomaspetit added the bug Something isn't working label Nov 27, 2023
thomaspetit added a commit to thomaspetit/vault-helm that referenced this issue Nov 27, 2023
…lates

Provide a default implementation for volumeClaimTemplates when applying resources with server side apply

See hashicorp#981
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant