Skip to content
This repository has been archived by the owner on Jan 26, 2021. It is now read-only.

az acr helm push job files on CD pipeline #52

Open
frdvo opened this issue Jul 24, 2020 · 4 comments
Open

az acr helm push job files on CD pipeline #52

frdvo opened this issue Jul 24, 2020 · 4 comments

Comments

@frdvo
Copy link

frdvo commented Jul 24, 2020

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 frdvo changed the title az acr helm push az acr helm push job files on CD pipeline Jul 24, 2020
@WilldotNET
Copy link

I raised this issue with the azure-cli open source team recently - Azure/azure-cli#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
Copy link

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.

@g0pinath-metricon
Copy link

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.

@ricklagerweij
Copy link

add export HELM_EXPERIMENTAL_OCI=1 infront of cd $projectName

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

4 participants