<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-family:comic sans ms,sans-serif"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jul 28, 2020 at 4:48 AM Bogdan Dobrelya <<a href="mailto:bdobreli@redhat.com">bdobreli@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">On 7/27/20 7:08 PM, Dan Smith wrote:<br>
>> Tagging with Nova and Neutron as they are mentioned and I thought some<br>
>> people from those teams had opinions on this.<br>
> <br>
> Nova already implements ping() on the compute RPC interface, which we<br>
> use to make sure compute waits to start up until conductor is available<br>
> to do its bidding. So if a new obligatory RPC server method is actually<br>
> added called ping(), it will break us.<br>
> <br>
>> Can you refresh my memory on why we dropped this before? I recall<br>
>> talking about it in Denver, but I can't for the life of me remember<br>
>> what the conclusion was. Did we intend to use something else for this<br>
>> that has since fallen through?<br>
> <br>
> The prior conversation I recall was about helm sitting on our bus to<br>
> (ab)use our ping method for health checks:<br>
> <br>
> <a href="https://opendev.org/openstack/openstack-helm/commit/baf5356a4fb61590a95f64a63c0dcabfebb3baaa" rel="noreferrer" target="_blank">https://opendev.org/openstack/openstack-helm/commit/baf5356a4fb61590a95f64a63c0dcabfebb3baaa</a><br>
> <br>
> I believe that has since been reverted.<br>
> <br>
> The primary concern was about something other than nova sitting on our<br>
> bus making calls to our internal services. I imagine that the proposal<br>
> to bake it into oslo.messaging is for the same purpose, and I'd probably<br>
> have the same concern. At the time I think we agreed that if we were<br>
> going to support direct-to-service health checks, they should be teensy<br>
> HTTP servers with oslo healthchecks middleware. Further loading down<br>
> rabbit with those pings doesn't seem like the best plan to<br>
> me. Especially since Nova (compute) services already check in over RPC<br>
> periodically and the success of that is discoverable en masse through<br>
> the API.<br>
<br>
Having RPC ping in the common messaging library could improve aliveness <br>
handling of long-running APIs, like listing multiple Neutron ports or <br>
Heat objects with full details, or running some longish Mistral workflow <br>
maybe. Indeed it should be made not breaking things already existing in <br>
Nova ofc.<br>
<br></blockquote><div><br></div><div><div style="font-family:comic sans ms,sans-serif" class="gmail_default">Not sure this is related to your concern about long running API's but O.M. has an optional RPC call heartbeat monitor that verifies the connectivity to the server while the call is in progress.  See the description of call_monitor_timeout in the RPC client docs [0].</div><div style="font-family:comic sans ms,sans-serif" class="gmail_default"><br></div><div style="font-family:comic sans ms,sans-serif" class="gmail_default">0: <a href="https://docs.openstack.org/oslo.messaging/latest/reference/rpcclient.html">https://docs.openstack.org/oslo.messaging/latest/reference/rpcclient.html</a></div><div style="font-family:comic sans ms,sans-serif" class="gmail_default"><br></div><div style="font-family:comic sans ms,sans-serif" class="gmail_default"><br></div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> <br>
> --Dan<br>
> <br>
<br>
<br>
-- <br>
Best regards,<br>
Bogdan Dobrelya,<br>
Irc #bogdando<br>
<br>
<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><span style="font-family:comic sans ms,sans-serif">Ken Giusti  (<a href="mailto:kgiusti@gmail.com" target="_blank">kgiusti@gmail.com</a>)</span></div></div></div></div>