Project

Profile

Help

Issue #3162

Pulp failed to parse updateinfo.xml file it generated

Added by daviddavis about 2 years ago. Updated 11 months ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Severity:
2. Medium
Version:
Platform Release:
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:

Description

We had an issue where a reference in an updateinfo.xml file didn't have an href attribute:

  <reference type="other" id="ref_0" title="other_reference_0" />

Pulp simply threw an "Importer indicated a failed response" error and wasn't able to sync the repository. The backtrace wasn't particularly helpful either ( File "/usr/lib/python2.7/site-packages/pulp_rpm/plugins/importers/yum/repomd/updateinfo.py", line 88, in _parse_reference
'href': element.attrib['href'],
KeyError: 'href'
).

First, per https://bugzilla.redhat.com/show_bug.cgi?id=1503916, it seems that Pulp is generating these updateinfo.xml files that it can't consume. We should consider either rejecting errata that are invalid and missing attributes.

Second, we probably need better logging. Particularly, it would be helpful to know where in the parsed file our code is choking. It took us a couple hours to figure out the particular erratum in this case that caused the problem.

Related BZs:

History

#1 Updated by daviddavis about 2 years ago

  • Description updated (diff)

#2 Updated by daviddavis about 2 years ago

  • Description updated (diff)

#3 Updated by dalley about 2 years ago

  • Triaged changed from No to Yes

#4 Updated by daviddavis about 2 years ago

  • Subject changed from Pulp failed to parse updateinfo.xml file. Gave unhelpful log message. to Pulp failed to parse updateinfo.xml file it generated
  • Description updated (diff)

#5 Updated by bmbouter 11 months ago

  • Status changed from NEW to CLOSED - WONTFIX

Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.

#6 Updated by bmbouter 11 months ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF