[Devel] Re: [lxc-devel] [PATCH 1/1] lxc-ps: Nit - Avoid blank CONTAINER column

[Michel Normand]

Le jeudi 11 février 2010 à 11:08 +0100, Michel Normand a écrit :
> Le mercredi 10 février 2010 à 20:25 -0800, Sukadev Bhattiprolu a écrit :
> > From: Sukadev Bhattiprolu <sukadev at linux.vnet.ibm.com>
> > Date: Wed, 10 Feb 2010 20:09:17 -0800
> > Subject: [PATCH 1/1] lxc-ps: Nit - Avoid blank CONTAINER column
> > 
> > The contianer name is not printed in the lxc-ps output unless the --lxc
> > option is specified.  But the CONTAINER column is printed (and left
> > blank) even if the --lxc option is not specified.
> > 
> > 	$ /usr/local/bin/lxc-ps -n foo
> > 	CONTAINER    PID TTY          TIME CMD
> > 		   19525 pts/2    00:00:01 bash
> > 		   20311 pts/2    00:00:00 lxc-ps
> > 		   20312 pts/2    00:00:00 ps
> 
> For me the error in the script is 
> to report default pid of ps command,
> rather than to report the pids of specified 'foo' 
> container in this case.
> 
> So a patch should address this point,
> rather to remove the container column.

In fact there is no error in the script :)
The -n option for lxc-ps is a ps option ! 
and not the --name option that I assume you should have specified
as detailed in the online help of lxc-ps.

---
Michel

> 
> ---
> Michel
> 
> > 
> > Like normal ps, it would be less confusing (IMHO) if the CONTAINER
> > column is skipped unless the --lxc option is specified.
> > 
> > 	$ ./lxc-ps -n foo
> > 	  PID TTY          TIME CMD
> > 	19525 pts/2    00:00:01 bash
> > 	20339 pts/2    00:00:00 lxc-ps
> > 	20340 pts/2    00:00:00 ps
> > 
> > 	$ ./lxc-ps --lxc -n foo
> > 	CONTAINER    PID TTY          TIME CMD
> > 	foo        19646 pts/1    00:00:00 lxc-init
> > 	foo        19647 pts/1    00:00:00 bash
> > 
> > Signed-off-by: Sukadev Bhattiprolu <sukadev at linux.vnet.ibm.com>
> > ---
> >  src/lxc/lxc-ps.in |   12 ++++++++++--
> >  1 files changed, 10 insertions(+), 2 deletions(-)
> > 
> > diff --git a/src/lxc/lxc-ps.in b/src/lxc/lxc-ps.in
> > index 249b104..e1aa655 100755
> > --- a/src/lxc/lxc-ps.in
> > +++ b/src/lxc/lxc-ps.in
> > @@ -124,7 +124,11 @@ sub get_container {
> >  }
> >  
> >  sub display_headers {
> > -    printf "%-10s %s", "CONTAINER", $PS_HEADERS;
> > +    if ($LXC_DISPLAY) {
> > +        printf "%-10s %s", "CONTAINER", $PS_HEADERS;
> > +    } else {
> > +        printf "%s", $PS_HEADERS;
> > +    }
> >  }
> >  
> >  sub display_usage {
> > @@ -195,7 +199,11 @@ for (@PS_LINES) {
> >      my $container = get_container $a[$PS_PID_INDEX];
> >      if ($LXC_DISPLAY == 2 and not $LXC_NAMES{$container}) {next;}
> >      if ($LXC_DISPLAY == 1 and $container eq '') {next;}
> > -    printf "%-10s %s", $container, $_;
> > +    if ($LXC_DISPLAY) {
> > +        printf "%-10s %s", $container, $_;
> > +    } else {
> > +        printf "%s", $_;
> > +    }
> >  }
> >  
> >  exit 0;
> 
> 
> 
> ------------------------------------------------------------------------------
> SOLARIS 10 is the OS for Data Centers - provides features such as DTrace,
> Predictive Self Healing and Award Winning ZFS. Get Solaris 10 NOW
> http://p.sf.net/sfu/solaris-dev2dev
> _______________________________________________
> Lxc-devel mailing list
> Lxc-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/lxc-devel


_______________________________________________
Containers mailing list
Containers at lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers