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

Consume the reusable workflows from upbound/uptest@standard-runners #106

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

ulucinar
Copy link
Collaborator

@ulucinar ulucinar commented Feb 28, 2024

Description of your changes

Relevant PR: upbound/official-providers-ci#184

The uptest, lint & publish-artifacts jobs are currently failing because they are assigned to runners that are unavailable for this organization. This PR switches them to the standard runners.

I have:

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

How has this code been tested

The PR pipeline (from the upbound/uptest/.github/workflows/provider-ci.yml reusable workflow) has been tested with this PR. Unfortunately, the e2e pipeline (which runs uptest) will be tested once these changes are in the main branch (in #105).

- 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.

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

@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!

Copy link
Collaborator

@turkenf turkenf left a comment

Choose a reason for hiding this comment

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

Thank you @ulucinar, LGTM.

@ulucinar ulucinar merged commit 5b88f6e into crossplane-contrib:main Feb 28, 2024
9 checks passed
@ulucinar ulucinar deleted the no-large-runners branch February 28, 2024 16:30
Copy link

Successfully created backport PR #107 for release-0.15.

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.

None yet

3 participants