Red Hat Bugzilla – Bug 855461
Manifest upload history reports success when part of the import process fails
Last modified: 2015-08-17 13:52:05 EDT
Created attachment 610846 [details]
Screenshot on failed import.
Description of problem:
When a user uploads a manifest, there are a couple of high-level phases that occur: 1) the manifest is uploaded to candlepin and then 2) katello will create repositories (using the info from the imported manifest and the cdn).
The issue that this bugzilla raises is that if the first step succeeds and the second fails (e.g. timeout with the CDN), the import will still be reported as successful on the Subscriptions page Upload History. The reason being, the Upload History is based upon the information from candlepin and not the overall completion of the upload from katello's point of view.
This issue was observed while working on bug 809259 .
Version-Release number of selected component (if applicable):
Steps to Reproduce:
follow the steps described in bug 809259
Upload history report success. E.g.
ACME_Corporation file imported successfully.
Upload history reports failure. E.g.
ACME_Corporation file import failed. Reason: timeout interfacing with CDN.
Attached is a screen shot that shows that a failure has occured, but the history shows successful. Ignore the "killed thread" in the reason as I was working in the debugger at the time which generated that particular reason.
*** Bug 858665 has been marked as a duplicate of this bug. ***
Believe this is a much higher priority issue. Giving a successful indication to the user for a failed import is going to lead to confusion.
Note that there is an error notice generated for the pulp failure.
*** Bug 859218 has been marked as a duplicate of this bug. ***
*** Bug 867596 has been marked as a duplicate of this bug. ***
I believe that this one is related to current changes of manifest import, namely bug #790064 and #896265. I suggest review it once PR with changes is merged so I retake it and I will raise discussion when PR is merged.
I changed this behavior in katello master - 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
So now import should finish with success state even when CDN has some problems. Should we still change Upload History or is it OK now? If it's OK, it means that this PR solved this issue already and should be moved to POST.
There is still an issue in that the "Manifest History" shown is only displayed from candlepin. So if, for example, pulp is completely down and an exception is thrown, the async job will rollback the candlepin results, clearing the subscriptions imported there. The "Manifest History" will just show the candlepin message of "import successful" while a red error dialog will popup.
In other words, the "Manifest History" incorrectly states that the import was successful when the overall task failed.
I see, thank you for clarification. So if pulp is unreacheable we'll display error in Manifest History.
This bug was closed because of a lack of activity. If you feel this bug should be reconsidered for attention please feel free to re-open the bug with a comment stating why it should be reconsidered.