← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1369386] Re: ipset_enabled = True default is not upgrade friendly

 

I agree with all these points, but similar decisions were made in the
past, so I am not sure why this one should be treated any differently.

For instance, at the end of the Icehouse cycle, Neutron got a new
dependency on novaclient and yet we enabled notification by default.
This led to a similar unfriendly upgrade path the reason where Neutron
needed novaclient installed when previously it didn't.

As far this is concerned, I think this issue raised enough awareness
that we can fix the actual problem where it belongs, like TripleO,
package specs, puppet/chef cookbooks, rather than churning more changes
than are strictly required. Even deploying from source does not prevent
you from incurring from issues like these from time to time. Another
example is DVR: that requires L3 and Firewall agents to run on the
compute boxes now. This may lead to new dependencies that need to be
solved. We chose to keep DVR disabled by default, and that clearly gives
more time to people to catch up, but that's not why we did it.


** Changed in: neutron
       Status: In Progress => Invalid

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1369386

Title:
  ipset_enabled = True default is not upgrade friendly

Status in OpenStack Neutron (virtual network service):
  Invalid

Bug description:
  Adds ipset support for Security Groups

  In commit 2562a9271c828e982a74593e8fd07be13b0cfc4a we recently added
  ipset support for Security Groups.

  The default is currently set to True which is not upgrade friendly in
  that anyone upgrading to the most recent Neutron code immediately has
  to have the ipset binary installed or their commands fail.

  It seems like we should set this to false by default (as a safe
  default) and allow users to opt in since it is really an
  optimization...

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1369386/+subscriptions


References