Open Stack

Hi guys,

Small question about [0].

The patch fixes a bug wrt VMs live migration but it also adds a new
configuration flag to allow admins to disable multi-attach. IMHO,
adding a new configuration flag shouldn't be allowed in a backport but
the patch in matter fixes a bug. What would the right behavior be in
that case?

1) Modify the patch so that it doesn't adds a new configuration
parameter.

2) Let it land as-is.

3) Don't let it land at all.

I'm more for #1 and #3 (maybe more 1 since it fixes a bug). Notice
that adding that parameter doesn't change current behavior.


Cheers,
FF

[0] https://review.openstack.org/#/c/29488/1

-- 
{ name: "Flavio Percoco",
   gpg: "87112EC1", 
   internal: "8261386",
   phone: "+390687502386",
   irc: ["fpercoco", "flaper87"]}