[neutron][drivers][ironic] FFE request - Use openstacksdk for ironic notifiers

Slawek Kaplonski skaplons at redhat.com
Mon Sep 23 14:13:47 UTC 2019


Hi,

Thx Riccardo for working on this :)

On Mon, Sep 23, 2019 at 10:32:15AM +0200, Riccardo Pittau wrote:
> The patch has merged
> Thanks all!
> 
> On Wed, 18 Sep 2019 at 23:35, Miguel Lavalle <miguel at mlavalle.com> wrote:
> >
> > Hi,
> >
> > This FFE is approved
> >
> > Thanks
> >
> > On Wed, Sep 18, 2019 at 2:45 PM Slawek Kaplonski <skaplons at redhat.com> wrote:
> >>
> >> Hi,
> >>
> >> Personally I think we can go with this is You will implement it now.
> >> As per discussion on IRC, Ironic code which will use those notifications isn't
> >> really ready yet, and will not be for Train. So even if something would possible
> >> go wrong (but won't for sure ;)) we shouldn't break Ironic.
> >>
> >> On Wed, Sep 18, 2019 at 11:04:54AM -0500, Eric Fried wrote:
> >> > > I'd like to open an FFE request to convert the ironic events notifier
> >> > > from the current ironicclient to openstacksdk with the change
> >> > > https://review.opendev.org/682040
> >> >
> >> > This is kind of none of my business, but since the existing ironic stuff
> >> > was only introduced in Train [1], IMO it is important to allow this FFE
> >> > so neutron doesn't have to go through the pain of supporting and
> >> > deprecating the conf options (e.g. `ironic_url`) and code paths through
> >> > python-ironicclient.
> >>
> >> Thx. I agree. That's another good point to accept this FFE.
> >>
> >> >
> >> > efried
> >> >
> >> > [1] https://review.opendev.org/#/c/658787/
> >> >
> >>
> >> --
> >> Slawek Kaplonski
> >> Senior software engineer
> >> Red Hat
> >>
> >>

-- 
Slawek Kaplonski
Senior software engineer
Red Hat




More information about the openstack-discuss mailing list