stakewise/helm-charts

[Operator] Helm install failed

Closed this issue · 6 comments

On the operator the component annotation/label seems to be double defined...

      Helm install failed: error while running post render on files: map[string]interface {}(nil): yaml: unmarshal errors:    line 13: mapping key "component" already defined at line 10  

@anthr76 Thank you 🙏 for reporting the issue, fixed

 reconciliation failed: Helm install failed: error while running post render on files: map[string]interface {}(nil): yaml: unmarshal errors:
  line 13: mapping key "component" already defined at line 10

Appreciate a quick turnaround.. This appears to still be an issue. I'm going to post my values incase you suspect it's something with them. I'm on chart version 1.0.9

enabled: false
geth:
  network: goerli
graffiti: kutara
initImage:
  repository: europe-west4-docker.pkg.dev/stakewiselabs/public/validator-init
  tag: v1.0.2
initImageBusybox:
  repository: docker.io/library/busybox
  tag: "1.34"
lighthouse:
  network: goerli
network: prater
nodeSelector:
  kubernetes.io/arch: amd64
persistence:
  storageClassName: ceph-block
prometheusRule:
  enabled: true
  namespace: crypto
prysm:
  network: goerli
serviceMonitor:
  enabled: true
  namespace: crypto
type: lighthouse
vaultAddr: vault.infra.svc.cluster.local

@anthr76 sorry, I can't reproduce the problem:

$ helm upgrade --install operator stakewise/operator   -f test.yaml   --namespace test
Release "operator" has been upgraded. Happy Helming!
NAME: operator
LAST DEPLOYED: Mon Jan 24 15:14:09 2022
NAMESPACE: test
STATUS: deployed
REVISION: 2

Also your values.yaml looks a bit strange, it should look like this:

enabled: false
geth:
  enabled: true
  network: goerli
graffiti:   kutara
lighthouse:
  network: goerli
network:    prater
nodeSelector: {}
persistence:
  storageClassname:  ceph-block
prometheusRule:
  enabled:    true
  namespace:  crypto
prysm:
  enabled: true
  network:  goerli
serviceMonitor:
  enabled:   true
  namespace: test
type: lighthouse

Yes the values it is indeed indented because it's driven with Flux. So I'm just copying it out of a CRD from a describe. Which for some reason capitailzes everything... I updated it to better reflect that...

@anthr76 Could you check with version 1.0.10, please? It's already uploaded at https://charts.stakewise.io

That did it! Thank you @unxnn :)