-
Notifications
You must be signed in to change notification settings - Fork 85
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
Failover state before initiating restores in pra test #2144
Conversation
Hello williamlardier,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
e0835ab
to
865be0d
Compare
Request integration branchesWaiting for integration branch creation to be requested by the user. To request integration branches, please comment on this pull request with the following command:
Alternatively, the |
tests/ctst/steps/pra.ts
Outdated
await this.zenkoDrCtl?.failover({ | ||
sinkZenkoNamespace: 'default', | ||
wait: true, | ||
timeout: '6m', |
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.
Should'nt we make the timeout a bit lower in the command than in the timeout for ctst ? so that we see the command fail before the step and we can have more logs..
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.
I'll update all functions yes
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.
Done -> 9208878
# Restore on DR site | ||
When i restore object "obj2-1" for 2 days on "DR" site | ||
Then object "obj2-1" should "" be "restored" and have the storage class "e2e-cold" on "DR" site | ||
And object "obj2-1" should "" be "transitioned" and have the storage class "e2e-cold" on "Primary" site | ||
|
||
# Switch to failback | ||
When I resume operations for the DR | ||
Then the DR sink should be in phase "Running" |
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.
we should test that the object is expired (on DR site) during failback (may need to increase the restore duration to ensure we don't succeed thanks to race conditions)
9208878
to
170d48d
Compare
/create_integration_branches |
Integration data createdI have created the integration data for the additional destination branches.
The following branches will NOT be impacted:
You can set option
The following options are set: create_integration_branches |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
The following options are set: create_integration_branches |
/approve |
Build failedThe build for commit did not succeed in branch w/2.10/improvement/ZENKO-4887. The following options are set: approve, create_integration_branches |
Build failedThe build for commit did not succeed in branch w/2.8/improvement/ZENKO-4887. The following options are set: approve, create_integration_branches |
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue ZENKO-4887. Goodbye williamlardier. The following options are set: approve, create_integration_branches |
Switch to failover before restoring object on the DR site.