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

github: add mcs manifest deployment #799

Merged
merged 1 commit into from
Jul 22, 2024
Merged

github: add mcs manifest deployment #799

merged 1 commit into from
Jul 22, 2024

Conversation

lsf37
Copy link
Member

@lsf37 lsf37 commented Jul 19, 2024

Add deployment of the mcs-devel.xml manifest to mcs.xml.

The pattern is to use the branch name in github.ref as a guard in the initial code freeze action, and then make any subsequent actions depend on either code or mcs-code, respectively.

All of this lives on the master branch (but does trigger on pushes to rt), so that it can in the future also trigger on manifest updates from repository_dispatch events.

Currently the latter does not happen yet, because github.ref will be refs/head/master for repository_dispatch events, so only the main deployment job runs for those.

Add deployment of the mcs-devel.xml manifest to mcs.xml.

The pattern is to use the branch name in `github.ref` as a guard in the
initial code freeze action, and then make any subsequent actions depend
on either `code` or `mcs-code`, respectively.

All of this lives on the master branch (but does trigger on pushes to
`rt`), so that it can in the future also trigger on manifest updates
from `repository_dispatch` events.

Currently the latter does not happen yet, because `github.ref` will be
`refs/head/master` for `repository_dispatch` events, so only the main
deployment job runs for those.

Signed-off-by: Gerwin Klein <[email protected]>
@lsf37 lsf37 added the CI continuous integration and testing label Jul 19, 2024
@lsf37 lsf37 requested a review from corlewis July 19, 2024 06:56
@lsf37 lsf37 self-assigned this Jul 19, 2024
@lsf37
Copy link
Member Author

lsf37 commented Jul 19, 2024

I'm reasonably certain this won't break manifest deployment for the master branch, but the first time this will really be properly tested for the rt branch is when we merge this there.

@lsf37 lsf37 merged commit 5dd35d3 into master Jul 22, 2024
13 of 14 checks passed
@lsf37 lsf37 deleted the mcs-deploy branch July 22, 2024 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI continuous integration and testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants