[Ocfs2-test-devel] [PATCH 1/1] ocfs2-test: making all multiple runners be aware of newly added interface argument.

Tristan Ye tristan.ye at oracle.com
Sat May 14 06:34:09 PDT 2011


On 05/14/2011 01:12 AM, Sunil Mushran wrote:
> Can it handle the case in which eth0 is unused. We have some
> machine with flaky onboard cards which get assigned eth0 but
> are not used.


Sure it does;-), default interface was set to be eth0, however, just
specify other interface explicitly in case of unused eth0 by means of
'-i ethx'


> 
> On 05/13/2011 12:21 AM, Tristan Ye wrote:
>> as ommon func 'openmpi_run()' is supporting a new agrument of
>> interface used for
>> MPI messaging, corresponding modification efforts have to be made to
>> teach all
>> multiple_nodes runners become aware of this new change, also
>> multiple_run.sh caller
>> needs to be more sophisticated to recognize new interface;-)
>>
>> Signed-off-by: Tristan Ye<tristan.ye at oracle.com>
>> ---
>>   programs/cross_delete/cross_delete.py              |    2 +-
>>   programs/flock_tests/run_flock_unit_test.py        |   12 ++++++++++++
>>   programs/multi_node_mmap/run_multi_mmap.py         |   12 ++++++++++++
>>   programs/open_delete/open_delete.py                |   11 +++++++++++
>>   programs/python_common/multiple_run.sh             |   14
>> +++++++-------
>>   .../run_write_append_truncate.py                   |   12 ++++++++++++
>>   6 files changed, 55 insertions(+), 8 deletions(-)
>>
>> diff --git a/programs/cross_delete/cross_delete.py
>> b/programs/cross_delete/cross_delete.py
>> index 1ce00cf..9582c67 100755
>> --- a/programs/cross_delete/cross_delete.py
>> +++ b/programs/cross_delete/cross_delete.py
>> @@ -327,8 +327,8 @@ for y in range(count):
>>           (cmdline, stagedir, options.logfile) ),
>>           ','.join(nodelist),
>>           'ssh',
>> -        logfile,
>>           options.interface,
>> +        logfile,
>>           'WAIT')
>>       if not ret:
>>           o2tf.printlog(
>> diff --git a/programs/flock_tests/run_flock_unit_test.py
>> b/programs/flock_tests/run_flock_unit_test.py
>> index ca576ba..5f13255 100644
>> --- a/programs/flock_tests/run_flock_unit_test.py
>> +++ b/programs/flock_tests/run_flock_unit_test.py
>> @@ -37,12 +37,14 @@ DEBUGON = os.getenv('DEBUG',0)
>>   uname = os.uname()
>>   lhostname = str(socket.gethostname())
>>   logfile = config.LOGFILE
>> +interface = 'eth0'
>>   procs = 1
>>   cmd = config.BINDIR+'/flock_unit_test'
>>   #
>>   Usage = """
>>   %prog
>>   [-l | --logfile<logfile>]
>> +[-i | --interface<interface>]
>>   [-n | --nodelist<nodelist>]
>>   [-t | --type]              lock type:should be flock or fcntl.
>>   [-e | --file1<filename1>]
>> @@ -67,6 +69,12 @@ if __name__=='__main__':
>>           type='string',
>>           help='Logfile used by the process.')
>>   #
>> +    parser.add_option('-i',
>> +        '--interface',
>> +        dest='interface',
>> +        type='string',
>> +        help='NIC used for MPI messaging.')
>> +#
>>       parser.add_option('-n',
>>           '--nodelist',
>>           dest='nodelist',
>> @@ -129,6 +137,9 @@ if __name__=='__main__':
>>           if not options.cleanup:
>>               parser.error('Invalid node list.')
>>
>> +    if options.interface:
>> +        interface = options.interface
>> +
>>   if DEBUGON:
>>       o2tf.printlog('flock_unit_test: main - current directory %s' %
>>           os.getcwd(), logfile, 0, '')
>> @@ -145,6 +156,7 @@ ret = o2tf.openmpi_run(DEBUGON, procs,
>>       logfile)),
>>       options.nodelist,
>>       'ssh',
>> +    interface,
>>       logfile,
>>       'WAIT')
>>   #
>> diff --git a/programs/multi_node_mmap/run_multi_mmap.py
>> b/programs/multi_node_mmap/run_multi_mmap.py
>> index bd6f769..23aac13 100644
>> --- a/programs/multi_node_mmap/run_multi_mmap.py
>> +++ b/programs/multi_node_mmap/run_multi_mmap.py
>> @@ -36,6 +36,7 @@ DEBUGON = os.getenv('DEBUG',0)
>>   uname = os.uname()
>>   lhostname = str(socket.gethostname())
>>   logfile = config.LOGFILE
>> +interface = 'eth0'
>>   procs = 1
>>   cmd = config.BINDIR+'/multi_mmap'
>>   #
>> @@ -44,6 +45,7 @@ Usage = """
>>   [-i | --iterations<iterations>]
>>   [-C | --cleanup]
>>   [-l | --logfile<logfile>]
>> +[-I | --interface<interface>]
>>   [-n | --nodelist<nodelist>]
>>   [-t | --truncate]          Don\'t create or trunc the file - will
>> fail if it doesn\'t exist.
>>   [-c | --cache]               Populate the local cache by reading the
>> full file first.
>> @@ -138,6 +140,12 @@ if __name__=='__main__':
>>           type='string',
>>           help='Logfile used by the process.')
>>   #
>> +    parser.add_option('-I',
>> +        '--interface',
>> +        dest='interface',
>> +        type='string',
>> +        help='NIC used by MPI messaging.')
>> +#
>>       parser.add_option('-n',
>>           '--nodelist',
>>           dest='nodelist',
>> @@ -221,6 +229,9 @@ if __name__=='__main__':
>>       if options.logfile:
>>           logfile = options.logfile
>>
>> +    if options.interface:
>> +        interface = options.interface
>> +
>>       if options.nodelist:
>>           nodelist = options.nodelist.split(',')
>>           nodelen = len(nodelist)
>> @@ -257,6 +268,7 @@ ret = o2tf.openmpi_run(DEBUGON, procs,
>>       logfile)),
>>       options.nodelist,
>>       'ssh',
>> +    interface,
>>       logfile,
>>       'WAIT')
>>   #
>> diff --git a/programs/open_delete/open_delete.py
>> b/programs/open_delete/open_delete.py
>> index 5772643..d9f12dc 100755
>> --- a/programs/open_delete/open_delete.py
>> +++ b/programs/open_delete/open_delete.py
>> @@ -42,10 +42,12 @@ uname = os.uname()
>>   lhostname = str(socket.gethostname())
>>   numnodes = 0
>>   logfile = config.LOGFILE
>> +interface = 'eth0'
>>   #
>>   Usage = '\n     %prog [-l|-logfile logfilename] \
>>   [-f | --file filename] \
>>   [-i | --interactions count] \
>> +[-I | --interface] \
>>   [-n | --nodes nodelist] \
>>   [-h|--help]'
>>   #
>> @@ -77,6 +79,12 @@ if __name__=='__main__':
>>               type='string',
>>               help='Logfile used by the process.')
>>   #
>> +    parser.add_option('-I',
>> +            '--interface',
>> +            dest='interface',
>> +            type='string',
>> +            help='Nic used by MPI messaging.')
>> +#
>>       parser.add_option('-n',
>>               '--nodes',
>>               dest='nodelist',
>> @@ -89,6 +97,8 @@ if __name__=='__main__':
>>           parser.error('incorrect number of arguments')
>>       if options.logfile:
>>           logfile = options.logfile
>> +    if options.interface:
>> +        interface = options.interface
>>       count = options.count
>>       filename = options.filename
>>       if options.nodelist:
>> @@ -115,6 +125,7 @@ ret = o2tf.openmpi_run(DEBUGON,
>>       filename) ),
>>       options.nodelist,
>>       'ssh',
>> +    interface,
>>       options.logfile,
>>       'WAIT')
>>   if not ret:
>> diff --git a/programs/python_common/multiple_run.sh
>> b/programs/python_common/multiple_run.sh
>> index a6912a8..26b66c2 100755
>> --- a/programs/python_common/multiple_run.sh
>> +++ b/programs/python_common/multiple_run.sh
>> @@ -289,7 +289,7 @@ run_open_delete_test()
>>       local logfile=${logdir}/open_delete_${DATE}.log
>>
>>       run_common_testcase "open_delete" "sparse,unwritten,inline-data" \
>> -"${BINDIR}/open_delete.py -f ${testfile} -i 10000 -l ${logfile} -n
>> ${NODE_LIST}"
>> +"${BINDIR}/open_delete.py -f ${testfile} -i 10000 -I ${INTERFACE} -l
>> ${logfile} -n ${NODE_LIST}"
>>
>>   }
>>
>> @@ -300,7 +300,7 @@ run_cross_delete_test()
>>       local workplace=${MOUNT_POINT}/cross_delete_test
>>
>>       run_common_testcase "cross_delete" "sparse,unwritten,inline-data" \
>> -"${BINDIR}/cross_delete.py -c 1 -d ${workplace} -n ${NODE_LIST} -t
>> ${KERNELSRC}"
>> +"${BINDIR}/cross_delete.py -c 1 -i ${INTERFACE} -d ${workplace} -n
>> ${NODE_LIST} -t ${KERNELSRC}"
>>   }
>>
>>   run_write_append_truncate_test()
>> @@ -311,7 +311,7 @@ run_write_append_truncate_test()
>>       local testfile=${workplace}/write_append_truncate_test_file
>>
>>       run_common_testcase "write_append_truncate"
>> "sparse,unwritten,inline-data" \
>> -"${BINDIR}/run_write_append_truncate.py -i 20000 -l ${logfile} -n
>> ${NODE_LIST} -f ${testfile}"
>> +"${BINDIR}/run_write_append_truncate.py -i 20000 -I ${INTERFACE} -l
>> ${logfile} -n ${NODE_LIST} -f ${testfile}"
>>   }
>>
>>   run_multi_mmap_test()
>> @@ -322,7 +322,7 @@ run_multi_mmap_test()
>>       local testfile=${workplace}/multi_mmap_test_file
>>
>>       run_common_testcase "multi_mmap" "sparse,unwritten,inline-data" \
>> -"${BINDIR}/run_multi_mmap.py -i 20000 -n ${NODE_LIST} -c -b 6000
>> --hole -f ${testfile}"
>> +"${BINDIR}/run_multi_mmap.py -i 20000 -I ${INTERFACE} -n ${NODE_LIST}
>> -c -b 6000 --hole -f ${testfile}"
>>   }
>>
>>   run_create_racer_test()
>> @@ -332,7 +332,7 @@ run_create_racer_test()
>>       local workplace=${MOUNT_POINT}/create_racer_test
>>
>>       run_common_testcase "create_racer" "sparse,unwritten,inline-data" \
>> -"${BINDIR}/run_create_racer.py -i 40000 -l ${logfile} -n ${NODE_LIST}
>> -p ${workplace}"
>> +"${BINDIR}/run_create_racer.py -c 40000 -i ${INTERFACE} -l ${logfile}
>> -n ${NODE_LIST} -p ${workplace}"
>>   }
>>
>>   run_xattr_test()
>> @@ -412,7 +412,7 @@ ${DEVICE} ${FEATURES} ${JOURNALSIZE} ${BLOCKS}
>>
>>       LogMsg "Run lvb_torture, CMD: ${BINDIR}/run_lvb_torture.py -d
>> /dlm/ -i 60000 \
>>   -H ${DEVICE} -l ${logfile} -n ${NODE_LIST} "${UUID}" "${LOCK}""
>> -    ${BINDIR}/run_lvb_torture.py -d /dlm/ -i 60000 -H ${DEVICE} -l \
>> +    ${BINDIR}/run_lvb_torture.py -d /dlm/ -c 60000 -i ${INTERFACE} -H
>> ${DEVICE} -l \
>>   ${logfile} -n ${NODE_LIST} "${UUID}" "${LOCK}">>  ${LOGFILE} 2>&1
>>       LogRC $?
>>
>> @@ -437,7 +437,7 @@ run_flock_unit_test()
>>       run_common_testcase "flock_unit" "sparse,unwritten,inline-data" \
>>   "${TOUCH_BIN} ${testfile1}&&  ${TOUCH_BIN} ${testfile2}&& 
>> ${BINDIR}/run_flock_unit_test.py \
>>   -l ${fcntl_logfile} -n ${NODE_LIST} -t fcntl -e ${testfile1} -f
>> ${testfile2} \
>> -&&  ${BINDIR}/run_flock_unit_test.py -l ${flock_logfile} -n
>> ${NODE_LIST} -t \
>> +&&  ${BINDIR}/run_flock_unit_test.py -l ${flock_logfile} -i
>> ${INTERFACE} -n ${NODE_LIST} -t \
>>   flock -e ${testfile1} -f ${testfile2}"
>>   }
>>
>> diff --git
>> a/programs/write_append_truncate/run_write_append_truncate.py
>> b/programs/write_append_truncate/run_write_append_truncate.py
>> index 981a42a..60f3739 100644
>> --- a/programs/write_append_truncate/run_write_append_truncate.py
>> +++ b/programs/write_append_truncate/run_write_append_truncate.py
>> @@ -35,12 +35,14 @@ DEBUGON = os.getenv('DEBUG',0)
>>   uname = os.uname()
>>   lhostname = str(socket.gethostname())
>>   logfile = config.LOGFILE
>> +interface = 'eth0'
>>   procs = 1
>>   cmd = config.BINDIR+'/write_append_truncate'
>>   #
>>   Usage = """
>>   %prog
>>   [-i | --iterations<iterations>]
>> +[-I | --interface<interface>]
>>   [-C | --cleanup]
>>   [-l | --logfile<logfile>]
>>   [-n | --nodelist<nodelist>]
>> @@ -84,6 +86,12 @@ if __name__=='__main__':
>>           type='string',
>>           help='Logfile used by the process.')
>>   #
>> +    parser.add_option('-I',
>> +        '--interface',
>> +        dest='interface',
>> +        type='string',
>> +        help='NIC used by MPI messaging.')
>> +#
>>       parser.add_option('-n',
>>           '--nodelist',
>>           dest='nodelist',
>> @@ -119,6 +127,9 @@ if __name__=='__main__':
>>       if options.logfile:
>>           logfile = options.logfile
>>
>> +    if options.interface:
>> +        interface = options.interface
>> +
>>       if options.nodelist:
>>           nodelist = options.nodelist.split(',')
>>           nodelen = len(nodelist)
>> @@ -148,6 +159,7 @@ ret = o2tf.openmpi_run(DEBUGON, procs,
>>       logfile)),
>>       options.nodelist,
>>       'ssh',
>> +    interface,
>>       logfile,
>>       'WAIT')
>>   #
> 




More information about the Ocfs2-test-devel mailing list