23.02.2015 21:50, David Vossel wrote:


----- Original Message -----

On 14 Feb 2015, at 1:10 am, Vladislav Bogdanov <bub...@hoster-ok.com>
wrote:

Hi,

I believe that is a bug that 'stop' operation uses set of instance
attributes from the original 'start' op, not what successful 'reload' had.
Corresponding pe-input has correct set of attributes, and pre-stop 'notify'
op uses updated set of attributes too.
This is easily reproducible with 3.9.6 resource agents and trace_ra.

pacemaker is c529898.

Should I provide more information?

Yes please.
I suspect the lrmd needs to update it's parameter cache for the reload
operation.

David?

This falls on the crmd I believe. I haven't tested it, but something like
this should fix it I bet.

diff --git a/crmd/lrm.c b/crmd/lrm.c
index ead2e05..45641d2 100644
--- a/crmd/lrm.c
+++ b/crmd/lrm.c
@@ -186,6 +186,7 @@ update_history_cache(lrm_state_t * lrm_state, 
lrmd_rsc_info_t * rsc, lrmd_event_

          if (op->params &&
              (safe_str_eq(CRMD_ACTION_START, op->op_type) ||
+             safe_str_eq("reload", op->op_type) ||
               safe_str_eq(CRMD_ACTION_STATUS, op->op_type))) {

              if (entry->stop_params) {



This definitely fixes the issue,
thank you!


_______________________________________________
Pacemaker mailing list: Pacemaker@oss.clusterlabs.org
http://oss.clusterlabs.org/mailman/listinfo/pacemaker

Project Home: http://www.clusterlabs.org
Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
Bugs: http://bugs.clusterlabs.org



_______________________________________________
Pacemaker mailing list: Pacemaker@oss.clusterlabs.org
http://oss.clusterlabs.org/mailman/listinfo/pacemaker

Project Home: http://www.clusterlabs.org
Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
Bugs: http://bugs.clusterlabs.org

Reply via email to