Browse Source

Update patch set 2

Patch Set 2:

I'm good with this ps's, concept of making instances possibly remote. In this particular case, I'd argue for making it optional though, with a conditional block. most sites won't ever use this functionality so making them create a set of pv/pvc's for it seems extraneous. It also breaks existing deployments.

Also, can you please rename it nova-instance-pv from novainstance-pv? its a little cleaner that way.

With those two changes, I'm +2 on it.

Patch-set: 2
Gerrit User 9237 5 years ago committed by Gerrit Code Review

Diff Content Not Available