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

librbd: snap_remove should ignore -ENOENT errors #4245

Merged
2 commits merged into from Apr 27, 2015
Merged

Conversation

dillaman
Copy link

@dillaman dillaman commented Apr 1, 2015

If the attempt to deregister the snapshot from the parent
image fails with -ENOENT, ignore the error as it is safe
to assume that the child is not associated with the parent.

Fixes: #11113
Signed-off-by: Jason Dillaman dillaman@redhat.com
(cherry picked from commit cf80949)

@dillaman dillaman added this to the firefly milestone Apr 1, 2015
@loic-bot
Copy link

loic-bot commented Apr 1, 2015

SUCCESS: the output of run-make-check.sh on centos-7 for 1a8ecbc is http://paste2.org/5egDh14O

:octocat: Sent from GH.

Jason Dillaman added 2 commits April 1, 2015 15:53
get_parent_info should return -ENOENT if the image does not
have an associated parent image.

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit 21afd0e)
If the attempt to deregister the snapshot from the parent
image fails with -ENOENT, ignore the error as it is safe
to assume that the child is not associated with the parent.

Fixes: #11113
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit cf80949)
@loic-bot
Copy link

loic-bot commented Apr 1, 2015

SUCCESS: the output of run-make-check.sh on centos-7 for cea502c is http://paste2.org/3nVPBMx4

:octocat: Sent from GH.

@ghost ghost assigned xinxinsh Apr 27, 2015
ghost pushed a commit that referenced this pull request Apr 27, 2015
librbd: snap_remove should ignore -ENOENT errors

Reviewed-by: Shu, Xinxin <xinxin.shu@intel.com>
Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost merged commit 1a10a7e into firefly Apr 27, 2015
@ghost
Copy link

ghost commented Apr 27, 2015

@dillaman dillaman deleted the wip-11113-firefly branch May 18, 2015 20:35
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants