New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Moving Tiller to new location #8884
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Saw one tiny gotcha in the CI script. Everything else looks good otherwise. Thanks for working on this!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
otherwise this lgtm 👍
I was going to suggest perhaps a GitHub action could simplify auth now that we're omving to ghcr, but I see the GITHUB_TOKEN doesn't have the right perms for that yet during the beta. Plus we want to auth for the other registries anyway. So could be a follow-up if we want to do that.
.circleci/deploy.sh
Outdated
az storage blob upload -f "KEYS" -n "KEYS" -c "$AZURE_STORAGE_CONTAINER_NAME" --connection-string "$AZURE_STORAGE_CONNECTION_STRING" | ||
echo "Pushing KEYS file to Azure" | ||
az storage blob upload -f "KEYS" -n "KEYS" -c "$AZURE_STORAGE_CONTAINER_NAME" --connection-string "$AZURE_STORAGE_CONNECTION_STRING" | ||
endif |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shellify. also adding newline at EOF
endif | |
fi | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed. thanks
.circleci/deploy.sh
Outdated
|
||
# Canary version is used with helm init --canary-image flag. | ||
# Does not push canary binary which is Helm v3. | ||
ifneq ($(VERSION),"canary") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like some make crept in. shellify-ing:
ifneq ($(VERSION),"canary") | |
if [ "$VERSION" != "canary" ]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch. fixed. thanks.
3867d50
to
c63eb1e
Compare
The tiller primary location is now GitHub Container Registry. It is also replicatated to GCR, Docker Hub, and Quay. Signed-off-by: Matt Farina <matt@mattfarina.com>
c63eb1e
to
01dc62e
Compare
The tiller primary location is now GitHub Container Registry.
It is also replicatated to GCR, Docker Hub, and Quay.
What this PR does / why we need it:
GCR is going away. We want to preserve tiller beyond the end of life for Helm v2. This moves Tiller to GHCR along with making it available on Quay and Docker Hub.
Special notes for your reviewer:
The environment variables are already setup in CI.
If applicable: