[torquedev] new node attribute: "note"

Garrick Staples garrick at clusterresources.com
Tue Mar 20 18:44:08 MDT 2007


On Tue, Mar 20, 2007 at 05:34:31PM -0600, Marcus R. Epperson alleged:
> Garrick Staples wrote:
> >On Mon, Mar 19, 2007 at 09:59:17PM -0600, Marcus R. Epperson alleged:
> >>Garrick Staples wrote:
> >>>On Mon, Mar 19, 2007 at 02:02:21PM -0600, Marcus R. Epperson alleged:
> >>>>Do you have any suggestions for how to do this argument parsing 
> >>>>correctly?
> >>>Indeed, we definitely need to stay away from GNU extensions.
> >>>
> >>>How about we use lowercase n for listing, and capital N for setting?
> >>Ok, I think this is a lot cleaner.  Let me know what you think.
> >
> >I think this makes me all gushy inside.
> >
> >Since this doesn't apply to trunk, can you whip up another patch or
> >should I just copy 2.1-fixes' pbsnodes.c into trunk?
> 
> Here's one for 2.2.0-snap.200703191105.  Though I think the trunk's 
> pbsnodes -l is a little broken.  It seems to be doing strcmp() comparisons 
> on the state attribute instead of strstr().  So if a node is both down and 
> offline, it isn't shown.

Thanks for letting me know about broken pbsnodes.c.  It is fixed up in
svn.



More information about the torquedev mailing list