Bug 790064

Summary:	Better error message when content url for red hat provider is not accessible
Product:	Red Hat Satellite	Reporter:	Jeff Weiss <jweiss>
Component:	WebUI	Assignee:	Marek Hulan <mhulan>
Status:	CLOSED WORKSFORME	QA Contact:	Katello QA List <katello-qa-list>
Severity:	low	Docs Contact:
Priority:	unspecified
Version:	6.0.0	CC:	dajohnso, mhulan, mmccune, tomckay
Target Milestone:	Unspecified	Keywords:	Triaged
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-01-22 16:35:49 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Jeff Weiss 2012-02-13 15:20:22 UTC

Description of problem:


Version-Release number of selected component (if applicable):
katello-0.1.236-1.git.1.c7be00c.el6.noarch

How reproducible:


Steps to Reproduce:
1. Download the zoo manifest here: http://inecas.fedorapeople.org/fakerepos/cds/
2. Import the manifest without first updating the CDN url.
3.
  
Actual results:
500 error

Expected results:
Should say something about the content url being inaccessible, which is the root cause.

Additional info:

Comment 2 Marek Hulan 2013-01-10 12:01:44 UTC

I cannot reproduce this. If I don't touch CDN and leave default https://cdn.redhat.com then import finishes with success. I also tried to use invalid URL like "http://" (avoiding validation) and I received error notification as expected.

Error message: "Subscription manifest upload for provider 'Red Hat' failed.
Reason: bad URI(absolute but no path): https://
bad URI(absolute but no path): https:// (URI::InvalidURIError)"

Could you please retest with current version? In case you still think there's a bug please provide more detailed instructions to reproduce.

Comment 3 Jeff Weiss 2013-01-10 14:17:46 UTC

If the import succeeds with the zoo manifest but the content URL pointing to CDN, then that is a problem, isn't it?  The content defined in the zoo manifest is not present on the cdn, katello should say so.

Comment 4 Mike McCune 2013-01-22 16:33:00 UTC

it fails for me with the zoo manifest:

Subscription manifest upload for provider 'Red Hat' failed.
Reason: CDN loading error: access denied to https://cdn.redhat.com/content/zoo/listing
CDN loading error: access denied to https://cdn.redhat.com/content/zoo/listing (Errors::SecurityViolation)

Comment 5 Mike McCune 2013-01-22 16:35:49 UTC

based on the fact that it correctly generates an error message now we are closing this.

Comment 6 Marek Hulan 2013-01-29 15:11:20 UTC

I changed this behavior in commit https://github.com/Katello/katello/commit/f7938ea06c40d50e16f3923b90d927a7b200ec75 so now it won't raise error, just inform user via green notification.

More info in pull request https://github.com/Katello/katello/pull/1456 and wiki https://fedorahosted.org/katello/wiki/ManifestImportErrorHandlingDesign

Comment 7 Marek Hulan 2013-01-30 14:12:44 UTC

I just realized that this merge made Red Hat repositories page not working in headpin mode. Fix is in new PR https://github.com/Katello/katello/pull/1479 that is awaiting for review and merge.

Comment 8 Marek Hulan 2013-01-31 13:09:51 UTC

To follow up comment #7 - merged in
https://github.com/Katello/katello/commit/3c9a4e44a2c92e38f10f0dd7e99bbef035a1987a