File pacemaker-crm_resource-not-send-history-erasure... of Package pacemaker.8750

Overview Repositories Revisions Requests Users Attributes Meta

File pacemaker-crm_resource-not-send-history-erasure-for-none.patch of Package pacemaker.8750

commit 1eacc741d04098d24b97631a273501ef0aff2fc1
Author: Andrew Beekhof <andrew@beekhof.net>
Date:   Thu Sep 21 20:59:16 2017 +1000

crm_resource: Do not send history erasure commands for nodes where this is none
    
    If a resource is not known on a given node, thats because we have no
    operation history for it there.  So there is no benefit to requesting a
    cleanup for that resource/node combination.

diff --git a/tools/crm_resource_runtime.c b/tools/crm_resource_runtime.c
index 8f9ae0dac..210ea7d15 100644
--- a/tools/crm_resource_runtime.c
+++ b/tools/crm_resource_runtime.c
@@ -573,7 +573,7 @@ send_lrm_rsc_op(crm_ipc_t * crmd_channel, const char *op,
         rc = 0;
 
     } else {
-        CMD_ERR("Could not send %s op to the crmd", op);
+        crm_debug("Could not send %s op to the crmd", op);
         rc = -ENOTCONN;
     }
 
@@ -627,8 +627,9 @@ cli_resource_delete(crm_ipc_t *crmd_channel, const char *host_uname,
 
     } else if (host_uname == NULL) {
         GListPtr lpc = NULL;
+        GListPtr nodes = g_hash_table_get_values(rsc->known_on);
 
-        for (lpc = data_set->nodes; lpc != NULL; lpc = lpc->next) {
+        for (lpc = nodes; lpc != NULL; lpc = lpc->next) {
             node = (node_t *) lpc->data;
 
             if (node->details->online) {
@@ -637,6 +638,7 @@ cli_resource_delete(crm_ipc_t *crmd_channel, const char *host_uname,
             }
         }
 
+        g_list_free(nodes);
         return pcmk_ok;
     }

Places

File pacemaker-crm_resource-not-send-history-erasure-for-none.patch of Package pacemaker.8750

Places