Project

Profile

Help

Issue #1828

pulp doesn't sync reference title correctly from errata

Added by jluza over 4 years ago. Updated over 1 year ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
High
Assignee:
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
2.8.0
Platform Release:
2.8.3
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Easy Fix, Pulp 2
Sprint:
Quarter:

Description

if an errata's reference is in format:
<reference href="href" type="self" title="title"/>
pulp doesn't sync title at all.

Associated revisions

Revision 7bcfb290 View on GitHub
Added by jluza over 4 years ago

  • fixed sync for reference title

https://pulp.plan.io/issues/1828 fixes #1828

History

#2 Updated by jluza over 4 years ago

  • Status changed from NEW to POST

#3 Updated by bmbouter over 4 years ago

  • Assignee set to jluza

jluza since you put in a PR I put you as the assignee. That field is used to track who did the work issue by issue.

#5 Updated by mhrivnak over 4 years ago

  • Subject changed from pulp doesn't sync reference title correctly to pulp doesn't sync reference title correctly from errata
  • Description updated (diff)
  • Priority changed from Normal to High
  • Version set to 2.8.0
  • Platform Release set to 2.8.3

#6 Updated by mhrivnak over 4 years ago

  • Triaged changed from No to Yes

#7 Updated by jluza over 4 years ago

  • Triaged changed from Yes to No

Reproducer:

1. create rpm repo
2. sync from already existing repo that has at least one advisory with at least one reference with existing, non-empty title attribute
3. publish repo
4. published advisory in updateinfo will have title=null

#8 Updated by jluza over 4 years ago

  • Triaged changed from No to Yes

#9 Updated by bmbouter over 4 years ago

jluza, I looked for a repo that meets the criteria of step (2) in comment 7, but I could not find one. Can you identify the repo and errata inside of that repo more specifically which meets the critiera? I really want to review this, but I'm blocked finding the thing I should test it with.

#11 Updated by bmbouter over 4 years ago

I reproduced the issue by syncing RHEL7.0 server and looking at the metadata entry for errata RHBA-2015:2255 it shows the snippet:

<reference href="https://rhn.redhat.com/errata/RHBA-2015-2255.html" id="" title="" type="self" />

After applying the fix from the linked PR that same line publishes with:

<reference href="https://rhn.redhat.com/errata/RHBA-2015-2255.html" id="" title="RHBA-2015:2255" type="self" />

#12 Updated by jluza over 4 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#13 Updated by semyers over 4 years ago

  • Status changed from MODIFIED to 5

#14 Updated by semyers over 4 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

#15 Updated by bmbouter over 1 year ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF