Skip to content

Conversation

djs55
Copy link
Collaborator

@djs55 djs55 commented Apr 30, 2015

This is needed because the volumes may already have the correct
signature (e.g. from a previous run) but now contain stale data
which must be discarded.

  • When host.create creates {to,from}LVM volumes, it explicitly
    erases them.
  • Vgcreate erases the xenvmd journal volume

This relies on [djs55/mirage-block-volume#64]

Signed-off-by: David Scott dave.scott@citrix.com

This is needed because the volumes may already have the correct
signature (e.g. from a previous run) but now contain stale data
which must be discarded.

- When host.create creates {to,from}LVM volumes, it explicitly
  erases them.
- Vgcreate erases the xenvmd journal volume

This relies on [djs55/mirage-block-volume#64]

Signed-off-by: David Scott <dave.scott@citrix.com>
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.28%) to 32.3% when pulling da13277 on avoid-infinite-loops into 11e31e8 on master.

Signed-off-by: David Scott <dave.scott@citrix.com>
@djs55 djs55 force-pushed the avoid-infinite-loops branch from ec219b0 to c135579 Compare April 30, 2015 13:28
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.28%) to 32.3% when pulling c135579 on avoid-infinite-loops into 11e31e8 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.28%) to 32.3% when pulling c135579 on avoid-infinite-loops into 11e31e8 on master.

djs55 added a commit that referenced this pull request Apr 30, 2015
Erase metadata volumes which we create
@djs55 djs55 merged commit a12bb07 into master Apr 30, 2015
@djs55 djs55 deleted the avoid-infinite-loops branch April 30, 2015 13:46
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.

2 participants