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: Remove azure-based workers pipelines #442

Merged
merged 1 commit into from
Sep 10, 2024

Conversation

ldoktor
Copy link
Contributor

@ldoktor ldoktor commented Sep 9, 2024

the github free-runners seem to perform well and stable, let's disable the azure runners.

the github free-runners seem to perform well and stable, let's disable
the azure runners.

Signed-off-by: Lukáš Doktor <[email protected]>
Copy link
Member

@stevenhorsman stevenhorsman left a comment

Choose a reason for hiding this comment

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

Makes sense to me. Thanks @ldoktor!

@mythi
Copy link
Contributor

mythi commented Sep 9, 2024

is this dropping any of the targets that keeps failing for me in #429?

@stevenhorsman
Copy link
Member

Once we have two approvals I guess we should remove the az jobs from the required list and add the free runners?

@stevenhorsman
Copy link
Member

It looks like the s390x-large runner is offline and @BbolroC is on vacation for a few days, so we'll need to ignore that job too

@ldoktor
Copy link
Contributor Author

ldoktor commented Sep 9, 2024

Once we have two approvals I guess we should remove the az jobs from the required list and add the free runners?

Yep, actually let me do that now since both jobs are currently being executed and the az ones are now actually less reliable than the GH ones.

@ldoktor
Copy link
Contributor Author

ldoktor commented Sep 10, 2024

2acks and the s390x test is not related so, let me merge this.

@ldoktor ldoktor merged commit 5b5ca8d into confidential-containers:main Sep 10, 2024
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants