Bug 1388951

Summary:	rhn mirror role switch is still displayed in the product
Product:	Red Hat CloudForms Management Engine	Reporter:	Josh Carter <jocarter>
Component:	Appliance	Assignee:	Gregg Tanzillo <gtanzill>
Status:	CLOSED NOTABUG	QA Contact:	Aziza Karol <akarol>
Severity:	high	Docs Contact:
Priority:	high
Version:	5.7.0	CC:	abellott, bdunne, cpelland, hkataria, jhardy, mpovolny, ncarboni, obarenbo
Target Milestone:	GA	Flags:	bdunne: needinfo? (jhardy)
Target Release:	cfme-future
Hardware:	All
OS:	All
Whiteboard:	settings
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-11-09 20:34:50 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:

Description Josh Carter 2016-10-26 14:33:26 UTC

Description of problem:

Since CF pulled RHN support when registering the product we should pull the rhn mirror role from the product. 

Version-Release number of selected component (if applicable):
 Version 5.7.0.7-beta1.20161025153249_9376fbd 

How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 2 Harpreet Kataria 2016-10-26 20:10:11 UTC

UI calls model method https://github.com/ManageIQ/manageiq/blob/master/app/controllers/ops_controller/settings/common.rb#L1103 to get list of roles to be displayed on screen, so sending this BZ over to core team.

Comment 4 Nick Carboni 2016-11-01 19:57:29 UTC

We removed support for Satellite 5 when registering the product (think that's what you're referring to by RHN).

I'm not sure that requires this feature to be removed as it is really just a yum repo mirror, maybe a rename would be enough.

Although if the feature does indeed not work, it seems like the best solution would be to remove it as this is a feature provided (likely in a much better way) by other products which are much more suited to serving up rpm updates.

Thoughts?

Comment 5 Brandon Dunne 2016-11-01 21:39:17 UTC

This feature was added several years ago to bridge the gap for customers that didn't have Satellite and needed to proxy updates because of Internet restrictions in their lab.  It is not specific to RHN, it should work with CDN as well, but I still vote for removing it in favor of better products (Satellite / Pulp)

Comment 7 Gregg Tanzillo 2016-11-04 21:32:29 UTC

Harpreet, I see that there are references to RHN in the setting UI code. Like here - https://github.com/ManageIQ/manageiq/blob/c3c2474f57a59fc9f41dce5f0d91cce3147c245c/app/controllers/ops_controller/settings/rhn.rb

Should that code also be removed?

Comment 8 Nick Carboni 2016-11-09 20:34:50 UTC

This feature actually still works with the other subscription types.

There is a separate issue with subscribing to the yum repo created by the mirror appliance, but that can be fixed as a separate bug which, I think, shouldn't be a blocker.

I'm going to go ahead and close this as NOTABUG.