On Wed, Nov 02, 2016 at 10:00:54AM +0100, David Disseldorp wrote: > From: Joao Eduardo Luis <joao@xxxxxxx> > > Add basic CephFS support. No new CephFS specific tests are included, > just basic enablement, with ./check now accepting a -ceph parameter. We don't add command line parameters to specify the filesystem anymore if we can avoid it (they are legacy options, really). Specifying FSTYP=ceph in the environment or config file should be sufficient to do they right thing - does this work for ceph? > @@ -1499,6 +1508,15 @@ _require_scratch_nocheck() > _notrun "this test requires a valid \$SCRATCH_MNT" > fi > ;; > + ceph) > + echo $SCRATCH_DEV | grep -q ":/" > /dev/null 2>&1 > + if [ -z "$SCRATCH_DEV" -o "$?" != "0" ]; then > + _notrun "this test requires a valid \$SCRATCH_DEV" > + fi > + if [ ! -d "$SCRATCH_MNT" ]; then > + _notrun "this test requires a valid \$SCRATCH_MNT" > + fi > + ;; This is the same check as for nfs*). Can you make those common? > cifs) > echo $SCRATCH_DEV | grep -q "//" > /dev/null 2>&1 > if [ -z "$SCRATCH_DEV" -o "$?" != "0" ]; then > @@ -1584,6 +1602,15 @@ _require_test() > _notrun "this test requires a valid \$TEST_DIR" > fi > ;; > + ceph) > + echo $TEST_DEV | grep -q ":/" > /dev/null 2>&1 > + if [ -z "$TEST_DEV" -o "$?" != "0" ]; then > + _notrun "this test requires a valid \$TEST_DEV" > + fi > + if [ ! -d "$TEST_DIR" ]; then > + _notrun "this test requires a valid \$TEST_DIR" > + fi > + ;; Same. Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
