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.

Reply via email to