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 e2e test cases #949

Merged
merged 7 commits into from
Apr 7, 2022
Merged

Split e2e test cases #949

merged 7 commits into from
Apr 7, 2022

Conversation

fgksgf
Copy link
Member

@fgksgf fgksgf commented Apr 5, 2022

Type of change:

Split e2e test cases to reduce overall time consumption.

What this PR does / why we need it:

Close #932

Pre-submission checklist:

  • Did you explain what problem does this PR solve? Or what new features have been added?
  • Have you added corresponding test cases?
  • Have you modified the corresponding document?
  • Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

@codecov-commenter
Copy link

codecov-commenter commented Apr 5, 2022

Codecov Report

Merging #949 (a488402) into master (de33d05) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #949      +/-   ##
==========================================
- Coverage   31.76%   31.75%   -0.01%     
==========================================
  Files          73       72       -1     
  Lines        7936     7935       -1     
==========================================
- Hits         2521     2520       -1     
  Misses       5139     5139              
  Partials      276      276              
Impacted Files Coverage Δ
test/e2e/e2e.go

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update de33d05...a488402. Read the comment docs.

@tao12345666333
Copy link
Member

2022-04-06 14-54-04屏幕截图

LGTM! Thanks!!! The CI time cut in half

Copy link
Member

@tao12345666333 tao12345666333 left a comment

Choose a reason for hiding this comment

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

Thanks!
LGTM

test/e2e/suite-features/global_rule.go Outdated Show resolved Hide resolved
@tao12345666333
Copy link
Member

Since #942 was recently merged, are you free to help resolve the conflict? @fgksgf

@fgksgf
Copy link
Member Author

fgksgf commented Apr 7, 2022

Since #942 was recently merged, are you free to help resolve the conflict? @fgksgf

No problem.

@tao12345666333 tao12345666333 merged commit 4bdc947 into apache:master Apr 7, 2022
@fgksgf fgksgf deleted the split-e2e branch April 7, 2022 05:41
@fgksgf fgksgf mentioned this pull request Apr 8, 2022
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI: Split e2e test cases to reduce overall time consumption
4 participants