Bug #25576: Pulp should handle cleanup of Puppet directories - Katello - Foreman

Bug #25576

Pulp should handle cleanup of Puppet directories

Added by Eric Helms over 4 years ago. Updated about 4 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Category:

-

Target version:

Difficulty:

Triaged:

Yes

Bugzilla link:

Pull request:

https://github.com/Katello/runcible/pull/214, https://github.com/theforeman/foreman-packaging/pull/3414, https://github.com/theforeman/foreman-packaging/pull/3408, https://github.com/Katello/katello/pull/7880

Fixed in Releases:

Found in Releases:

Red Hat JIRA:

Description

While the Pulp issue is fixed, I don't think we use that feature as we don't set the subdir config element in https://github.com/Katello/runcible/blob/master/lib/runcible/models/puppet_install_distributor.rb.

The patch should quite easy to create, but is nevertheless required.

See -- https://github.com/theforeman/foreman-packaging/pull/3185

Related issues

Associated revisions

Revision c0df0f4c (diff)
Added by Evgeni Golov over 4 years ago

Refs #25576 - allow passing "subdir" to PuppetInstallDistributor

The relevant pulp-puppet feature was added in 2.13 [1].
It allows pulp-puppet to better clean up the distributor.

[1] https://pulp.plan.io/issues/2108

Revision 42b1fcb7 (diff)
Added by Evgeni Golov about 4 years ago

Fixes #25576 - pass subdir=modules to PuppetInstallDistributor

The relevant pulp-puppet feature was added in 2.13 [1].
It allows pulp-puppet to better clean up the distributor.

This requires support in runcible [2].

[1] https://pulp.plan.io/issues/2108
[2] https://github.com/Katello/runcible/pull/214

History

#1 Updated by Christine Fouant over 4 years ago

Triaged changed from No to Yes
Target version set to Katello 3.11.0

#2 Updated by Evgeni Golov over 4 years ago

Assignee set to Evgeni Golov

#3 Updated by Evgeni Golov over 4 years ago

Pull request https://github.com/Katello/runcible/pull/214 added

#4 Updated by The Foreman Bot over 4 years ago

Status changed from New to Ready For Testing
Pull request https://github.com/Katello/katello/pull/7880 added

#5 Updated by Evgeni Golov about 4 years ago

Status changed from Ready For Testing to Closed

Applied in changeset katello|42b1fcb73452c06552b28b286726af43f0e5c413.

#6 Updated by The Foreman Bot about 4 years ago

Pull request https://github.com/theforeman/foreman-packaging/pull/3408 added

#7 Updated by The Foreman Bot about 4 years ago

Pull request https://github.com/theforeman/foreman-packaging/pull/3414 added

#8 Updated by Jonathon Turel over 3 years ago

Has duplicate Bug #17620: Content View Puppet repo not deleted from puppet master added

Also available in: Atom PDF