<div dir="ltr">Crap, didn&#39;t realize I just had to restart vdsm for the new API.py to take effect.<div>Anyway I edited fencing.py and added back the compatibility code, and its all working for now.</div><div><br></div><div>
I&#39;m going to work on touch up changes on fence_raritan so that we can get it submitted into the next release hopefully, all of these other hacks will become moot at that time.</div></div><div class="gmail_extra"><br><br>
<div class="gmail_quote">On Tue, Mar 25, 2014 at 2:08 PM, Dan Kenigsberg <span dir="ltr">&lt;<a href="mailto:danken@redhat.com" target="_blank">danken@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="">On Tue, Mar 25, 2014 at 11:37:13AM -0700, David Smith wrote:<br>
&gt; I created fence_raritan.<br>
&gt;<br>
&gt; Unfortunately it looks like fencing.py is where the legacy --option was<br>
&gt; translated to --action. I believe ovirt calls fencing.py as a proxy to<br>
&gt; execute fence_raritan?<br>
&gt;<br>
&gt; The version of fence-agents I pulled from git has the backwards<br>
&gt; compatibility removed;<br>
&gt; <a href="https://lists.fedorahosted.org/pipermail/cluster-commits/2013-February/003090.html" target="_blank">https://lists.fedorahosted.org/pipermail/cluster-commits/2013-February/003090.html</a><br>
&gt;<br>
&gt;<br>
&gt; So if I added support for it in fence_raritan I&#39;m not sure it would work (I<br>
&gt; can try) but also the fence-agents guys probably wouldn&#39;t take the<br>
&gt; submission to have it added to future code.<br>
<br>
</div>Indeed - I do not expect upstream fence-agents to accept the legacy<br>
option. However, you could backport your own code to el6, and create a<br>
&quot;fence-agents-raritan&quot; rpm from it.<br>
<div class=""><br>
&gt;<br>
&gt; I edited API.py as in <a href="http://gerrit.ovirt.org/#/c/26075/" target="_blank">http://gerrit.ovirt.org/#/c/26075/</a> and re-compiled<br>
&gt; API.py to pyc and pyo files and copied to all my hosts, yet I&#39;m still<br>
&gt; getting an error, so I&#39;m wondering if that change is incomplete or I missed<br>
&gt; some other step to make this actually work.  It&#39;s really a bummer<br>
&gt; especially since the default behavior with no action is reboot ;)<br>
<br>
</div>I suppose that you have missed something. My don&#39;t you place API.py<br>
under /usr/share/vdsm ? And make sure you restart vdsmd, to make the new<br>
code take effect?<br>
<br>
&gt;<br>
&gt;<br>
&gt; *Thread-199071::DEBUG::2014-03-25 10:28:10,587::API::1110::vds::(fenceNode)<br>
&gt; fenceNode(addr=10.105.128.25,port=,agent=raritan,user=admin,passwd=XXXX,action=status,secure=,options=port=/system1/outlet6)*<br>
&gt;<br>
&gt;<br>
&gt; *Thread-199071::DEBUG::2014-03-25 10:28:13,104::API::1136::vds::(fenceNode)<br>
&gt; rc 0 in agent=fence_raritan*<br>
&gt; *ipaddr=10.105.128.25*<br>
&gt; *login=admin*<br>
&gt; *option=status*<br>
&gt; *passwd=XXXX*<br>
&gt; *port=/system1/outlet6 out Success: Rebooted*<br>
&gt; * err Parse error: Ignoring unknown option &#39;option=status&#39;*<br>
<div class="HOEnZb"><div class="h5">&gt;<br>
&gt;<br>
&gt; On Tue, Mar 25, 2014 at 6:28 AM, Dan Kenigsberg &lt;<a href="mailto:danken@redhat.com">danken@redhat.com</a>&gt; wrote:<br>
&gt;<br>
&gt; &gt; On Tue, Mar 25, 2014 at 05:46:58AM -0400, Eli Mesika wrote:<br>
&gt; &gt; &gt; AFAIK this is related to <a href="http://gerrit.ovirt.org/#/c/24343/" target="_blank">http://gerrit.ovirt.org/#/c/24343/</a> which<br>
&gt; &gt; changed --option to --action<br>
&gt; &gt; &gt; danken ?<br>
&gt; &gt;<br>
&gt; &gt; I smells very much related.<br>
&gt; &gt; Where is fence_raritan comming from? Could you have the el6 version of<br>
&gt; &gt; it support the legacy --option name?<br>
&gt; &gt;<br>
&gt; &gt; If not, we could consider taking <a href="http://gerrit.ovirt.org/#/c/26075/" target="_blank">http://gerrit.ovirt.org/#/c/26075/</a>.<br>
&gt; &gt;<br>
&gt; &gt; Dan.<br>
&gt; &gt;<br>
</div></div></blockquote></div><br></div>