HomePhorge

(PUP-3653) Create/set empty Windows groups
5c3ee97d1cb3Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

(PUP-3653) Create/set empty Windows groups

Allow setting a Windows group to no users. Previously it was impossible
to set Windows groups with no members due to the way the checking of
should versus current was done. Without this change Puppet will not be
able to set Windows groups to empty lists.

  • group/windows_adsi.members_insync? - Remove the short circuit check on current.empty? versus should_empty - as it results in incorrect behavior when non-authoritative
  • group/windows_adsi.members_insync? - Return true when non-athoritative and should is empty. This prevents a check on keys of an empty hash, which errors.
  • adsi.set_members - return early only if desired_members is nil
  • adsi.set_members - skip adding members when desired_members is empty
  • adsi.set_members - if desired_members is empty, remove all current members

Details

Provenance
Rob Reynolds <ferventcoder@gmail.com>Authored on
Ethan J. Brown <Iristyle@github>Committed on Apr 7 2015, 5:09 PM
vanmeeuwenPushed on Jun 2 2015, 2:22 PM
Parents
rPU22da8405a94e: Merge remote-tracking branch 'upstream/pr/3782' into 3.x
Branches
Unknown
Tags
Unknown

Event Timeline

Ethan J. Brown <Iristyle@github> committed rPU5c3ee97d1cb3: (PUP-3653) Create/set empty Windows groups (authored by Rob Reynolds <ferventcoder@gmail.com>).Apr 7 2015, 5:09 PM