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

Finish migrating packaging pipeline to Github Actions. #753

Merged
merged 31 commits into from
Aug 17, 2020

Conversation

bcipriano
Copy link
Collaborator

Link the Issue(s) this Pull Request is related to.
#715

Summarize your change.
This migrates all remaining build steps from the Azure file to the Github Actions pipeline. All artifacts now get uploaded to S3, to be used later by the release pipeline when creating the release.

@bcipriano
Copy link
Collaborator Author

@gregdenton @larsbijl @DiegoTavares Ping, could I get a review on this? Nothing new here, just migrating existing CI logic to Github Actions.

@larsbijl
Copy link
Contributor

How can I run the action @bcipriano to test?

@bcipriano
Copy link
Collaborator Author

This pipeline relies on secrets which are stored at the ASWF level, so I don't think it will run in a fork or as a PR-driven action. In general that's ok because the pipeline is really only meant to run when new commits are merged into master.

As a committer you should be able to do the following:

  • Check out my branch, clone it to a new uniquely-named branch, then push that new branch to the upstream repo (i.e. AcademySoftwareFoundation/OpenCue, as opposed to your personal fork).
  • Edit the on > push > branches property of the pipeline (line 6) to run on commits to your branch, rather than master as it does now.
  • Any time you git push your branch to the upstream repo, it should trigger the pipeline to run. It'll show up on the Actions page.

I used a setup like this to test the pipeline while I was working on it. See for example https://github.com/AcademySoftwareFoundation/OpenCue/actions/runs/199732120.

@larsbijl
Copy link
Contributor

LGTM!

@bcipriano bcipriano merged commit 113a57f into master Aug 17, 2020
@bcipriano bcipriano deleted the more-release-pipeline branch August 17, 2020 17:51
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.

None yet

2 participants