From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Chinner Subject: Re: [PATCH] common: enable tests that require scratch device on CIFS Date: Thu, 13 Nov 2014 15:27:56 +1100 Message-ID: <20141113042756.GE28565@dastard> References: <1415617191-17246-1-git-send-email-eguan@redhat.com> <20141110160309.GI2863@dhcp-13-216.nay.redhat.com> <20141113032824.GB28565@dastard> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Eryu Guan , fstests-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, "linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" To: Steve French Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-cifs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: On Wed, Nov 12, 2014 at 10:08:34PM -0600, Steve French wrote: > On Wed, Nov 12, 2014 at 9:28 PM, Dave Chinner wrote: > > On Wed, Nov 12, 2014 at 05:26:13PM -0600, Steve French wrote: > >> I get newly enabled test 294 passing on cifs mounts to Samba. > >> > >> What is best way to get the mount options passed to the mount of > >> scratch dev so it doesn't prompt for it? > > > > # MOUNT_OPTIONS="-o ...." ./check -g auto > > much better - that worked - thanks > > I had been using TEST_FS_MOUNT_OPTS (so I could have multiple sections > in local.config) rather > than MOUNT_OPTIONS BTW, why aren't you using CIFS_MOUNT_OPTIONS to define the login creds for both the test and scratch devices? That's what it was added for.... Cheers, Dave. -- Dave Chinner david-FqsqvQoI3Ljby3iVrkZq2A@public.gmane.org