From a98122450c5cee36234790fca72603e08e5ddda1 Mon Sep 17 00:00:00 2001 From: Ken Gaillot Date: Feb 24 2021 16:08:42 +0000 Subject: Refactor: scheduler: don't include originally unpacked resource parameters in digest Previously, when calculating an operation digest, calculate_main_digest() would grab the following, in order of highest to lowest precedence: * instance attributes evaluated for the appropriate node * instance attributes specified with the operation * instance attributes as originally unpacked (without evaluating for any node) * resource meta-attributes Adding the originally unpacked instance attributes was redundant, since node-evaluated instance attributes would always be a superset of those and would take precedence. --- diff --git a/lib/pengine/pe_digest.c b/lib/pengine/pe_digest.c index 1e119f9..dd6b753 100644 --- a/lib/pengine/pe_digest.c +++ b/lib/pengine/pe_digest.c @@ -162,7 +162,6 @@ calculate_main_digest(op_digest_cache_t *data, pe_resource_t *rsc, } g_hash_table_foreach(local_rsc_params, hash2field, data->params_all); g_hash_table_foreach(action->extra, hash2field, data->params_all); - g_hash_table_foreach(rsc->parameters, hash2field, data->params_all); g_hash_table_foreach(action->meta, hash2metafield, data->params_all); #if ENABLE_VERSIONED_ATTRS