<div dir="ltr">Hi,<div><br></div><div>This FFE is approved</div><div><br></div><div>Thanks</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Sep 18, 2019 at 2:45 PM Slawek Kaplonski <<a href="mailto:skaplons@redhat.com">skaplons@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
Personally I think we can go with this is You will implement it now.<br>
As per discussion on IRC, Ironic code which will use those notifications isn't<br>
really ready yet, and will not be for Train. So even if something would possible<br>
go wrong (but won't for sure ;)) we shouldn't break Ironic.<br>
<br>
On Wed, Sep 18, 2019 at 11:04:54AM -0500, Eric Fried wrote:<br>
> > I'd like to open an FFE request to convert the ironic events notifier<br>
> > from the current ironicclient to openstacksdk with the change<br>
> > <a href="https://review.opendev.org/682040" rel="noreferrer" target="_blank">https://review.opendev.org/682040</a><br>
> <br>
> This is kind of none of my business, but since the existing ironic stuff<br>
> was only introduced in Train [1], IMO it is important to allow this FFE<br>
> so neutron doesn't have to go through the pain of supporting and<br>
> deprecating the conf options (e.g. `ironic_url`) and code paths through<br>
> python-ironicclient.<br>
<br>
Thx. I agree. That's another good point to accept this FFE.<br>
<br>
> <br>
> efried<br>
> <br>
> [1] <a href="https://review.opendev.org/#/c/658787/" rel="noreferrer" target="_blank">https://review.opendev.org/#/c/658787/</a><br>
> <br>
<br>
-- <br>
Slawek Kaplonski<br>
Senior software engineer<br>
Red Hat<br>
<br>
<br>
</blockquote></div>