Open Stack

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
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-discuss/attachments/20190918/4409de53/attachment-0001.html>