Update deployment to work on Helm 2.14.1 - #453
Conversation
Signed-off-by: Maartje Eyskens <[email protected]>
dpordomingo
left a comment
dpordomingo
left a comment
There was a problem hiding this comment.
This breaks in Staging
UPGRADE FAILED
Error: incompatible versions client[v2.14.1] server[v2.8.2]
Error: UPGRADE FAILED: incompatible versions client[v2.14.1] server[v2.8.2]
time="2019-07-16T11:03:17Z" level=fatal msg="Error running helm command: upgrade
--install landing ./helm-charts/landing --set image.tag=commit-da21f71
--values .helm-staging.yml --tiller-namespace kube-system --wait"
If I understood the roadmap at https://github.com/src-d/infrastructure/issues/982#issuecomment-510949827, we're still at stage 2, so waiting for https://github.com/src-d/infrastructure/pull/983 to be merged and released.
|
Yes, we are not ready yet to merge, just needed approval. It will work once we have updated the tiller. |
|
@dpordomingo , helm updated in staging. Could you please check it again if it works in staging? Same thing for src-d/code-annotation#254 |
|
(... you can disregard this one; it worked.) |
dpordomingo
left a comment
dpordomingo
left a comment
There was a problem hiding this comment.
It worked in staging https://drone.srcd.host/src-d/landing/724
https://landing-staging.srcd.run/, so LGTM
Feel free to merge and release according to your plans. Many thanks!
3 participants
This upgrades the Helm plugin to work with the latest Helm version.