Skip to content

integration: Adds test for disk snapshot revert on failed deployment - #546

Open
claudiubelu wants to merge 1 commit into
cloudbase:mainfrom
claudiubelu:coveragability-increasability
Open

claudiubelu wants to merge 1 commit into
cloudbase:mainfrom
claudiubelu:coveragability-increasability

Conversation

@claudiubelu

Copy link
Copy Markdown
Member

With clone_disks=False, the deployment modifies the replica's own disks in place, so a snapshot of them is taken beforehand. If the deployment fails:

  • the snapshot is kept (its deletion task does not run on error);
  • CLEANUP_FAILED_INSTANCE_DEPLOYMENT runs;
  • RESTORE_TRANSFER_DISK_SNAPSHOTS then reverts the disks to the snapshot, so the replica is left in its pre-deployment state.

This test checks that the last two tasks run to completion.

With clone_disks=False, the deployment modifies the replica's own disks
in place, so a snapshot of them is taken beforehand. If the deployment fails:
 - the snapshot is kept (its deletion task does not run on error);
 - CLEANUP_FAILED_INSTANCE_DEPLOYMENT runs;
 - RESTORE_TRANSFER_DISK_SNAPSHOTS then reverts the disks to the snapshot,
   so the replica is left in its pre-deployment state.

This test checks that the last two tasks run to completion.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant