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

[renayama19661014 at ybb.ne.jp]

Hi Yan,

This problem seems to still occur somehow or other. 
The Orphaned resource of the clone is extremely rarely displayed. 

I confirm a condition and report it again.

Best Regards,
Hideo Yamauchi.


--- renayama19661014 at ybb.ne.jp wrote:

> 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
> > 
> 
> 
> _______________________________________________
> Pacemaker mailing list
> Pacemaker at oss.clusterlabs.org
> http://oss.clusterlabs.org/mailman/listinfo/pacemaker
>