Bug 1301736 - Repo syncs resulting in PLP0000: Importer indicated a failed response do not index successfully obtained packages
Summary: Repo syncs resulting in PLP0000: Importer indicated a failed response do not ...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Repositories
Version: 6.1.6
Hardware: All
OS: Linux
unspecified
high
Target Milestone: Unspecified
Assignee: satellite6-bugs
QA Contact: Katello QA List
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-01-25 21:11 UTC by Craig Donnelly
Modified: 2019-09-12 09:49 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-09-04 18:01:36 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Craig Donnelly 2016-01-25 21:11:47 UTC
Description of problem:

When syncing a custom repository (product), if there are some packages that fail to download we now receive the error:

PLP0000: Importer indicated a failed response.

The rest of the sync information is still there indicating successfully downloaded packages (if any) and how many failed.

For my example, I synced EPEL 5 and received:

New packages: 4941 (3.68 GB).
Failed to download 3296 packages.
PLP0000: Importer indicated a failed response.

When we goto the EPEL product and check this EPEL 5 repo now, it shows 0 packages and 0 errata available.

How reproducible: So far, 100%.


Steps to Reproduce:
1. Add custom product (Like EPEL).
2. Sync EPEL (typically, it will fail to download /all/ packages on the first sync, and you will see the results I have above.
3. Check product page under repo tab and view that it shows 0 packages in repo even though there were some successfully downloaded per the task data.

Actual results: Packages are not indexed after encountering a partial sync completion and do not show up under the product repo.


Expected results: Packages that were successfully synced should show up under the product.


Additional info: I was able to get some of the package count to show up after I ran a re-index following recieving this error during a sync of EPEL 5, although the number that appears under the product is smaller than the number the task reported to download successfully.

Comment 1 Craig Donnelly 2016-01-25 21:15:57 UTC
This bug may relate to these two fixes that were implemented into Satellite 6:

https://github.com/Katello/katello/pull/5680
https://github.com/Katello/katello/pull/5567

Comment 2 Bryan Kearney 2016-07-26 19:06:03 UTC
Moving 6.2 bugs out to sat-backlog.

Comment 3 Michael Hrivnak 2016-12-05 20:03:30 UTC
Brad, I think the component for this should not be Pulp. Perhaps you can find a better one for it?

Comment 4 Michael Hrivnak 2016-12-12 21:46:04 UTC
I'm changing the component since the issue appears to be with how and when katello queries pulp. Let me know if there's anything to be done on the pulp side for this.

Comment 5 Brad Buckingham 2017-01-05 15:26:39 UTC
Justin, with the changes that have gone in to Satellite 6.2, would the described behavior still exist?

Comment 6 Justin Sherrill 2017-01-05 19:06:58 UTC
I believe this behavior still exists.  What works now on (some time before 6.2.6) is that re-syncing will properly re-index the content even if nothing changed on the 2nd sync.

We might have to think about this a bit, lets say I sync a repo and it syncs some of the rpms, but not all.  Does the metadata for that repo get generated? Should it?   

If it doesn't get generated (and thus the client will not see the new rpms), should we index them?

Comment 9 Michael Hrivnak 2017-01-05 19:50:12 UTC
(In reply to Justin Sherrill from comment #6)
> We might have to think about this a bit, lets say I sync a repo and it syncs
> some of the rpms, but not all.  Does the metadata for that repo get
> generated? Should it?   

Yes it gets generated. Download errors are so common for yum repos that we just note the error in the task status, but the overall task succeeds, and a publish task gets queued as normal.

You can know for sure if new metadata gets generated based on whether a publish task gets queued and succeeds.

Comment 10 Bryan Kearney 2018-09-04 18:01:36 UTC
Thank you for your interest in Satellite 6. We have evaluated this request, and we do not expect this to be implemented in the product in the foreseeable future. We are therefore closing this out as WONTFIX. If you have any concerns about this, please feel free to contact Rich Jerrido or Bryan Kearney. Thank you.


Note You need to log in before you can comment on or make changes to this bug.