Skip to content

Development: Configure server tests job to run on self-hosted GitHub runners #10724

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

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

ignacio-gn
Copy link
Member

Checklist

General

→ this pull request doesn't make any changes to the application or its deployment, as it is only related to the configuration of the test workflow

Server

No changes

Client

No changes

Changes affecting Programming Exercises

No changes

Motivation and Context

The runtime of the Test workflow should be able to be improved by running the workflow in a host with more resources.

Description

  • On the infrastructure's side, a pool of around 20 large self-hosted GitHub runners was configured with the requirements to process Artemis-related jobs and to be available on the ls1intum organization.
  • On the Artemis codebase, the runs-on label of the server-tests job in .github/workflow/test.yml was changed to aet-large-ubuntu.

Steps for Testing

Not applicable

Exam Mode Testing

Not applicable

Testserver States

Not applicable

Review Progress

Performance Review

Not applicable

Code Review

Not applicable

Manual Tests

Not applicable

Exam Mode Test

Not applicable

Performance Tests

Not applicable

Test Coverage

Not applicable

Screenshots

Not applicable

@ignacio-gn ignacio-gn requested a review from a team as a code owner April 22, 2025 13:14
@github-project-automation github-project-automation bot moved this to Work In Progress in Artemis Development Apr 22, 2025
coderabbitai[bot]
coderabbitai bot previously approved these changes Apr 22, 2025
@ignacio-gn
Copy link
Member Author

ignacio-gn commented Apr 22, 2025

  • the timeouts have to be re-checked
  • the configuration of client-test should be defined

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Work In Progress
Development

Successfully merging this pull request may close these issues.

2 participants