clustermq (develop) torque/PBS- In worker cleanup, seems to be sending job names instead of PBS ids (Apparent regression of #186)

(develop) torque/PBS- In worker cleanup, seems to be sending job names instead of PBS ids (Apparent regression of #186)

Open strazto opened this issue 3 years ago • 3 comments

This appears to be a regression of #186 in the develop branch. Not sure what's changed

qdel: illegally formed job identifier: cmq7547

May 14 '21 05:05 strazto

Maybe #187 never properly fixed it? From what I can see, not much has changed

May 14 '21 05:05 strazto

Wasn't this your PR? :smile:

May 16 '21 17:05 mschubert

Wasn't this your PR? 😄

Yep! I swear it worked at the time, but somehow, for a reason that isn't clear to me, the job-name is being used for qdel instead of the id.

It could be a bug from some other change that somehow changed which QSys subclass is instantiated, but I haven't fully delved into it yet

May 17 '21 04:05 strazto

I think I fixed this now, if not please reopen

Apr 07 '23 19:04 mschubert