Change in vdsm[ovirt-3.3]: fencing: stop using a deprecated command

[Dan Kenigsberg]

Dan Kenigsberg has uploaded a new change for review.

Change subject: fencing: stop using a deprecated command
......................................................................

fencing: stop using a deprecated command

Recently, fence-agents stopped supporting the "option" alias for its
"action" STDIN command.

https://lists.fedorahosted.org/pipermail/cluster-commits/2013-February/00309...

Let us conform to the new option name. The "action" name has been long
supported by el6 and f19's fence-agents, so it can be safely used.

Thanks to John Taylor for reporting the issue.

Bug-Url: https://bugzilla.redhat.com/1063472
Change-Id: Ia0e167e493412f95eda2325e0b028eb7c29c18bc
Signed-off-by: Dan Kenigsberg 
Reviewed-on: http://gerrit.ovirt.org/24303
Reviewed-by: Eli Mesika 
Reviewed-by: Douglas Schilling Landgraf 
Reviewed-on: http://gerrit.ovirt.org/24343
Reviewed-by: Yaniv Bronhaim 
(cherry picked from commit 02efd473ac8886ec860023815aba2ae5af43b91c)
---
M vdsm/API.py
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/75/26075/1

diff --git a/vdsm/API.py b/vdsm/API.py
index 37bb908..f379e9c 100644
--- a/vdsm/API.py
+++ b/vdsm/API.py
@@ -1123,7 +1123,7 @@
                 return errCode['fenceAgent']
             raise

-        inp = ('agent=fence_%s\nipaddr=%s\nlogin=%s\noption=%s\n'
+        inp = ('agent=fence_%s\nipaddr=%s\nlogin=%s\naction=%s\n'
                'passwd=%s\n') % (agent, addr, username, action, password)
         if port != '':
             inp += 'port=%s\n' % (port,)

[Dan Kenigsberg]

Dan Kenigsberg has abandoned this change.

Change subject: fencing: stop using a deprecated command
......................................................................

Abandoned

No urgency here, let's avoid the backporting headache.