CA-48768: allow late cancellation of vm-import #913

Merged
merged 1 commit into from Nov 9, 2012

Conversation

Projects
None yet
3 participants
@mcclurmc
Contributor

mcclurmc commented Nov 7, 2012

There was a ten second window where a vm-import could be cancelled, but the operation would ignore the cancellation. Now we check just before completing the operation, and then disable cancellation for the task after the last checkpoint.

Signed-off-by: Mike McClurg mike.mcclurg@citrix.com

Mike McClurg
CA-48768: allow late cancellation of vm-import
Signed-off-by: Mike McClurg <mike.mcclurg@citrix.com>
@xen-git

This comment has been minimized.

Show comment Hide comment
@xen-git

xen-git Nov 7, 2012

Owner

mcclurmc/xen-api@1439206dbd1565: Build succeeded. Can merge pull request.

Owner

xen-git commented Nov 7, 2012

mcclurmc/xen-api@1439206dbd1565: Build succeeded. Can merge pull request.

@xen-git

This comment has been minimized.

Show comment Hide comment
@xen-git

xen-git Nov 9, 2012

Owner

mcclurmc/xen-api@14392062c0fbe1: Build succeeded. Can merge pull request.

Owner

xen-git commented Nov 9, 2012

mcclurmc/xen-api@14392062c0fbe1: Build succeeded. Can merge pull request.

@jonludlam

This comment has been minimized.

Show comment Hide comment
@jonludlam

jonludlam Nov 9, 2012

Owner

@xen-git Approved

Owner

jonludlam commented Nov 9, 2012

@xen-git Approved

@xen-git

This comment has been minimized.

Show comment Hide comment
@xen-git

xen-git Nov 9, 2012

Owner

mcclurmc/xen-api@14392062c0fbe1: Build succeeded. Pull request merged.

Owner

xen-git commented Nov 9, 2012

mcclurmc/xen-api@14392062c0fbe1: Build succeeded. Pull request merged.

@xen-git xen-git merged commit 1439206 into xapi-project:master Nov 9, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment