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: bump macos version to 13 #18609
base: master
Are you sure you want to change the base?
ci: bump macos version to 13 #18609
Conversation
Hi @ComradeProgrammer. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: ComradeProgrammer The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Can one of the admins verify this patch? |
@spowelljr @medyagh could you please type an ok-to-test? I think it may work |
The additional tags are for self-hosted runners where you specify the tags on the runner itself, I don't think this is going to work. |
I see. I shall try something else later |
d1cce35
to
3dfd18a
Compare
I think this time it might work. This workflow will try to use gh commandline to check the log of its depedent PR workflow to see if there are warnings about incorrect architecture( If such error is found in the log, then it will try to fetch the jobid of I created a test repo and used a simplified version of it, and I think it works. However, it seems that this change cannot take effect before it is merged. |
@@ -0,0 +1,35 @@ | |||
# This is a basic workflow to help you get started with Actions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add better comments and explain what is this and why we need it
cool, lets see if you can make the Test Pass too ? |
3dfd18a
to
6e57e00
Compare
ci: bump macos version to 13