<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
On 02/22/2017 03:42 PM, Yaniv Kaul wrote:<br>
<blockquote
cite="mid:CAJgorsaL15MmmHX-c-7Qr3pb-rBjKp+sBtExD8mH2zh6hiMWWQ@mail.gmail.com"
type="cite">
<div dir="ltr"><br>
<br>
<div class="gmail_quote">
<div dir="ltr">On Wed, Feb 22, 2017 at 4:32 PM Francesco
Romani <<a moz-do-not-send="true"
href="mailto:fromani@redhat.com">fromani@redhat.com</a>>
wrote:<br>
</div>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000" class="gmail_msg"> On
02/22/2017 01:53 PM, Simone Tiraboschi wrote:<br
class="gmail_msg">
<blockquote type="cite" class="gmail_msg">
<div dir="ltr" class="gmail_msg"><br class="gmail_msg">
<div class="gmail_extra gmail_msg"><br
class="gmail_msg">
<div class="gmail_quote gmail_msg">On Wed, Feb 22,
2017 at 1:33 PM, Simone Tiraboschi <span
dir="ltr" class="gmail_msg"><<a
moz-do-not-send="true"
href="mailto:stirabos@redhat.com"
class="gmail_msg" target="_blank">stirabos@redhat.com</a>></span>
wrote:<br class="gmail_msg">
<blockquote class="gmail_quote gmail_msg"
style="margin:0px 0px 0px 0.8ex;border-left:1px
solid rgb(204,204,204);padding-left:1ex">
<div dir="ltr" class="gmail_msg">When
ovirt-ha-agent checks the status of the engine
VM we get:
<div class="gmail_msg">
<pre style="color:rgb(0,0,0)" class="gmail_msg">2017-02-21 22:21:14,738-0500 ERROR (jsonrpc/2) [api] FINISH getStats error=Virtual machine does not exist: {'vmId': u'2ccc0ef0-cc31-45b8-8e91-a78fa4cad671'} (api:69)
Traceback (most recent call last):
File "/usr/lib/python2.7/site-packages/vdsm/common/api.py", line 67, in method
ret = func(*args, **kwargs)
File "/usr/share/vdsm/API.py", line 335, in getStats
vm = self.vm
File "/usr/share/vdsm/API.py", line 130, in vm
raise exception.NoSuchVM(vmId=self._UUID)
NoSuchVM: Virtual machine does not exist: {'vmId': u'2ccc0ef0-cc31-45b8-8e91-a78fa4cad671'}</pre>
<pre class="gmail_msg">While in ovirt-ha-agent logs we have:<pre style="color:rgb(0,0,0)" class="gmail_msg"><pre class="gmail_msg">MainThread::<a moz-do-not-send="true" class="m_-8855782447270567219moz-txt-link-freetext gmail_msg">INFO::2017-02-21</a> 22:21:18,583::hosted_engine::453::ovirt_hosted_engine_ha.agent.hosted_engine.HostedEngine::(start_monitoring) Current state UnknownLocalVmState (score: 3400)</pre><pre class="gmail_msg">...</pre></pre><pre style="color:rgb(0,0,0)" class="gmail_msg">MainThread::<a moz-do-not-send="true" class="m_-8855782447270567219moz-txt-link-freetext gmail_msg">INFO::2017-02-21</a> 22:21:31,199::state_decorators::25::ovirt_hosted_engine_ha.agent.hosted_engine.HostedEngine::(check) Unknown local engine vm status no actions taken</pre></pre>
Probably it's a bug or a regression
somewhere on master.</div>
</div>
</blockquote>
<div class="gmail_msg">
</div>
<div class="gmail_msg">On ovirt-ha-broker side the
detection is based on a strict string match on
the error message that is expected to be exactly
'Virtual machine does not exist' to set down
status otherwise we set unknown status as in
this case:</div>
<div class="gmail_msg"><a moz-do-not-send="true"
href="https://gerrit.ovirt.org/gitweb?p=ovirt-hosted-engine-ha.git;a=blob;f=ovirt_hosted_engine_ha/broker/submonitors/engine_health.py;h=d633cb860b811e84021221771bf706a9a4ac1d63;hb=refs/heads/master#l54"
class="gmail_msg" target="_blank">https://gerrit.ovirt.org/gitweb?p=ovirt-hosted-engine-ha.git;a=blob;f=ovirt_hosted_engine_ha/broker/submonitors/engine_health.py;h=d633cb860b811e84021221771bf706a9a4ac1d63;hb=refs/heads/master#l54</a>
</div>
<div class="gmail_msg"> </div>
<div class="gmail_msg">Adding Francesco here to
understand if something has recently changed
there on vdsm side.</div>
</div>
</div>
</div>
</blockquote>
</div>
<div bgcolor="#FFFFFF" text="#000000" class="gmail_msg">
It has changed indeed; we had a series of changes which
added context to some exceptions. I believe the straw who
broke the camel's back was
I32ec3f86f8d53f8412f4c0526fc85e2a42e30ea5
It is unfortunate that this change broke HA.
Could you perhaps fixing it checking that the message
*begins* with that string, and/or checking the error code.
bests,</div>
</blockquote>
<div><br>
</div>
<div>On the bright side, this is exactly why we need o-s-t
running Hosted-Engine - though we probably need to exercise
more HE flows (global and local maint., for example).</div>
<div>On the downside, how come <span
style="font-size:13px;color:rgb(33,33,33)">I32ec3f86f8d53f8412f4c0526fc85</span><span
style="font-size:13px;color:rgb(33,33,33)">e2a42e30ea5 was
merged on Jan1st, and we only saw the regression now? Is
there another bug that hid this one until now?</span></div>
<div><font color="#212121">Y.</font></div>
<br>
</div>
</div>
</blockquote>
<br>
It was merged on Jan 29 on master, backported on Feb 8 on 4.1 branch
(because it was part of the vmleases feature, needed on 4.1.z).<br>
<br>
Bests,<br>
<br>
<pre class="moz-signature" cols="72">--
Francesco Romani
Red Hat Engineering Virtualization R & D
IRC: fromani</pre>
</body>
</html>