Project

General

Profile

Bug #25576

Pulp should handle cleanup of Puppet directories

Added by Eric Helms 10 months ago. Updated 8 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:

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

Has duplicate Katello - Bug #17620: Content View Puppet repo not deleted from puppet masterDuplicate

Associated revisions

Revision c0df0f4c (diff)
Added by Evgeni Golov 9 months 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 8 months 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 10 months ago

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

#2 Updated by Evgeni Golov 9 months ago

  • Assignee set to Evgeni Golov

#3 Updated by Evgeni Golov 9 months ago

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

#4 Updated by The Foreman Bot 9 months ago

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

#5 Updated by Evgeni Golov 8 months ago

  • Status changed from Ready For Testing to Closed

#6 Updated by The Foreman Bot 8 months ago

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

#7 Updated by The Foreman Bot 8 months ago

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

#8 Updated by Jonathon Turel 21 days ago

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

Also available in: Atom PDF