-
Notifications
You must be signed in to change notification settings - Fork 51
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
Updates to /ci
behavior
#3772
Updates to /ci
behavior
#3772
Conversation
/ci |
build 4867 CANCELLED. |
build 4869 SUCCEEDED. |
Test workflow 4206 is started. |
build 4870 SUCCEEDED. |
build 4872 SUCCEEDED. |
/ci |
1 similar comment
/ci |
PAX build 4940 SUCCEEDED. |
Test workflow 4252 is started. |
Signed-off-by: MarkAckert <[email protected]>
Signed-off-by: MarkAckert <[email protected]>
Signed-off-by: MarkAckert <[email protected]>
Signed-off-by: MarkAckert <[email protected]>
Signed-off-by: MarkAckert <[email protected]>
Signed-off-by: MarkAckert <[email protected]>
Signed-off-by: MarkAckert <[email protected]>
Signed-off-by: 1000TurquoisePogs <[email protected]> Signed-off-by: MarkAckert <[email protected]>
This reverts commit 62ee455. Signed-off-by: MarkAckert <[email protected]>
Signed-off-by: MarkAckert <[email protected]>
dd4469a
to
33e77ab
Compare
build 5028 SUCCEEDED. |
…dates Signed-off-by: MarkAckert <[email protected]>
build 5029 SUCCEEDED. |
Test workflow 4307 is started. |
Test workflow 4308 is started. |
Signed-off-by: MarkAckert <[email protected]>
Signed-off-by: MarkAckert <[email protected]>
Signed-off-by: MarkAckert <[email protected]>
Signed-off-by: MarkAckert <[email protected]>
Fix PSWI build trigger (#3773) * fix conditional Signed-off-by: MarkAckert <[email protected]> * clean up description Signed-off-by: MarkAckert <[email protected]> * context dump for debugging issue trigger Signed-off-by: MarkAckert <[email protected]> --------- Signed-off-by: MarkAckert <[email protected]>
build 5030 SUCCEEDED. |
Test workflow 4309 is started. |
Signed-off-by: MarkAckert <[email protected]>
build 5041 SUCCEEDED. |
build 5042 CANCELLED. |
Signed-off-by: MarkAckert <[email protected]>
Signed-off-by: MarkAckert <[email protected]>
build 5043 is started, please wait... |
build 5044 SUCCEEDED. |
Test workflow 4314 is started. |
Signed-off-by: MarkAckert <[email protected]>
This PR tests and iterates on previously contributed
/ci
comment behavior, as it couldn't be tested prior to merge into a default branch.After testing, this PR makes the following changes:
/ci
comment trigger into a new workflow file. This is necessary (a) to avoid running a lot of emptyZowe Build and Packaging
runs, which creates a lot of noise for an important workflow, and (b) to fix issues with triggering workflows from the default branch - theissue_comment
trigger always hits against the default branch, so the build triggered would use the default branch as well. This split allows the comment-trigger workflow to determine the proper headRef using the issue number (a.k.a PR number) , and then calls Build and Packaging with that ref.build-packaging.yml
to support this new workflow_dispatch call pattern so everything runs like you'd expect (post a status comment to PR thread, use proper ref for build, respect labels, etc.).v2.x/rc
as part of a PSWI catch up release; this option is disabled by default and should only be triggered manually by the systems squad in rare cases.