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

Move release automation to Github actions #415

Merged
merged 5 commits into from
Oct 5, 2020

Conversation

yaelharel
Copy link
Contributor

@yaelharel yaelharel commented Sep 24, 2020

For local development and commits to main, we derive the lifecycle version from git describe --tags.
For new commits to release branches, we derive the lifecycle version from the branch name.
New commits to main and release branches will trigger a new lifecycle image.
After publishing new releases, the lifecycle image will be re-tagged.
PRs to any branch won't trigger a new lifecycle image.

Signed-off-by: Natalie Arellano narellano@vmware.com

This PR should fix issue #389

@yaelharel yaelharel requested a review from a team as a code owner September 24, 2020 19:49
@yaelharel yaelharel force-pushed the fix/389-github-actions-release branch from 054ee00 to 0dc4ca0 Compare September 24, 2020 19:52
Makefile Show resolved Hide resolved
})
.then(workflow_runs_result => {
let workflow_runs = workflow_runs_result.data.workflow_runs
.filter(run => run.conclusion === "success");
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We plan to additionally add a check that the SHA is correct.

Copy link
Member

@dfreilich dfreilich left a comment

Choose a reason for hiding this comment

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

Party

.github/workflows/build.yml Show resolved Hide resolved
.github/workflows/build.yml Outdated Show resolved Hide resolved
.github/workflows/build.yml Outdated Show resolved Hide resolved
.github/workflows/draft-release.yml Outdated Show resolved Hide resolved
.github/workflows/draft-release.yml Outdated Show resolved Hide resolved
.github/workflows/draft-release.yml Show resolved Hide resolved
.github/workflows/build.yml Outdated Show resolved Hide resolved
tools/packager/main.go Outdated Show resolved Hide resolved
tools/version/main.go Show resolved Hide resolved
@natalieparellano natalieparellano force-pushed the fix/389-github-actions-release branch from aef5782 to 55ec216 Compare October 2, 2020 20:26
@natalieparellano
Copy link
Member

@ekcasey we think this should be good to go - but we'd like to run the acceptance on our fork one more time (likely on Monday).

Yael Harel added 3 commits October 2, 2020 16:33
For local development and commits to main, we derive the lifecycle version
from `git describe --tags`.
For new commits to release branches, we derive the lifecycle version
from the branch name.
New commits to main and release branches will trigger a new lifecycle image.
After publishing new releases, the lifecycle image will be re-tagged.
PRs to any branch won't trigger a new lifecycle image.

Signed-off-by: Natalie Arellano <narellano@vmware.com>
Signed-off-by: Yael Harel <yharel@vmware.com>
Signed-off-by: Natalie Arellano <narellano@vmware.com>
- update github-script to the latest version v3.0.0
- refactor packager
- publish the linux and windows lifecycle images after all tests passed (on ubuntu)
- determine download urls for linux and windows in one step
- check that the head sha of the last successful workflow is the last sha

Signed-off-by: Yael Harel <yharel@vmware.com>
Signed-off-by: Natalie Arellano <narellano@vmware.com>
@natalieparellano natalieparellano force-pushed the fix/389-github-actions-release branch from 55ec216 to eb5db1a Compare October 2, 2020 20:34
Without clean, the artifact will be what was built from the acceptance tests
(with a test version) since the source code didn't change between builds.

Signed-off-by: Natalie Arellano <narellano@vmware.com>
Signed-off-by: Natalie Arellano <narellano@vmware.com>
@ekcasey ekcasey merged commit 4f48d5a into main Oct 5, 2020
@ekcasey ekcasey deleted the fix/389-github-actions-release branch October 5, 2020 21:06
@ekcasey ekcasey added this to the lifecycle-0.9.2 milestone Oct 6, 2020
@ekcasey ekcasey linked an issue Oct 6, 2020 that may be closed by this pull request
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.

Move release automation to GHA
4 participants