Project

Profile

Help

Issue #1289

install distributor does not normalize file permissions and ownership

Added by mhrivnak about 6 years ago. Updated over 2 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
High
Assignee:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
2.6.3
Platform Release:
2.8.0
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

When the install distributor extracts puppet modules from their tarballs, it does not make any attempt to normalize the file permissions of the files contained in the tarball. This can lead to at least three types of problems:

1. A file cannot be read by puppet.
2. Extracted files cannot be moved by pulp from the temporary extraction area to the permanent destination.
3. A file may have unsafe permissions, such as being world-writable.

Pulp should choose a standard way to normalize these permissions, perhaps ensuring that at a minimum the owner can rwx, and the world can r (and x for directories).

Associated revisions

Revision cef8643c View on GitHub
Added by mhrivnak about 6 years ago

Install distributor now normalizes uid, gid, and filesystem permissions.

Also resurrected a bunch of unit tests for the install distributor that had been removed during the mongoengine conversion.

fixes #1289

History

#1 Updated by mhrivnak about 6 years ago

  • Triaged changed from No to Yes

#3 Updated by mhrivnak about 6 years ago

  • Subject changed from install distributor does not normalize file permissions to install distributor does not normalize file permissions and ownership
  • Priority changed from Normal to High

#5 Updated by mhrivnak about 6 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to mhrivnak

#6 Updated by mhrivnak about 6 years ago

  • Status changed from ASSIGNED to POST

#7 Updated by mhrivnak about 6 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#8 Updated by rbarlow almost 6 years ago

  • Status changed from MODIFIED to 5

#9 Updated by dkliban@redhat.com over 5 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

#12 Updated by bmbouter over 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF