search

WIPACrepo / pyglidein

Some python scripts to launch HTCondor glideins
MIT License
6 stars 20 forks source link

Fixed unsafe dict lookup (another one). #135

Closed samary closed 6 years ago

samary commented 6 years ago

Fixed unsafe dict lookup (another one). Related to #126.

dsschult commented 6 years ago

I'm not sure why exactly these dict lookups are all unsafe. There is a value in the default config, so it should always be defined. (and if we're not loading the default config, I consider that case an error)

samary commented 6 years ago

Yes, #129 should have set the value in the default config file but it seems it haven't made his way to master...

dsschult commented 6 years ago

That's easily fixed. @hskarlupka just needs to merge his dev branch at some point.

samary commented 6 years ago

Yes, it seems I just missed it was in not merged master yet. Closing this one.