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:
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.
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.
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)
based on the fact that it correctly generates an error message now we are closing this.
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
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.
To follow up comment #7 - merged in https://github.com/Katello/katello/commit/3c9a4e44a2c92e38f10f0dd7e99bbef035a1987a