[openstack-dev] [Nova][Baremetal][DB] Core review request: bugfix for 1221620

Roman Podolyaka rpodolyaka at mirantis.com
Mon Sep 9 11:45:47 UTC 2013


Robert,

Cool! That would be really nice! nova-bm functional tests are the bare
minimum we need.

Thanks,
Roman


On Mon, Sep 9, 2013 at 2:27 PM, Robert Collins <robertc at robertcollins.net>wrote:

> We deseperately want TripleO in the gate and are working towards it.
> However, doing that acceptably fast makes it non-trivial [nested VM's
> are kindof slow...]. If there was BYOI in RackSpace and in HP's
> non-Beta regions, and custom networking in both clouds, then we could
> avoid a layer of nesting and test the different TripleO layers
> independently (and concurrently) but sadly thats not the case yet, so
> we're having to work on tuning and the occasional hack.
>
> We hope to have at least functional tests for nova-bm in soon, with
> TripleO as a whole following on subsequently to that.
>
> -Rob
>
> On 9 September 2013 23:12, Roman Podolyaka <rpodolyaka at mirantis.com>
> wrote:
> > Hi,
> >
> > I'm ok with both accepting this patch and reverting the commit, which
> > introduced the regression, but it would be really nice to have these DB
> > optimizations in Nova.
> >
> > As for your concern of accepting such optimizations. I don't think, it's
> a
> > problem of such patches themselves, but rather with the lack of
> > comprehensive tests of complex OpenStack installations in our CI (at the
> > same time I personally believe our CI is the best thing ever happened to
> > OpenStack :), CI team you really rock!).
> >
> > Anyway, TripleO-CI found this regression. Maybe we should consider adding
> > its job to Nova check/gate pipelines?
> >
> > Thanks,
> > Roman
> >
> >
> > On Mon, Sep 9, 2013 at 1:59 PM, Nikola Đipanov <ndipanov at redhat.com>
> wrote:
> >>
> >> On 09/09/13 11:25, Roman Podolyaka wrote:
> >> > Hi,
> >> >
> >> > There is a patch on review (https://review.openstack.org/#/c/45422/)
> >> > fixing https://bugs.launchpad.net/tripleo/+bug/1221620 which has
> >> > importance 'Critical' in Nova and TripleO (long story short: currently
> >> > Nova Baremetal deployments with more than one baremetal node won't
> >> > work).
> >> >
> >> > It would be really nice to have this patch reviewed by core
> developers,
> >> > so we can fix the bug ASAP.
> >> >
> >>
> >> Hey - thanks for responding quickly - I commented on the patch and tbh I
> >> am starting to be -1 on this due to issues mentioned on the review.
> >>
> >> I will accept that my take on this is too conservative :) and remove a
> >> -1 if needed to get this in, but at this point, I have some doubts
> >> weather this is the right approach.
> >>
> >> Cheers,
> >>
> >> N.
> >>
> >>
> >> > Thanks,
> >> > Roman
> >> >
> >> >
> >> > _______________________________________________
> >> > OpenStack-dev mailing list
> >> > OpenStack-dev at lists.openstack.org
> >> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> >> >
> >>
> >>
> >> _______________________________________________
> >> OpenStack-dev mailing list
> >> OpenStack-dev at lists.openstack.org
> >> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> >
> >
> >
> > _______________________________________________
> > OpenStack-dev mailing list
> > OpenStack-dev at lists.openstack.org
> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> >
>
>
>
> --
> Robert Collins <rbtcollins at hp.com>
> Distinguished Technologist
> HP Converged Cloud
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20130909/3cb81b9d/attachment.html>


More information about the OpenStack-dev mailing list