Now that I re-read the patch.
Shouldn't the version checking  need to be converted into a sanity check?

Miguel Ángel Ajo


On Thursday, 8 de January de 2015 at 12:51, Kevin Benton wrote:

> Thanks for the insight.
>  
> On Thu, Jan 8, 2015 at 3:41 AM, Miguel Ángel Ajo <majop...@redhat.com 
> (mailto:majop...@redhat.com)> wrote:
> > Correct, that’s the problem, what Kevin said should be the ideal case, but 
> > distros have
> > proven to fail satisfying this kind of requirements earlier.
> >  
> > So at least a warning to the user may be good to have IMHO.  
> >  
> > Miguel Ángel Ajo
> >  
> >  
> > On Thursday, 8 de January de 2015 at 12:36, Ihar Hrachyshka wrote:
> >  
> > > The problem is probably due to the fact that some operators may run 
> > > neutron from git and manage their dependencies in some other way; or 
> > > distributions may suck sometimes, so packagers may miss the release note 
> > > and fail to upgrade dnsmasq; or distributions may have their specific 
> > > concerns on upgrading dnsmasq version, and would just backport the needed 
> > > fix to their 'claimed to 2.66' dnsmasq (common story in Red Hat world).
> > >  
> > > On 01/08/2015 05:25 AM, Kevin Benton wrote:
> > > > If the new requirement is expressed in the neutron packages for the 
> > > > distro, wouldn't it be transparent to the operators?  
> > > >  
> > > > On Wed, Jan 7, 2015 at 6:57 AM, Kyle Mestery <mest...@mestery.com 
> > > > (mailto:mest...@mestery.com)> wrote:
> > > > > On Wed, Jan 7, 2015 at 8:21 AM, Ihar Hrachyshka <ihrac...@redhat.com 
> > > > > (mailto:ihrac...@redhat.com)> wrote:
> > > > > > Hi all,
> > > > > >  
> > > > > > I've found out that dnsmasq < 2.67 does not work properly for IPv6 
> > > > > > clients when it comes to MAC address matching (it fails to match, 
> > > > > > and so clients get 'no addresses available' response). I've 
> > > > > > requested version bump to 2.67 in: 
> > > > > > https://review.openstack.org/145482
> > > > > >  
> > > > > Good catch, thanks for finding this Ihar!
> > > > >   
> > > > > > Now, since we've already released Juno with IPv6 DHCP stateful 
> > > > > > support, and DHCP agent still has minimal version set to 2.63 
> > > > > > there, we have a dilemma on how to manage it from stable 
> > > > > > perspective.
> > > > > >  
> > > > > > Obviously, we should communicate the revealed version dependency to 
> > > > > > deployers via next release notes.
> > > > > >  
> > > > > > Should we also backport the minimal version bump to Juno? This will 
> > > > > > result in DHCP agent failing to start in case packagers don't bump 
> > > > > > dnsmasq version with the next Juno release. If we don't bump the 
> > > > > > version, we may leave deployers uninformed about the fact that 
> > > > > > their IPv6 stateful instances won't get any IPv6 address assigned.
> > > > > >  
> > > > > > An alternative is to add a special check just for Juno that would 
> > > > > > WARN administrators instead of failing to start DHCP agent.
> > > > > >  
> > > > > > Comments?
> > > > > >  
> > > > > Personally, I think the WARN may be the best route to go. Backporting 
> > > > > a change which bumps the required dnsmasq version seems like it may 
> > > > > be harder for operators to handle.
> > > > >  
> > > > > Kyle
> > > > >   
> > > > > > /Ihar
> > > > > >  
> > > > > > _______________________________________________
> > > > > > OpenStack-dev mailing list
> > > > > > OpenStack-dev@lists.openstack.org 
> > > > > > (mailto:OpenStack-dev@lists.openstack.org)
> > > > > > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> > > > >  
> > > > >  
> > > > > _______________________________________________
> > > > > OpenStack-dev mailing list
> > > > > OpenStack-dev@lists.openstack.org 
> > > > > (mailto:OpenStack-dev@lists.openstack.org)
> > > > > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> > > > >  
> > > >  
> > > >  
> > > >  
> > > > --  
> > > > Kevin Benton  
> > > >  
> > > > _______________________________________________ OpenStack-dev mailing 
> > > > list OpenStack-dev@lists.openstack.org 
> > > > (mailto:OpenStack-dev@lists.openstack.org) 
> > > > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev  
> > > _______________________________________________
> > > OpenStack-dev mailing list
> > > OpenStack-dev@lists.openstack.org 
> > > (mailto:OpenStack-dev@lists.openstack.org)
> > > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> > >  
> > >  
> > >  
> >  
> >  
> >  
> > _______________________________________________
> > OpenStack-dev mailing list
> > OpenStack-dev@lists.openstack.org (mailto:OpenStack-dev@lists.openstack.org)
> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> >  
>  
>  
>  
> --  
> Kevin Benton  
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev@lists.openstack.org (mailto:OpenStack-dev@lists.openstack.org)
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>  
>  


_______________________________________________
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Reply via email to