[openstack-dev] [puppet] Action parameter missing in generated stonith fence manifests

Devon Mizelle dev+stack at devon.so
Fri Jun 17 22:05:14 UTC 2016


I was directed here from #puppet-openstack to submit an e-mail.

Noted here, there is an 'action' parameter defined for fence_ifmib:


However, in the generated manifest here, it seems that the 'action'
parameter is missing:


It looks like the 'action' parameter is missing in every fence_*
manifest (and has never existed.) I need this available to me so that
I can override the default action of fence_ifmib (which is 'reboot')
and set it to 'off'.

It looks like that in the agent_generator.rb script, action is being
specifically ignored in addition to 'help' and 'version' parameters.
I've attached a quick plain-text patch. I don't think this is the
norm, but considering that its such a small one word change I'm hoping
it would be OK.

Thanks for your time and for a wonderful project,

diff --git a/agent_generator/agent_generator.rb
index 9c1ab0a..171ca50 100755
--- a/agent_generator/agent_generator.rb
+++ b/agent_generator/agent_generator.rb
@@ -40,7 +40,7 @@ class FencingMetadataParser
       param['default'] = REXML::XPath.match(p, 'string(./content/@default)')[0]
       param['description'] = REXML::XPath.match(p, 'string(./shortdesc)')[0]
       ## remove parameters that are not usable during automatic execution
-      @params.push(param) unless %w(help version
+      @params.push(param) unless %w(help version).include?(param['name'])

More information about the OpenStack-dev mailing list