Skip to content
This repository has been archived by the owner on Jun 10, 2019. It is now read-only.

bootstrap-vz fails to unmount /dev if interrupted #292

Open
nbraud opened this issue Feb 24, 2016 · 5 comments
Open

bootstrap-vz fails to unmount /dev if interrupted #292

nbraud opened this issue Feb 24, 2016 · 5 comments

Comments

@nbraud
Copy link
Contributor

nbraud commented Feb 24, 2016

I interrupted a bootstrap-vz run just after it bind-mounted /dev into the VM's rootfs, and it failed to clean it up: see log 20160224161357_jessie-vagrant.log.

@andsens
Copy link
Owner

andsens commented Feb 24, 2016

I think you actually managed to interrupt it during the mounting process, meaning bootstrap-vz hasn't had the chance to flag the mount as completed, which in turn causes it to not unmount it during rollback.
I'm closing this in favor of #292.

@andsens andsens closed this as completed Feb 24, 2016
@nbraud
Copy link
Contributor Author

nbraud commented Feb 24, 2016

@andsens My point was that this incident exposed a race condition, regardless of #293.

@nbraud
Copy link
Contributor Author

nbraud commented Feb 29, 2016

@andsens I guess you don't think this is worth re-opening?

@andsens
Copy link
Owner

andsens commented Feb 29, 2016

Oh! I see what you mean now (reopening).
Hm, so the problem is we manage the state of a mount separately from what is actually happening on the system, which can be fixed. Either that, or we register a mount as successful before actually mounting it, and then simply make the unmounting operation resistant against unmounts failing specifically because something isn't mounted.

I like the solution of fixing the synchronization between bootstrap-vz's state and the system a lot more.

Any other ideas?

@andsens andsens reopened this Feb 29, 2016
@nbraud
Copy link
Contributor Author

nbraud commented Mar 1, 2016

Well, it's not a part of bootstrap-vz I'm familiar with, at all.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants