leoluk / configmaster

Enterprise-grade network device config management.
2 stars 1 forks source link

Make device locking mechanism more robust #47

Closed rockTA closed 5 years ago

rockTA commented 5 years ago

Sometimes admins would run configmaster as root users which would lead to lock files being inaccessible for subsequent runs of the normal configmaster user. This is now avoided by explicitly making the lockfiles readable and writable by anyone.

Also added a try-finally block to make sure that locks are always released, as not doing so could lead to blocking and timeout if several tasks would be run for one device with one task failing.

CON-41

rockTA commented 5 years ago

Removed chmod call