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 test case R00001 #1068

Merged
merged 1 commit into from
Dec 28, 2023
Merged

Add test case R00001 #1068

merged 1 commit into from
Dec 28, 2023

Conversation

bzsuni
Copy link
Collaborator

@bzsuni bzsuni commented Dec 18, 2023

  1. Add test R00001 and R00002

  2. Fix Night CI 2023-12-20: Failed #1077
    use:

Eventually(ctx, func() error {
  return common.LabelNodes(ctx, cli, egNodes, labels)
}).WithTimeout(time.Second * 6).WithPolling(time.Second * 2).Should(Succeed())

@bzsuni bzsuni added the pr/not-ready not ready for merging label Dec 18, 2023
@bzsuni bzsuni requested a review from weizhoublue as a code owner December 18, 2023 12:11
Copy link

codecov bot commented Dec 18, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b5b60c9) 60.89% compared to head (fac11d1) 60.89%.
Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1068   +/-   ##
=======================================
  Coverage   60.89%   60.89%           
=======================================
  Files          17       17           
  Lines        3025     3025           
=======================================
  Hits         1842     1842           
  Misses        935      935           
  Partials      248      248           
Flag Coverage Δ
unittests 60.89% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@bzsuni bzsuni force-pushed the bz/e2e/R00001 branch 22 times, most recently from 3384e4d to f418cba Compare December 21, 2023 11:01
@bzsuni bzsuni requested a review from lou-lan December 21, 2023 11:17
@bzsuni bzsuni added release/none should not recoded in new release kind/e2e and removed pr/not-ready not ready for merging labels Dec 21, 2023
@bzsuni bzsuni added the pr/not-ready not ready for merging label Dec 21, 2023
@weizhoublue weizhoublue temporarily deployed to commit December 28, 2023 06:08 Inactive
@bzsuni bzsuni force-pushed the bz/e2e/R00001 branch 4 times, most recently from bbacfcc to 366245f Compare December 28, 2023 10:36
Signed-off-by: bzsuni <[email protected]>
@bzsuni bzsuni removed the pr/not-ready not ready for merging label Dec 28, 2023
@lou-lan lou-lan merged commit ff95cae into main Dec 28, 2023
61 of 64 checks passed
@lou-lan lou-lan deleted the bz/e2e/R00001 branch December 28, 2023 12:09
bzsuni added a commit that referenced this pull request Dec 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/e2e release/none should not recoded in new release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Night CI 2023-12-20: Failed
3 participants