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

Optimize github tests #55

Open
ohdearaugustin opened this issue Feb 15, 2024 · 1 comment
Open

Optimize github tests #55

ohdearaugustin opened this issue Feb 15, 2024 · 1 comment

Comments

@ohdearaugustin
Copy link
Member

ohdearaugustin commented Feb 15, 2024

Only run the molecule test if there was a change in the role.
Currently all molecule tests will run on a push or pullrequest, even thou nothing was changed in the role.

Something like that would be nice:

on:
  push:
    paths:
    - 'folder1/**'
  pull_request:
    paths: 
    - 'folder1/**'

This will shorten the run time of the tests and lower cost of test.

Maybe also add approval for pull request. So the test are not executed automatically.

@MarkusTeufelberger
Copy link
Contributor

This is partially by design, as some roles depend on external state (e.g. newer/updated packages in repos). splitting tests by role is possible, but since tests are currently run in parallel anyways, the only speedup would be from testing faster than the current slowest role test.

Tests only run automatically for users from the mgit organization.

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

No branches or pull requests

2 participants