teuthology-queue --pause traceback w/o --machine-type

[zmc]

Example:

teuthworker@teuthology:~$ teuthology-queue --pause 99999
Traceback (most recent call last):
  File "/home/teuthworker/src/git.ceph.com_git_teuthology_main/virtualenv/bin/teuthology-queue", line 8, in <module>
    sys.exit(main())
  File "/home/teuthworker/src/git.ceph.com_git_teuthology_main/scripts/queue.py", line 36, in main
    teuthology.beanstalk.main(args)
  File "/home/teuthworker/src/git.ceph.com_git_teuthology_main/teuthology/beanstalk.py", line 202, in main
    pause_tube(connection, machine_type, pause_duration)
  File "/home/teuthworker/src/git.ceph.com_git_teuthology_main/teuthology/beanstalk.py", line 164, in pause_tube
    tubes = sorted(connection.tubes())
TypeError: '<' not supported between instances of 'NoneType' and 'str'

[Annmool]

Looks good. I believe I can work on this. Kindly assign it to me.

[Annmool]

I’ve tried to fix the issue. Please have a look whenever you get a chance.

[kshtsk]

According to docs:

  -p, --pause SECONDS   Pause queues for a number of seconds. A value of 0
                        will unpause. If -m is passed, pause that queue,
                        otherwise pause all queues.

and

  -m, --machine_type MACHINE_TYPE [default: multi]
                        Which machine type queue to work on.

And usage:

usage: teuthology-queue -h
       teuthology-queue [-s|-d|-f] -m MACHINE_TYPE
       teuthology-queue [-r] -m MACHINE_TYPE
       teuthology-queue -m MACHINE_TYPE -D PATTERN
       teuthology-queue -p SECONDS [-m MACHINE_TYPE]

In all cases -m machine queue is mandatory option, however for thee --pause it is not and uses multi, so we either make -m mandatory for the -p or fix generating list of pipes for the machine_type multi.