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

Transition Azure Pipelines to Github Actions #1287

Open
wants to merge 42 commits into
base: develop
Choose a base branch
from

Conversation

chapman39
Copy link
Contributor

@chapman39 chapman39 commented Dec 6, 2024

Migrates Azure Pipelines to GitHub Actions.

Some benefits:

  • You can now cancel a job at any point.
  • More storage (14GB instead of Azure's 10GB) (sources: azure, github actions).
  • Jobs will no longer run on push. They will only run if there is an update to a pull request.
  • Previous of a PR jobs are automatically canceled if an update has been made.
  • Fails fast. If one job fails, the entire pipeline will cancel.
  • You can re-try failed jobs.

Things todo that I can't do:

fixes #1265

@chapman39 chapman39 self-assigned this Dec 10, 2024
@chapman39 chapman39 added the CI Continuous Integration label Dec 10, 2024
@chapman39 chapman39 marked this pull request as ready for review December 10, 2024 17:50
@chapman39 chapman39 marked this pull request as draft December 10, 2024 17:51
@chapman39
Copy link
Contributor Author

chapman39 commented Dec 10, 2024

alternative to hacky solution for setting docker image name vars: https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/store-information-in-variables#creating-configuration-variables-for-a-repository

but this would require users to access admin settings every time docker images are updated.

@chapman39 chapman39 marked this pull request as ready for review December 10, 2024 20:32
@chapman39
Copy link
Contributor Author

I'm not sure how to remove the azure pipeline LLNL.serac from the ci checks.

Copy link
Collaborator

@tupek2 tupek2 left a comment

Choose a reason for hiding this comment

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

Chris should weigh in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Switch Azure CI to GitHub Actions
3 participants