[Pacemaker] [GUI][PATCH]An orphan resource was displayed by GUI.

[renayama19661014 at ybb.ne.jp]

Hi Yan,

> Could you please try the attached patch?

I confirmed that a problem was solved with your patch.
Please reflect your patch in the latest edition.

Best Regards,
Hideo Yamauchi.

--- Yan Gao <ygao at novell.com> wrote:

> Hi,
> 
> renayama19661014 at ybb.ne.jp wrote:
> > Hi,
> > 
> > An orphan resource was displayed by GUI.
> > There is the displayed orphan resource nowhere.
> > 
> > Step1) I start corosync service.
> > 
> > Step2) I spend cib.xml by a cibadmin command.
> > 
> > Step3) I confirm a resource in GUI.
> > 
> > The orphan resource of the clone is displayed by GUI.
> > However, there cannot be this orphan resource (clnUmResource:2).
> > 
> > I attached hard copy of GUI and a result of hb_report.
> > 
> > And I attached the patch of the revision again. 
> > However, this patch may be unfinished.
> > 
> > Please confirm a patch, and please solve this problem.
> Thanks for reporting and the patch!
> Could you please try the attached patch?
> 
> Regards,
>   Yan
> 
> -- 
> Yan Gao <ygao at novell.com>
> Software Engineer
> China Server Team, OPS Engineering, Novell, Inc.
> > diff -r 8b56e4d03dc8 mgmt/daemon/mgmt_crm.c
> --- a/mgmt/daemon/mgmt_crm.c	Tue Jan 12 13:25:09 2010 +0800
> +++ b/mgmt/daemon/mgmt_crm.c	Tue Jan 19 17:11:53 2010 +0800
> @@ -1475,7 +1475,10 @@
>  	ret = strdup(MSG_OK);
>  	while (cur != NULL) {
>  		resource_t* rsc = (resource_t*)cur->data;
> -		ret = mgmt_msg_append(ret, rsc->id);
> +		gboolean is_active = rsc->fns->active(rsc, FALSE);
> +		if (is_not_set(rsc->flags, pe_rsc_orphan) || is_active) {
> +			ret = mgmt_msg_append(ret, rsc->id);
> +		}
>  		cur = g_list_next(cur);
>  	}
>  	free_data_set(data_set);
> > _______________________________________________
> Pacemaker mailing list
> Pacemaker at oss.clusterlabs.org
> http://oss.clusterlabs.org/mailman/listinfo/pacemaker
>