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

Updates to /ci behavior #3772

Merged
merged 20 commits into from
Apr 16, 2024

Conversation

MarkAckert
Copy link
Member

@MarkAckert MarkAckert commented Mar 22, 2024

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:

  • Splits out the /ci comment trigger into a new workflow file. This is necessary (a) to avoid running a lot of empty Zowe 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 - the issue_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.
  • Updates 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.).
  • Updates PSWI builds to allow for PSWI builds using existing PAX files. This was cherry-picked from changes that went into 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.

@MarkAckert MarkAckert added Build: PAX Used in CI Test: None Used in CI labels Mar 22, 2024
@MarkAckert
Copy link
Member Author

MarkAckert commented Mar 22, 2024

/ci

Copy link

github-actions bot commented Mar 26, 2024

build 4867 CANCELLED.
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8437093405

Copy link

github-actions bot commented Mar 26, 2024

build 4869 SUCCEEDED.
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8437292117

Copy link

github-actions bot commented Mar 26, 2024

Test workflow 4206 is started.
Running install test: Convenience Pax
The zowe artifact being used by this test workflow: libs-snapshot-local/org/zowe/2.15.1-PR-3772/zowe-2.15.1-pr-3772-4869-20240326141005.pax
Running on machine: zzow08
Result: SUCCESS
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8437412919

Copy link

github-actions bot commented Mar 26, 2024

build 4870 SUCCEEDED.
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8439982287

Copy link

github-actions bot commented Mar 26, 2024

build 4872 SUCCEEDED.
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8440260931

@MarkAckert
Copy link
Member Author

MarkAckert commented Mar 26, 2024

/ci

1 similar comment
@MarkAckert
Copy link
Member Author

/ci

Copy link

github-actions bot commented Apr 2, 2024

PAX build 4940 SUCCEEDED.
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8526518476

Copy link

github-actions bot commented Apr 2, 2024

Test workflow 4252 is started.
Running install test: Convenience Pax
The zowe artifact being used by this test workflow: libs-snapshot-local/org/zowe/2.15.1-PR-3772/zowe-2.15.1-pr-3772-4940-20240402170111.pax
Running on machine: zzow07
Result: SUCCESS
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8526636813

MarkAckert and others added 10 commits April 15, 2024 16:31
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]>
@MarkAckert MarkAckert force-pushed the user/markackert/ci-label-behavior-updates branch from dd4469a to 33e77ab Compare April 15, 2024 20:31
Copy link

github-actions bot commented Apr 15, 2024

build 5028 SUCCEEDED.
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8695689374

Copy link

github-actions bot commented Apr 15, 2024

build 5029 SUCCEEDED.
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8695735438

Copy link

github-actions bot commented Apr 15, 2024

Test workflow 4307 is started.
Running install test: Convenience Pax
The zowe artifact being used by this test workflow: libs-snapshot-local/org/zowe/2.15.1-PR-3772/zowe-2.15.1-pr-3772-5028-20240415203643.pax
Running on machine: zzow07
Result: SUCCESS
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8695775953

Copy link

github-actions bot commented Apr 15, 2024

Test workflow 4308 is started.
Running install test: Convenience Pax
The zowe artifact being used by this test workflow: libs-snapshot-local/org/zowe/2.15.1-PR-3772/zowe-2.15.1-pr-3772-5029-20240415204159.pax
Running on machine: zzow06
Result: SUCCESS
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8695830290

MarkAckert and others added 5 commits April 15, 2024 16:45
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]>
Copy link

github-actions bot commented Apr 15, 2024

build 5030 SUCCEEDED.
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8695883395

Copy link

github-actions bot commented Apr 15, 2024

Test workflow 4309 is started.
Running install test: Convenience Pax
The zowe artifact being used by this test workflow: libs-snapshot-local/org/zowe/2.15.1-PR-3772/zowe-2.15.1-pr-3772-5030-20240415210343.pax
Running on machine: zzow08
Result: SUCCESS
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8696059038

Copy link

github-actions bot commented Apr 16, 2024

build 5041 SUCCEEDED.
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8710407415

Copy link

github-actions bot commented Apr 16, 2024

build 5042 CANCELLED.
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8710420407

Signed-off-by: MarkAckert <[email protected]>
Signed-off-by: MarkAckert <[email protected]>
Copy link

build 5043 is started, please wait...
Estimated build time: 8 mins. Check back around:
2:23:01 p.m. EST | 8:23:01 PM CET | 19:23:01 GMT | 11:23:01 AM PST
(This comment will get updated once build result is out)
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8710492228

Copy link

github-actions bot commented Apr 16, 2024

build 5044 SUCCEEDED.
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8710510507

Copy link

github-actions bot commented Apr 16, 2024

Test workflow 4314 is started.
Running install test: Convenience Pax
The zowe artifact being used by this test workflow: libs-snapshot-local/org/zowe/2.15.1-PR-3772/zowe-2.15.1-pr-3772-5041-20240416181411.pax
Running on machine: zzow07
Result: SUCCESS
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/8710508046

@MarkAckert MarkAckert added Build: None Used in CI and removed Build: PAX Used in CI labels Apr 16, 2024
@MarkAckert MarkAckert requested a review from struga0258 April 16, 2024 18:22
@MarkAckert MarkAckert changed the title Testing /ci behavior Updates to /ci behavior Apr 16, 2024
@MarkAckert MarkAckert merged commit 1b3f629 into v2.x/staging Apr 16, 2024
6 checks passed
@MarkAckert MarkAckert deleted the user/markackert/ci-label-behavior-updates branch April 16, 2024 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build: None Used in CI Test: None Used in CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants