Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

Kubernetes 1.7 chart failures #1294

Closed
foxish opened this issue Jun 13, 2017 · 8 comments
Closed

Kubernetes 1.7 chart failures #1294

foxish opened this issue Jun 13, 2017 · 8 comments

Comments

@foxish
Copy link
Member

foxish commented Jun 13, 2017

Reported failures from https://k8s-testgrid.appspot.com/kubernetes-apps

Cockroachdb:
W0613 14:19:20.823] Error: release wzq failed: StatefulSet.apps "wzq-cockroachdb" is invalid: spec.template.annotations.affinity.podAntiAffinity.preferredDuringSchedulingIgnoredDuringExecution[0].podAffinityTerm.topologyKey: Required value: can not be empty

Spinnaker:
W0613 14:19:49.774] Error: found in requirements.yaml, but missing in charts/ directory: redis, minio

Jenkins:
W0613 14:19:51.019] Error: error validating "": error validating data: unexpected type: types.UnixUserID

cc @kubernetes/charts-maintainers @viglesiasce

@foxish
Copy link
Member Author

foxish commented Jun 15, 2017

@foxish
Copy link
Member Author

foxish commented Jun 22, 2017

Jenkins issue is now resolved because upstream k8s reverted the backward incompatible API change. The test is now passing. Cockroachdb will pass after we merge #1306 on the day 1.7 releases.

@wclr
Copy link

wclr commented Jul 7, 2017

Wound it be useful if one could change an arbitrary needed field in the resources definitions (that can not be changed via values abstraction), not by cloning/copy/pasting entire chart code and modifying the template?

I think helm charts has a kind of flaw abstraction model, that doesn't allow to reuse them in more flexible way.

@paulwalker
Copy link

I'd like to add that concourse.ci is also victim to this.

What exactly is the yaml workaround btw?

@paralin
Copy link

paralin commented Jul 27, 2017

What's the fix for this?

@cmcconnell1
Copy link

cmcconnell1 commented Oct 13, 2017

Hello this issue persists confirmed installing according to README/doc.

helm install stable/concourse --namespace concourse
Error: release funky-cricket failed: StatefulSet.apps "funky-cricket-worker" is invalid: spec.template.annotations.affinity.podAntiAffinity.preferredDuringSchedulingIgnoredDuringExecution[0].podAffinityTerm.topologyKey: Required value: can not be empty

helm status funky-cricket | head -3
LAST DEPLOYED: Fri Oct 13 14:51:52 2017
NAMESPACE: concourse
STATUS: FAILED

Note that when deploying from source repo after checking for and installing any missing dependencies:

helm dep list
helm dep update 
helm dep build

. . . then helm install seems to get past the error and the deployment appears to be successful according to helm, but note that the postgresql and worker pods remain in pending state:

helm install stable/concourse -f $GIT_HOME/charts/stable/concourse/values.yaml --namespace concourse

helm status concourse | head -3
LAST DEPLOYED: Fri Oct 13 15:07:52 2017
NAMESPACE: concourse
STATUS: DEPLOYED

kubectl get all --namespace concourse
NAME                          DESIRED   CURRENT   UP-TO-DATE   AVAILABLE   AGE
deploy/concourse-postgresql   1         1         1            0           3m
deploy/concourse-web          1         1         1            0           3m

NAME                                 DESIRED   CURRENT   READY     AGE
rs/concourse-postgresql-4126483298   1         1         0         3m
rs/concourse-web-1449843452          1         1         0         3m

NAME                          DESIRED   CURRENT   UP-TO-DATE   AVAILABLE   AGE
deploy/concourse-postgresql   1         1         1            0           3m
deploy/concourse-web          1         1         1            0           3m

NAME                            DESIRED   CURRENT   AGE
statefulsets/concourse-worker   2         1         3m

NAME                                       READY     STATUS    RESTARTS   AGE
po/concourse-postgresql-4126483298-49w1v   0/1       Pending   0          3m
po/concourse-web-1449843452-1drm9          0/1       Running   1          3m
po/concourse-worker-0                      0/1       Pending   0          3m

Thanks

@jondlm
Copy link

jondlm commented Nov 16, 2017

I ran into a similar issue with stable/concourse. I had to run helm repo update to ensure that the newest chart version was getting installed. After that it worked for me.

@foxish
Copy link
Member Author

foxish commented Nov 18, 2017

This is likely out of date since 1.8 came out already. Individual issues should be filed for each of these against chart maintainers. Closing

@foxish foxish closed this as completed Nov 18, 2017
munnerz added a commit to munnerz/charts-1 that referenced this issue Feb 5, 2019
* Automated cherry pick of helm#1314 (cert-manager/cert-manager#1315)
* Automated cherry pick of helm#1294 (cert-manager/cert-manager#1296)
* Automated cherry pick of helm#1276 (cert-manager/cert-manager#1277)
* Automated cherry pick of helm#1258 helm#1266 (cert-manager/cert-manager#1273)
* Automated cherry pick of helm#1259 (cert-manager/cert-manager#1260)
* Update Chart.yaml in webhook (cert-manager/cert-manager#1249)
munnerz added a commit to munnerz/charts-1 that referenced this issue Feb 5, 2019
* Automated cherry pick of helm#1314 (cert-manager/cert-manager#1315)
* Automated cherry pick of helm#1294 (cert-manager/cert-manager#1296)
* Automated cherry pick of helm#1276 (cert-manager/cert-manager#1277)
* Automated cherry pick of helm#1258 helm#1266 (cert-manager/cert-manager#1273)
* Automated cherry pick of helm#1259 (cert-manager/cert-manager#1260)
* Update Chart.yaml in webhook (cert-manager/cert-manager#1249)

Signed-off-by: James Munnelly <james@munnelly.eu>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants