Skip to content

fix: add count_self_hosted_runners job to optimize e2e testing based … #1194

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 32 commits into
base: develop
Choose a base branch
from
Open

fix: add count_self_hosted_runners job to optimize e2e testing based … #1194

wants to merge 32 commits into from

Conversation

@jekabs-karklins
Copy link
Contributor

@jekabs-karklins jekabs-karklins commented Sep 23, 2025 •
edited by github-actions bot
Loading

Description

This PR optimizes the e2e testing by introducing a job to count self-hosted runners and distribute the e2e tests based on the available runners.

Motivation and Context

This change is required to improve efficiency and speed of the e2e testing process by better utilizing available resources and avoiding bottlenecks.

Changes

  1. Introduced a new job count_self_hosted_runners to count the number of idle self-hosted runners. This helps in the distribution of e2e tests.
  2. The e2e job is modified to depend on count_self_hosted_runners and the distribution of tests is based on the count of idle runners.
  3. Removed redundant repository download step in e2e tests.

How Has This Been Tested?

Fixes Jira Issue

https://jira.esss.lu.se/browse/

Depends On

Tests included/Docs Updated?

  • I have added tests to cover my changes.
  • All relevant doc has been updated

jekabskarklins and others added 10 commits September 23, 2025 19:54
fix: add count_self_hosted_runners job to optimize e2e testing based …
3bd4986 
…on available runners
fix: ensure count_self_hosted_runners job depends on build for proper…
d9b08ec 
… execution order
fix: update cypress run action to use electron browser for consistency
40e32c5
fix: remove redundant updateCall calls in Calls tests for cleaner code
00e5eec
@jekabs-karklins
Merge branch 'develop' into add-self-hosted-runners
ed613cb
Merge branch 'remove-flaky-test' into add-self-hosted-runners
82f2bfa
fix: rename idx to order for clarity in e2e test matrix and update ru…
edb530f 
…nner selection logic
fix: remove redundant logout call in PageTable component tests for cl…
559de0d 
…eaner code
Merge branch 'remove-flaky-test' into add-self-hosted-runners
bced579
fix: update technique and instrument names to use lorem.word for bett…
4a924ba 
…er consistency
@jekabs-karklins jekabs-karklins marked this pull request as ready for review October 7, 2025 13:09
@jekabs-karklins jekabs-karklins requested a review from a team as a code owner October 7, 2025 13:09
@jekabs-karklins jekabs-karklins requested review from TCMeldrum and removed request for a team October 7, 2025 13:09
@yoganandaness yoganandaness self-requested a review October 7, 2025 13:47
yoganandaness
yoganandaness approved these changes Oct 7, 2025
View reviewed changes
Copy link
Contributor

@yoganandaness yoganandaness left a comment

Choose a reason for hiding this comment

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

Seems good, although looking forward to see in action.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 4, 2025

⚠️ Automatic rebase failed due to conflicts. Please rebase manually.

2 similar comments
@github-actions
Copy link
Contributor

github-actions bot commented Nov 4, 2025

⚠️ Automatic rebase failed due to conflicts. Please rebase manually.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 4, 2025

⚠️ Automatic rebase failed due to conflicts. Please rebase manually.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 4, 2025

⚠️ Automatic rebase failed due to conflicts. Please rebase manually.

1 similar comment
@github-actions
Copy link
Contributor

github-actions bot commented Nov 4, 2025

⚠️ Automatic rebase failed due to conflicts. Please rebase manually.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2025

⚠️ Automatic rebase failed due to conflicts. Please rebase manually.

@TCMeldrum
Copy link
Contributor

TCMeldrum commented Nov 5, 2025

If we have updated the job names don't we have to update the jobs in the repos settings as well?

@github-actions
Copy link
Contributor

github-actions bot commented Nov 10, 2025

⚠️ Automatic rebase failed due to conflicts. Please rebase manually.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 21, 2025

⚠️ Automatic rebase failed due to conflicts. Please rebase manually.

1 similar comment
@github-actions
Copy link
Contributor

github-actions bot commented Nov 25, 2025

⚠️ Automatic rebase failed due to conflicts. Please rebase manually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TCMeldrum TCMeldrum TCMeldrum approved these changes

@yoganandaness yoganandaness yoganandaness approved these changes

Assignees

@yoganandaness yoganandaness

Labels

review: please

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@jekabs-karklins @TCMeldrum @yoganandaness @bolmsten @janosbabik