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

[ci] Backport GitHub Actions migrations from master #25200

Open
wants to merge 13 commits into
base: earlgrey_1.0.0
Choose a base branch
from

Conversation

nbdd0121 and others added 9 commits November 19, 2024 11:29
Signed-off-by: Gary Guo <[email protected]>
(cherry picked from commit 32d5ae5)
The summary uses a lot of screen space and they're not very useful.

Signed-off-by: Gary Guo <[email protected]>
(cherry picked from commit 169792a)
Signed-off-by: Gary Guo <[email protected]>
(cherry picked from commit 3e00eff)
Signed-off-by: Gary Guo <[email protected]>
(cherry picked from commit 389bd46)
Keep one enabled as a test, the rest will be enabled when
corresponding FPGA tests are migrated.

They'll keep running on push as it takes the responsibility
of caching to GCP from Azure.

Signed-off-by: Gary Guo <[email protected]>
(cherry picked from commit ced22db)
Signed-off-by: James Wainwright <[email protected]>
(cherry picked from commit 0b0aeb3)
Merged into `sw_build` since this part should be fairly quick.

Signed-off-by: James Wainwright <[email protected]>
(cherry picked from commit e8f7699)
The job itself doesn't take much time, but it needs time to wait
for license to become available.

Signed-off-by: Gary Guo <[email protected]>
(cherry picked from commit 651fd2f)
It appears that Bazel cache performance suffers when using WIF.

Signed-off-by: Gary Guo <[email protected]>
(cherry picked from commit 967144f)
Copy link
Contributor

@nbdd0121 nbdd0121 left a comment

Choose a reason for hiding this comment

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

Thanks!

@nbdd0121
Copy link
Contributor

Commit 1fc67b1 misses cherry-pick message.

@jwnrt
Copy link
Contributor Author

jwnrt commented Nov 19, 2024

Nice catch, fixed

Signed-off-by: Gary Guo <[email protected]>
(cherry picked from commit c226040)
Signed-off-by: Gary Guo <[email protected]>
(cherry picked from commit 7d2430f)
They have GitHub actions counterparts now.

Signed-off-by: Gary Guo <[email protected]>
(cherry picked from commit 77c14f7)
@jwnrt
Copy link
Contributor Author

jwnrt commented Nov 19, 2024

GitHub changes the commit hashes on merge, fixed again...

@jwnrt jwnrt marked this pull request as ready for review November 19, 2024 12:26
@nbdd0121
Copy link
Contributor

Needs #24868 first

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.

2 participants