Skip to content

fix(CI): Reduce test load & speed up tests#2567

Merged
sentrivana merged 32 commits into
masterfrom
ivana/reduce-py-versions
Dec 11, 2023
Merged

fix(CI): Reduce test load & speed up tests#2567
sentrivana merged 32 commits into
masterfrom
ivana/reduce-py-versions

Conversation

@sentrivana

@sentrivana sentrivana commented Dec 4, 2023

Copy link
Copy Markdown
Contributor
  • don't run every integration test suite separately; group them and run the groups in serial
  • reduce the number of Python versions we test against
    • always test against lowest and highest supported
    • always test against 3.11 and 3.12 if supported
    • if it makes sense, pick some versions in between
  • remove the retry action; we can apply it selectively later on to test suites that often fail; not every single one has to be rerun automatically

Once this has been reviewed it'll need some repo settings changes (marking different jobs as required). Right now it's complaining about "expected checks" which refers to the old "All <framework> tests passed or skipped" checks that are now obsolete.

@sentrivana sentrivana changed the title Reduce amount of py versions to test against Reduce test load Dec 5, 2023
@sentrivana sentrivana changed the title Reduce test load fix(CI): Reduce test load & speed up tests Dec 5, 2023
@sentrivana
sentrivana marked this pull request as ready for review December 5, 2023 09:13
@sentrivana
sentrivana marked this pull request as draft December 5, 2023 13:42
@sentrivana
sentrivana marked this pull request as ready for review December 5, 2023 15:14

@antonpirker antonpirker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

great change!

@sentrivana
sentrivana merged commit 4108662 into master Dec 11, 2023
@sentrivana
sentrivana deleted the ivana/reduce-py-versions branch December 11, 2023 09:36
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