Re: [PATCH] Use qa_user and qa_group for test 219

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Dec 18, 2012 at 06:50:06PM +0100, Jan Kara wrote:
> On Tue 18-12-12 08:38:06, Brian Foster wrote:
> > On 12/18/2012 06:33 AM, Jan Kara wrote:
> > > Test 219 requires a special user. Use $qa_user and $qa_group (added in this
> > > patch) for that purpose instead of hardcoded uid & gid. This also fixes
> > > a false failure when repquota does not report quota for users not in passwd.
> > > 
> > > Signed-off-by: Jan Kara <jack@xxxxxxx>
> > > ---
> > ...
> > > +# check for the fsgqa group on the machine
> > > +#
> > > +_require_group()
> > > +{
> > > +    qa_group=fsgqa
> > > +    _cat_group | grep -q $qa_user
> > > +    [ "$?" == "0" ] || _notrun "$qa_user user not defined."
> > 
> > I assume you mean to use $qa_group instead of $qa_user in the above two
> > lines?
>   Bah, right. The names are the same so I didn't notice my copy-and-paste
> mistake :). Fixed version attached.

The 2nd rev looks good to me.  It makes sense that you'd remove the uid
specific golden output since fsgqa isn't going to be the same uid/gid on all
systems.

Reviewed-by: Ben Myers <bpm@xxxxxxx>

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs


[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux