Issue #3406
closed
/var/cache/pulp should not be required by pulp workers
Description
In Pulp 2, /var/cache/pulp was used by workers to create symlinks before copying them to /var/lib/pulp. Since Pulp 3 is not going to be creating symlinks all the time, it's reasonable to believe that workers can use /var/lib/pulp/tmp as the root of their working directories.
We should switch the code that provides the working directory for a worker to return /var/lib/pulp/tmp/<workername>
The Artifact upload API already uses /var/lib/pulp/tmp as the temporary storage.
Updated by bmbouter almost 7 years ago
To avoid an artifact and worker name collision (which is unlikely), maybe the artifact path should go one level deeper to be /var/lib/pulp/tmp/artifacts/. Just an idea.
Updated by amacdona@redhat.com almost 7 years ago
This will affect pulp/devel also:
https://github.com/pulp/devel/blob/3.0-dev/ansible/roles/systemd/tasks/main.yml#L12
Updated by dalley almost 7 years ago
- Subject changed from /var/cache/pulp shoudl not be required by pulp workers to /var/cache/pulp should not be required by pulp workers
- Sprint/Milestone set to 56
- Triaged changed from No to Yes
Updated by jortel@redhat.com almost 7 years ago
- Sprint changed from Sprint 33 to Sprint 34
Updated by daviddavis almost 7 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to daviddavis
Updated by daviddavis almost 7 years ago
bmbouter I think we're good. Looks like artifacts won't create a collision as the storage service uses /var/lib/pulp/tmp/<task id>/:
https://github.com/pulp/pulp/blob/3.0-dev/pulpcore/pulpcore/tasking/services/storage.py#L130-L148
Added by daviddavis almost 7 years ago
Updated by daviddavis almost 7 years ago
- Status changed from ASSIGNED to POST
Added by daviddavis almost 7 years ago
Revision 4ec2f54b | View on GitHub
Using /var/lib/pulp/tmp instead of /var/cache/pulp
Added by daviddavis almost 7 years ago
Revision 4ec2f54b | View on GitHub
Using /var/lib/pulp/tmp instead of /var/cache/pulp
Updated by daviddavis almost 7 years ago
- Status changed from POST to MODIFIED
Applied in changeset pulp|4ec2f54be5b1341a832d5e4a3aa0c9c6e62ce990.
Updated by bmbouter about 5 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
.
Drop creation of the /var/cache/pulp directory
ref #3406 https://pulp.plan.io/issues/3406