Fix: libpe_status: Add fake local node to all resources' allowed_nodes

This fixes a regression introduced by bf7ffcd. As of that commit, the
fake local node is created after all resources have been unpacked. So it
doesn't get added to resources' allowed_nodes tables.

This prevents registration of fencing devices when the fencer receives a
CIB diff that doesn't contain the local node. For example, the user may
have replaced the CIB with a boilerplate configuration that has an empty
nodes section.

See the following pull requests from Hideo Yamauchi and their
discussions:
#3849
#3852

Thanks to Hideo for the report and finding the cause.

Signed-off-by: Reid Wahl <[email protected]>

Author: nrwahl2

lib/pengine/status.c

@@ -167,10 +167,22 @@ cluster_status(pcmk_scheduler_t * scheduler)
     if ((scheduler->priv->local_node_name != NULL)
         && (pcmk_find_node(scheduler,
                            scheduler->priv->local_node_name) == NULL)) {
+        pcmk_node_t *node = NULL;
+
         crm_info("Creating a fake local node for %s",
                  scheduler->priv->local_node_name);
-        pe_create_node(scheduler->priv->local_node_name,
-                       scheduler->priv->local_node_name, NULL, 0, scheduler);
+        node = pe_create_node(scheduler->priv->local_node_name,
+                              scheduler->priv->local_node_name, NULL, 0,
+                              scheduler);
+
+        for (GList *iter = scheduler->priv->resources; iter != NULL;
+             iter = iter->next) {
+             pcmk_resource_t *rsc = iter->data;
+             pcmk_node_t *copy = pe__copy_node(node);
+
+             g_hash_table_insert(rsc->priv->allowed_nodes,
+                                 (gpointer) copy->priv->id, copy);
+        }
     }
 
     pcmk__set_scheduler_flags(scheduler, pcmk__sched_have_status);