Uploaded image for project: 'Puppet'
  1. Puppet
  2. PUP-9267

The members property's 'retrieve' and 'should' methods should return arrays

    Details

    • Template:
    • Team:
      Platform OS
    • Sprint:
      Platform OS Kanban
    • Method Found:
      Needs Assessment
    • Release Notes:
      Bug Fix
    • Release Notes Summary:
      Hide
      The members property's been fixed to have the same API for retrieve and should as it did prior to 5.5.7's breaking changes while also reporting the right change notification. Specifically, providers can now return an array for the members property getter and accept an array for the members property setter (as before).
      Show
      The members property's been fixed to have the same API for retrieve and should as it did prior to 5.5.7's breaking changes while also reporting the right change notification. Specifically, providers can now return an array for the members property getter and accept an array for the members property setter (as before).
    • QA Risk Assessment:
      Needs Assessment

      Description

      https://github.com/puppetlabs/puppet/commit/da5c2df7e22f92e0bfbadf602930fe6b4c77885f refactored the members property to match the groups property of the User resource. This changed how the members property's data is internally represented. Specifically, retrieve and should now return a comma-separated string instead of an array. The core group providers were updated with this change; however, custom providers like e.g. https://github.com/onyxpoint/puppet-gpasswd broke.

      The work in this ticket should modify the members property to return the previous data representation instead (i.e. retrieve and should return an array as before).

        Attachments

          Issue Links

            Activity

              jsd-sla-details-panel

                People

                • Assignee:
                  enis.inan Enis Inan
                  Reporter:
                  enis.inan Enis Inan
                • Votes:
                  1 Vote for this issue
                  Watchers:
                  6 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: