Puppet: System Administration Automated

Support

Ticket #1169 (closed defect: fixed)

Opened 6 months ago

Last modified 6 months ago

Handling of High GIDs (and probably UIDs) for File is broken

Reported by: arogge Assigned to: luke
Priority: normal Milestone:
Component: library Version: 0.24.4
Severity: normal Keywords: file gid
Cc: Triage Stage: Needs design decision
Attached Patches: Code Complexity: Medium

Description

When trying to define a File's group as a group with a high id (I tested a group with GID 1001660) you get a "Tried to get gid field for silly id 1001660" (from Puppet::Util::POSIX get_posix_field) because there's a check that stops you from using high UID/GID.

In file/group.rb there's also a check for high UID/GID. As I have to use at least high GIDs in puppet, this is some kind of show-stopper for me...

I could probably write an appropriate patch if I knew why these checks are actually there.

I report this issue for 0.23.2 because that's what I'm using. However, it seems like this issue is still present in the current release.

Attachments

0001-Extend-workaround-from-56aad69f8cdf8b0b08fdb79850149.patch (2.7 kB) - added by arogge on 04/02/08 20:02:17.
Patch

Change History

04/02/08 20:02:17 changed by arogge

  • attachment 0001-Extend-workaround-from-56aad69f8cdf8b0b08fdb79850149.patch added.

Patch

04/02/08 20:08:08 changed by arogge

  • complexity changed from Unknown to Trivial.
  • version changed from 0.23.2 to 0.24.4.
  • patch changed from None to Code.

I added a patch that works for my installation. It should also fix an issue with owner-change when the uid > 1000000 

Extend workaround from 56aad69f8cdf8b0b08fdb7985014986223fa4455 to not only fix UIDs,
    but also GIDs
Fixes #1169
Fixes #804 (workaround was probably incomplete, as required changes in 
            lib/puppet/util/posix.rb were missing)

04/03/08 12:37:46 changed by jamtur01

  • owner changed from community to luke.
  • complexity changed from Trivial to Medium.
  • stage changed from Unreviewed to Needs design decision.

This is a duplicate of #804 but I'll leave the patch for Luke to make a call on.

04/03/08 17:39:38 changed by luke

  • status changed from new to closed.
  • resolution set to fixed.

Patch applied.