Skip to content
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

Start pushing alpha helm chart releases with every alpha build #269

Merged
merged 1 commit into from
Nov 1, 2019

Conversation

rvmiller89
Copy link
Contributor

Description

This will start generating alpha helm chart releases in our Github Pages-hosted chart: https://github.com/SumoLogic/sumologic-kubernetes-collection/tree/gh-pages

Will need to merge to test this on Travis CI

Testing performed
  • ci/build.sh
  • Redeploy fluentd and fluentd-events pods
  • Confirm events, logs, and metrics are coming in

@rvmiller89 rvmiller89 requested a review from lei-sumo November 1, 2019 17:11
Copy link
Contributor

@lei-sumo lei-sumo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one question, rest lgtm

@@ -143,7 +143,7 @@ if [ -n "$DOCKER_PASSWORD" ] && [ -n "$TRAVIS_TAG" ] && [[ $TRAVIS_TAG != *alpha
# Push new helm release
git checkout -- .
sudo helm init --client-only
sudo helm package deploy/helm/sumologic --version=$VERSION
sudo helm package deploy/helm/sumologic --version=$VERSION --app-version=$VERSION
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do we need to change this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Helm seems to be doing this already, but it makes me nervous relying on that undocumented behavior that could change. So to be explicit I added this.

@rvmiller89 rvmiller89 merged commit 75bc0ae into master Nov 1, 2019
@djsly
Copy link
Contributor

djsly commented Nov 1, 2019

It's working well

k8s-collection          1               Fri Nov  1 14:30:22 2019        DEPLOYED        sumologic-0.10.5-alpha          0.10.5-alpha    sumologic 

I updated our helm_repo_url with the /alpha and I can install 0.10.5-alpha!

thanks @rvmiller89

@rvmiller89
Copy link
Contributor Author

@djsly I'll be deleting this /alpha shortly, the team decided against storing the alpha builds separately. Please hold on using the alpha builds.

@rvmiller89
Copy link
Contributor Author

rvmiller89 commented Nov 1, 2019

@djsly we just set up the CI to generate an alpha helm chart release for every alpha build. They're hosted at the same URL as before (the /alpha URL will be going away soon).

To use an alpha helm chart release (and alpha image), you can do:

$ helm repo update
$ helm install sumologic/sumologic --name collection --namespace sumologic -f values.yaml --set=image.tag=0.10.10-alpha --version=0.10.10-alpha

The first version available is 0.10.10-alpha.

@djsly
Copy link
Contributor

djsly commented Nov 2, 2019 via email

@rvmiller89
Copy link
Contributor Author

@djsly just wanted to let you know we put together a small doc on the alpha releases, please let us know if you think of anything worth adding: https://github.com/SumoLogic/sumologic-kubernetes-collection/blob/master/deploy/docs/Alpha_Release_Guide.md

@djsly
Copy link
Contributor

djsly commented Nov 26, 2019 via email

@perk-sumo perk-sumo deleted the rmiller-gen-alpha-helm-chart branch May 21, 2020 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants