Skip to content

Use ubuntu-22.04 runner instead of buildjet for the UI tests#9053

Merged
jmartinesp merged 2 commits intodevelopfrom
ci/remove-buildjet
Jul 10, 2025
Merged

Use ubuntu-22.04 runner instead of buildjet for the UI tests#9053
jmartinesp merged 2 commits intodevelopfrom
ci/remove-buildjet

Conversation

@jmartinesp
Copy link
Member

Type of change

  • Feature
  • Bugfix
  • Technical
  • Other :

Content

  • Remove the BuildJet runner from our CI jobs.
  • Make the UI test one run only after merging PRs or manually for a branch, since it's not really maintained.

Motivation and context

We want to get rid of the BuildJet runners.

Checklist

@jmartinesp jmartinesp marked this pull request as ready for review July 10, 2025 09:45
Copy link
Member

@bmarty bmarty left a comment

Choose a reason for hiding this comment

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

Thanks, just one question

name: UI Tests (Synapse)
needs: should-i-run
runs-on: buildjet-4vcpu-ubuntu-2204
runs-on: ubuntu-22.04
Copy link
Member

Choose a reason for hiding this comment

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

Why not using ubuntu-latest?

Copy link
Member Author

Choose a reason for hiding this comment

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

We've used -latest in the past and sometimes that led to unintended dependency upgrades of the software included in the runners, so I thought it would be better to keep the same version.

tests:
name: Runs all tests
runs-on: buildjet-4vcpu-ubuntu-2204
runs-on: ubuntu-22.04
Copy link
Member

Choose a reason for hiding this comment

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

Same remark

@jmartinesp jmartinesp merged commit cba1bb7 into develop Jul 10, 2025
10 of 12 checks passed
@jmartinesp jmartinesp deleted the ci/remove-buildjet branch July 10, 2025 13:41
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