-
Notifications
You must be signed in to change notification settings - Fork 206
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
tests: add smoke test for nydusd hot upgrade #1533
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1533 +/- ##
=======================================
Coverage 61.27% 61.27%
=======================================
Files 144 144
Lines 46597 46597
Branches 44133 44133
=======================================
Hits 28550 28550
Misses 16595 16595
Partials 1452 1452 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the work!
@loheagn Can we also give a doc to explain how the failover & hot upgrade work and give the usage? For example |
@loheagn Please user rebase to update branch not |
fixed |
Signed-off-by: loheagn <[email protected]>
dd4fd1a
to
48fb346
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Relevant Issue (if applicable)
#1421
Details
failover-test
totakeover-test
as thetakeover-test
contains tests for both failover and hot upgradeTypes of changes
What types of changes does your PullRequest introduce? Put an
x
in all the boxes that apply:Checklist
Go over all the following points, and put an
x
in all the boxes that apply.