I'm sending out this patch-set in whole again as I have modified them quite extensively and I've also added a new patch to the set.
- make it clear in the design doc that the empty initial value means that feature is disabled (and that's because there is no sensible default value) - update module docstring in lib/uidpool.py to be more generic - move the code to add/remove uids/uid-ranges to lib/uidpool.py - add '<uid pool definition>' to the documentation of --add-uids/--remove-uids - implement ExecWithUnusedUid function - rename --add-uid to --add-uids - rename --remove-uid to --remove-uids - use utils.PathJoin() instead of direct string manipulation - check for string formatting style: space should be on the next line - change ssconf uid-pool storage format from comma separated to newline separated - move the LockedUid class from lib/objects.py to lib/uidpool.py - explicitly close the lock file on uid.Unlock() to avoid fd leak - error out in --remove-uid if a uid/uid-range to be removed doesn't exist in the uid pool - do an explicit Close() in ReleaseUid() - move lock files from /var/lock/ganeti-uid-pool to /var/run/ganeti/uid-pool - move UIDMIN/UIDMAX constants from "Add --uid-pool option..." patch to "Add lib/uidpool.py..." patch - sort the uid pool in ParseUidPool() before returning it - change low level sections to paragraphs in the design doc Balazs Lecz (11): Add user-id pool design doc Add uid_pool cluster parameter Add lib/uidpool.py module Add --uid-pool option to gnt-cluster modify Add --add-uids/--remove-uids to gnt-cluster modify Add printing of uid_pool to gnt-cluster info Add --uid-pool option to gnt-cluster init Add uid_pool to ssconf Add basic unittests for uid_pool Manage the assignment of uids from the uid pool Add ExecuteWithUnusedUid() to uidpool.py Makefile.am | 2 + doc/design-2.1.rst | 155 +++++++++++++++++- lib/bootstrap.py | 4 +- lib/cli.py | 20 +++ lib/cmdlib.py | 19 ++ lib/config.py | 4 + lib/constants.py | 8 + lib/objects.py | 4 + lib/opcodes.py | 3 + lib/ssconf.py | 11 ++ lib/uidpool.py | 353 +++++++++++++++++++++++++++++++++++++++ scripts/gnt-cluster | 32 ++++- test/ganeti.config_unittest.py | 1 + test/ganeti.uidpool_unittest.py | 48 ++++++ 14 files changed, 659 insertions(+), 5 deletions(-) create mode 100644 lib/uidpool.py create mode 100755 test/ganeti.uidpool_unittest.py -- To unsubscribe, reply using "remove me" as the subject.