phippyandfriends icon indicating copy to clipboard operation
phippyandfriends copied to clipboard

az acr helm push job files on CD pipeline

Open frdvo opened this issue 5 years ago • 4 comments

On all CI pipelines, on az acr helm push job, I'm getting the error:

##[error]This command is implicitly deprecated because command group 'acr helm' is deprecated and will be removed in a future release. Use 'helm v3' instead.

I can bypass the error setting failOnStderr to false on /common/ci-steps-template.yml 'line 42

What should be the proper way to resolve that?

frdvo avatar Jul 24 '20 02:07 frdvo

I raised this issue with the azure-cli open source team recently - https://github.com/Azure/azure-cli/issues/14467

It looks like az acr helm is being dropped, so you'll need to add in helm save and helm push to replace az acr helm push. I'm working through it at the moment on my own project and will update phippy and friends via pull request if someone hasn't beat me to it already :)

WilldotNET avatar Jul 28 '20 08:07 WilldotNET

Just as an update - I haven't been able to get the helm push and subsequent helm upgrade to work using helm v3, not sure what the issue is but I've burnt a lot of time on it, so I'm sticking with acr push at the moment.

WilldotNET avatar Aug 10 '20 12:08 WilldotNET

Updated the CI steps as below

  • bash: |
    cd $(projectName)/charts/$(projectName) echo $(registryPassword) | helm registry login $(registryServerName) --username $(registryLogin) --password-stdin helm chart save . $(projectName):v$(Build.BuildId) helm chart save . $(registryServerName)/helm/$(projectName):v$(Build.BuildId) helm chart push $(registryServerName)/helm/$(projectName):v$(Build.BuildId)

CD steps echo $(registryPassword) | helm registry login $(registryServerName) --username azaksdevgopi --password-stdin helm chart pull $(registryServerName)/helm/$(projectName):v$(Build.BuildId)

helm chart export $(registryServerName)/helm/$(projectName):v$(Build.BuildId) --destination ./install cd install helm install parrot ./parrot --namespace $(k8sNamespace) --wait --set AppVersion=$(Build.BuildId)

in values.yml add AppVersion to provide the tag value at runtime.

-- tag for helm has to start with a v -- helm upgrade fails when there is no release present, so I had to use helm install. -- there is no switches for appversion and version as far as I can tell - similar to helm package with version. I didnt dig much into it, and will be watching this repo for updates.

g0pinath-metricon avatar Aug 11 '20 01:08 g0pinath-metricon

add export HELM_EXPERIMENTAL_OCI=1 infront of cd $projectName

ricklagerweij avatar Sep 21 '20 11:09 ricklagerweij