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

Split stress test and BVT test #3801

Merged
merged 2 commits into from
Aug 10, 2023
Merged

Split stress test and BVT test #3801

merged 2 commits into from
Aug 10, 2023

Conversation

csujedihy
Copy link
Contributor

@csujedihy csujedihy commented Aug 10, 2023

Description

Artifacts uploaded in a GitHub workflow cannot be accessed before the entire workflow has completed. Split stress and BVT so we can get to the dumps generated from stress sooner.

Remove workflow_dispatch from reusable worfklows because they are not supposed to be run directly.

Testing

CI

@codecov
Copy link

codecov bot commented Aug 10, 2023

Codecov Report

Merging #3801 (8c5f239) into main (41f0aa1) will decrease coverage by 0.36%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3801      +/-   ##
==========================================
- Coverage   77.40%   77.05%   -0.36%     
==========================================
  Files          56       56              
  Lines       15632    15632              
==========================================
- Hits        12100    12045      -55     
- Misses       3532     3587      +55     

see 11 files with indirect coverage changes

nibanks
nibanks previously approved these changes Aug 10, 2023
@csujedihy csujedihy marked this pull request as ready for review August 10, 2023 00:57
@csujedihy csujedihy requested a review from a team as a code owner August 10, 2023 00:57
@csujedihy csujedihy merged commit 6d98536 into main Aug 10, 2023
420 of 422 checks passed
@csujedihy csujedihy deleted the huanyi/split-stress-bvt branch August 10, 2023 01:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants