API: fencer: Don't automatically pass nodeid as parameter
...for fence devices whose agent metadata advertises support for a
nodeid parameter.
No known fence agent advertises the nodeid parameter in its metadata, so
is_nodeid_required() should never return true unless some custom fence
agent advertises nodeid and expects Pacemaker to pass it automatically.
If anyone complains about this, we can deprecate this functionality for
a while and drop it at a major or minor release. However, it seems
likely this has been unused for a long time.
It was added by 08c78ad (and again by c2265a12, oddly) in 2012. The
commit message gives no information about what motivated the change, nor
does the linked pull request give any real insight. The PR mentions
"stonith scsi" support, but the fence_scsi agent at the time did not
support a nodeid parameter.
Signed-off-by: Reid Wahl <nrwahl@protonmail.com>