jobsub_prio still uses optparse
With jobsub 1.3, all server and client libraries were transitioned from optparse to argparse.
Except for jobsub_prio. We somehow missed that. Transition that over, and while we're doing this, remove all the commented-out optparse code from the client and server.
#1 Updated by Shreyas Bhat 6 months ago
In addition to this, there was a bug introduced in commit 6da965bd where we commented out the option for a uid. The call to jobsubClient.adjust_prio expects a uid to be provided (previously the default was None). We should change line 183 so that for now, we don't provide a UID, so that call would be:
http_code = js_client.adjust_prio(jobid=jid, uid=None)
That will fix the bug Alex saw.