Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Job identifiers potentially non-unique in HTCondor batchsystem adapter #101

Open
stefan-k opened this issue Oct 17, 2019 · 0 comments
Open
Labels
potential problem Avoid foreseeable misuse

Comments

@stefan-k
Copy link
Contributor

In htcondor_status_updater of the HTCondorAdapter (Batchsystem) there are two potentially dangerous lines during the looping over the rows of the status:

status_key = row['TardisDroneUuid'] or row['Machine'].split('.')[0]
htcondor_status[status_key] = row

If row['TardisDroneUuid'] is None, there could be several rows with equal row['Machine'], which would cause several rows to be assigned to the same key.

This could be solved by ensuring that TardisDroneUuid is always set to a unique value.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
potential problem Avoid foreseeable misuse
Projects
None yet
Development

No branches or pull requests

2 participants