Bug 1135513

Summary: [RFE] ability to remove puppet module from repository
Product: Red Hat Satellite Reporter: David Juran <djuran>
Component: Content ManagementAssignee: David Davis <daviddavis>
Status: CLOSED ERRATA QA Contact: Kedar Bidarkar <kbidarka>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.0.3CC: andrew.schofield, bkearney, chrobert, cwelton, daviddavis, djuran, kabbott, kbidarka, mfuruta, mmccune, riehecky, stijn.geselle, xdmoon
Target Milestone: UnspecifiedKeywords: FutureFeature
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
URL: http://projects.theforeman.org/issues/7810
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-08-12 05:15:34 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1139277    

Description David Juran 2014-08-29 13:47:16 UTC
Description of problem:
Currently there is no way of removing a puppet module from a repository. Having this ability would be useful

Additional info:
As pointed out by Walden Reines:

You can choose not to include the puppet module when constructing a content view which, for all intents and purposes, is like removing it from the repository.  You can also upload or sync a new version (or a version from a separate publisher) and select that version instead.

Comment 1 RHEL Program Management 2014-08-29 13:53:06 UTC
Since this issue was entered in Red Hat Bugzilla, the release flag has been
set to ? to ensure that it is properly evaluated for this release.

Comment 3 Andrew Schofield 2014-12-09 19:06:17 UTC
I understand the workaround as suggested by Walden but I'd like to know when this will be committed to a release. We really should be able to clear repos out of puppet modules which we may no longer wish for our downstream clients to have any access too.

Comment 4 David Juran 2015-01-07 16:09:51 UTC
Clearing needinfo

Comment 6 Mike McCune 2015-01-07 18:43:40 UTC
This is actually available upstream and will be shipped with Satellite 6.1, moving to 6.1 release.

Comment 7 Bryan Kearney 2015-01-09 14:43:30 UTC
Upstream bug assigned to daviddavis

Comment 8 Bryan Kearney 2015-01-09 15:53:13 UTC
Connecting redmine issue http://projects.theforeman.org/issues/5403 from this bug

Comment 9 Bryan Kearney 2015-01-09 15:54:19 UTC
Connecting redmine issue http://projects.theforeman.org/issues/5403 from this bug

Comment 10 Bryan Kearney 2015-01-09 15:56:38 UTC
The upstream bug is a dupe, but it was fixed by another. I am moving this to POST so that it is moved over when all the other POSTS are moved over.

Comment 13 Kedar Bidarkar 2015-02-13 11:28:36 UTC
We can now remove puppet-modules from puppet repos, with the "manage puppet modules" button.

Had synced content from a puppet-link which had 4 puppet modules and then removed 2 puppet modules via the "manage puppet modules" button.

We can see from the below result that the count is now just 2.

---------------
[root@xszas puppet_module]# hammer -u admin -p changeme repository info --id 11 --organization-id 1
/usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.4/lib/hammer_cli/./apipie/../abstract.rb:68: warning: already initialized constant DEFAULT_LABEL_INDENT
ID:                 11
Name:               puppet_repo
Label:              puppet_repo
Organization:       Default Organization
Red Hat Repository: no
Content Type:       puppet
URL:                http://cxds.fedorapeople.org/repos/random_puppet/
Publish Via HTTP:   yes
Published At:       http://xxxx.redhat.com/pulp/repos/Default_Organization/Library/custom/puppet_prd/puppet_repo
Product:            
    ID:   2
    Name: puppet_prd
GPG Key:            

Sync:               
    Status:         
    Last Sync Date: 2015/02/13 09:34:39
Created:            2015/02/13 09:34:10
Updated:            2015/02/13 09:34:11
Content Counts:     
    Puppet Modules: 2


----------------
removing of puppet-module even with hammer CLI is also now possible.

removed 1 puppet-module via cli.

-------------------

[root@xxxx puppet_module]# hammer -u admin -p changeme repository remove-content --content-ids e96ac298-b2ce-4d1b-a995-b7432d6ffbab --id 11 --organization-id 1
/usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.4/lib/hammer_cli/./apipie/../abstract.rb:68: warning: already initialized constant DEFAULT_LABEL_INDENT
Repository content removed



[root@xxxx puppet_module]# hammer -u admin -p changeme repository info --id 11 --organization-id 1
/usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.4/lib/hammer_cli/./apipie/../abstract.rb:68: warning: already initialized constant DEFAULT_LABEL_INDENT
ID:                 11
Name:               puppet_repo
Label:              puppet_repo
Organization:       Default Organization
Red Hat Repository: no
Content Type:       puppet
URL:                http://yy.fedorapeople.org/repos/random_puppet/
Publish Via HTTP:   yes
Published At:       http://xxxx.redhat.com/pulp/repos/Default_Organization/Library/custom/puppet_prd/puppet_repo
Product:            
    ID:   2
    Name: puppet_prd
GPG Key:            

Sync:               
    Status:         
    Last Sync Date: 2015/02/13 09:34:39
Created:            2015/02/13 09:34:10
Updated:            2015/02/13 09:34:11
Content Counts:     
    Puppet Modules: 1


VERIFIED with Sat6.1 Beta SNAP2

Comment 14 Bryan Kearney 2015-02-18 22:06:26 UTC
Connecting redmine issue http://projects.theforeman.org/issues/7810 from this bug

Comment 15 Andrew Schofield 2015-05-01 18:00:45 UTC
Confirmed, thanks.

Comment 16 Bryan Kearney 2015-08-11 13:28:32 UTC
This bug is slated to be released with Satellite 6.1.

Comment 17 errata-xmlrpc 2015-08-12 05:15:34 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2015:1592