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

Switch from larger runners to ubuntu-22.04 runner for the CI jobs #184

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

ulucinar
Copy link
Contributor

@ulucinar ulucinar commented Feb 28, 2024

Description of your changes

This PR prepares the standard-runners branch, which has been forked from the main branch, to switch to the ubuntu-22.04 job runner from the larger runners that we currently employ. This branch will be available until we need it (until we move all the official providers under to the crossplane-contrib org. We've moved https://github.com/upbound/provider-azuread repo under to the crossplane-contrib organization and there are other official providers that are still under the upbound org. The idea is to temporarily use this branch for the providers as we move them to the crossplane-contrib organization and to test the ones that we have not yet moved with the standard runners.

This will allow us to gradually transition to the standard runners in the official provider repos.
It will also allow us to check whether any CI jobs need larger runners in the official provider repositories.

A relevant PR is: #135

I have:

  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Please see: crossplane-contrib/provider-upjet-azuread#106

Signed-off-by: Alper Rifat Ulucinar <ulucinar@users.noreply.github.com>
Copy link
Member

@sergenyalcin sergenyalcin left a comment

Choose a reason for hiding this comment

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

Thanks @ulucinar LGTM! Let's try these machines in providers.

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