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

Add parallel test running #2486

Closed
wants to merge 1 commit into from
Closed

Conversation

vvarg229
Copy link
Contributor

@vvarg229 vvarg229 commented Aug 9, 2023

Both sanity tests and a full test run are run in parallel. This will allow to get the result and a link to the report in NeoFS after 30 minutes after creating a PR or a commit in the repo and get the result of all tests in a few hours.
This will be useful for developers - now they will be able to get test results before changes are pushed to the master.

Both sanity tests and a full test run are run in parallel.
This will allow to get the result and a link to the report in NeoFS after 30
minutes after creating a PR or a commit in the repo and get the result of all
tests in a few hours.
This will be useful for developers - now they will be able to get test results
before changes are pushed to the master.

Signed-off-by: Oleg Kulachenko <[email protected]>
@codecov
Copy link

codecov bot commented Aug 9, 2023

Codecov Report

Merging #2486 (b98fc99) into master (a7541ff) will increase coverage by 0.00%.
Report is 7 commits behind head on master.
The diff coverage is n/a.

❗ Current head b98fc99 differs from pull request most recent head 220d9c4. Consider uploading reports for the commit 220d9c4 to get more accurate results

@@           Coverage Diff           @@
##           master    #2486   +/-   ##
=======================================
  Coverage   29.24%   29.25%           
=======================================
  Files         399      399           
  Lines       30368    30368           
=======================================
+ Hits         8881     8883    +2     
+ Misses      20751    20749    -2     
  Partials      736      736           

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@vvarg229
Copy link
Contributor Author

vvarg229 commented Aug 9, 2023

For example, here in my fork is the operation of multiple PRs.
There is no long wait for runners on that many jobs at the same time.

image

Copy link
Member

@roman-khimov roman-khimov left a comment

Choose a reason for hiding this comment

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

Let's try splitting into gw/non-gw parts (roughly equal now, each takes ~1h to complete).

@vvarg229 vvarg229 marked this pull request as draft August 16, 2023 11:06
@roman-khimov
Copy link
Member

Not relevant for now, we're OK as is.

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