On Mon, Dec 05, 2016 at 01:31:15PM +0200, Alex Kiousis wrote: > Hello (I work with Yiannis at grnet), > the main reason for this patchset is that a command like > clear-os-params is needed if you are reinstalling between 2 os > providers with different parameters. Without a mechanism to remove a > defined osparam the reinstall will fail when ganeti tries to validate > the params against the new provider. > > The remove-os-params was added when we figured that there should be a > way to bring the Instance osparams to the desired state without > respecifying everything. > > Given the above, i believe that --clear-os-params is much more > intuitive than --remove-os-params "\*". We can add both to the patch > but unless there is considerable overhead for the extra option, we'd > like for clean-os-params to remain seperate.
OK, that's fine by me. I don't have strong feelings about it one way or the other, but I thought I'd suggest the alternative. Cheers, Brian.
