runTheMatrix.py `--command` option broken

[smuzaffar]

Looks like #35263 change broke the --command option e.g the igprof jobs for 12.1.X are failing with error

runTheMatrix.py -w upgrade -l 23434.21 --command '-n 10 --profile pp'
usage: usage: runTheMatrix.py --show -s  [-h] [-b BATCHNAME] [-m MEMORYOFFSET]
                                         [--addMemPerCore MEMPERCORE]
....
                                         [--force-cuda-runtime-version CUDARUNTIMEVERSION]
usage: runTheMatrix.py --show -s : error: argument --command: expected one argument

Strange thing is that the following works

• runTheMatrix.py -w upgrade -l 23434.21 --command ' -n 10 --profile pp'
  which shows that opt.command is -n 10 --profile pp

• runTheMatrix.py -w upgrade -l 23434.21 --command='-n 10 --profile pp'
  which shows that opt.command is -n 10 --profile pp

I will update the bot to add the extra space for igprof tests. I do not know if there is any fix to avoid this.

[smuzaffar]

FYI @srimanob

[cmsbuild]

A new Issue was created by @smuzaffar Malik Shahzad Muzaffar.

@Dr15Jones, @perrotta, @dpiparo, @makortel, @smuzaffar, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

[smuzaffar]

FYI @ddaina

[srimanob]

Thanks @smuzaffar
The only line that seems related to opt.command is
https://github.com/cms-sw/cmssw/blob/master/Configuration/PyReleaseValidation/scripts/runTheMatrix.py#L370

I will have a look.

[smuzaffar]

I think issue is with argparse ( https://github.com/cms-sw/cmssw/pull/35263/files#diff-d025624692806aeb382410b51b6bb2e5c5e783710d03f4c3f41400e6cce40310R107 )

[kpedro88]

@smuzaffar yes, the recommended practice for nesting arguments with argparse is the syntax with the = sign.

[smuzaffar]

closing this issue. bot has been fixed to avoid this issue.