Project

Profile

Help

Issue #1787

<display_order>None</display_order> in comps.xml

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

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Master
Platform Release:
2.9.0
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Sprint 2
Quarter:

Description

If a package_group unit is uploaded with no display_order,
yum distributor's generated XML will contain a group element
with

<display_order>None</display_order>

This is dangerous because yum will crash when parsing such XML.

It would be safer to omit the element, or apply a default, or
force display_order to be set on import.

Associated revisions

Revision 7b23738c View on GitHub
Added by ipanova@redhat.com over 4 years ago

Assign a default value to display_order.

closes #1787 https://pulp.plan.io/issues/1787

History

#1 Updated by mhrivnak over 4 years ago

  • Parent task set to #1683
  • Triaged changed from No to Yes

#2 Updated by mhrivnak over 4 years ago

We should compare with examples from upstream sources of this data, and make sure the data we publish matches. There is not a known authoritative spec for this file.

This is a great candidate for a smash test.

#3 Updated by mhrivnak over 4 years ago

  • Sprint/Milestone set to 20

#4 Updated by ipanova@redhat.com over 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ipanova@redhat.com

#5 Updated by ipanova@redhat.com over 4 years ago

jluza informed me that the default value for display_order is 1024, so if there was not specified any value, 1024 will be set by default

#6 Updated by ipanova@redhat.com over 4 years ago

  • Status changed from ASSIGNED to POST
  • Platform Release set to 2.9.0

#7 Updated by ipanova@redhat.com over 4 years ago

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

#8 Updated by Ichimonji10 over 4 years ago

The test for this issue is still failing in the nightly Pulp builds. See: http://pulp-smash.readthedocs.io/en/latest/api/pulp_smash.tests.rpm.api_v2.test_comps_groups.html#pulp_smash.tests.rpm.api_v2.test_comps_groups.CompsGroupsTestCase

Please re-open. (Doing so will fix a test failure in our automated test suite.)

Never mind. See: https://github.com/PulpQE/pulp-smash/pull/250

#9 Updated by pthomas@redhat.com over 4 years ago

  • Status changed from MODIFIED to 6

Verified as per comment above

#10 Updated by semyers over 4 years ago

  • Status changed from 6 to CLOSED - CURRENTRELEASE

#12 Updated by bmbouter over 2 years ago

  • Sprint set to Sprint 2

#13 Updated by bmbouter over 2 years ago

  • Sprint/Milestone deleted (20)

#14 Updated by bmbouter over 1 year ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF