Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1332922 - repomd.xml parsing fails when an empty revision tag is present
repomd.xml parsing fails when an empty revision tag is present
Status: CLOSED ERRATA
Product: Red Hat Satellite 6
Classification: Red Hat
Component: Pulp (Show other bugs)
6.1.8
All Linux
medium Severity high (vote)
: 6.1.9
: Unused
Assigned To: satellite6-bugs
Katello QA List
: Triaged
Depends On: 1286803
Blocks:
  Show dependency treegraph
 
Reported: 2016-05-04 07:21 EDT by Mike McCune
Modified: 2017-07-26 15:38 EDT (History)
10 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 1286803
Environment:
Last Closed: 2016-05-17 01:33:22 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)


External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2016:1084 normal SHIPPED_LIVE Satellite 6.1.9 bug fix update 2016-05-17 05:32:24 EDT

  None (edit)
Description Mike McCune 2016-05-04 07:21:59 EDT
+++ This bug was initially created as a clone of Bug #1286803 +++

The NSA SIMP 5.1.X repodata contains a <revision /> tag which busts sync
with the following splat:

Downloading metadata from https://dl.bintray.com/simp/5.1.X/.
 Starting new HTTPS connection (1): dl.bintray.com
 Parsing metadata.
  sync failed
  Traceback (most recent call last):
    File ".../sync.py", line 142, in run
      metadata_files = self.get_metadata()
    File ".../sync.py", line 226, in get_metadata
      metadata_files.parse_repomd()
    File ".../metadata.py", line 165, in parse_repomd
      self.revision = int(element.text)
  TypeError: int() argument must be a string or a number, not 'NoneType'

--- Additional comment from RHEL Product and Program Management on 2015-11-30 14:14:21 EST ---

Since this issue was entered in Red Hat Bugzilla, the pm_ack has been
set to + automatically for the next planned release

--- Additional comment from pulp-infra@redhat.com on 2015-11-30 14:29:34 EST ---

The Pulp upstream bug status is at MODIFIED. Updating the external tracker on this bug.

--- Additional comment from pulp-infra@redhat.com on 2015-11-30 14:29:36 EST ---

The Pulp upstream bug priority is at Normal. Updating the external tracker on this bug.

--- Additional comment from Bryan Kearney on 2016-01-04 10:21:52 EST ---

moving to POST. PLease verify once 2.8.0 of pulp is incorporated into the build.

--- Additional comment from pulp-infra@redhat.com on 2016-02-23 16:00:39 EST ---

The Pulp upstream bug status is at ON_QA. Updating the external tracker on this bug.

--- Additional comment from Bryan Kearney on 2016-02-26 10:37:01 EST ---

This bug was included in SNAP1 which was delivered on 25 February, 2016.

--- Additional comment from pulp-infra@redhat.com on 2016-03-23 15:01:10 EDT ---

The Pulp upstream bug status is at CLOSED - CURRENTRELEASE. Updating the external tracker on this bug.

--- Additional comment from errata-xmlrpc on 2016-03-31 03:41:59 EDT ---

This bug has been dropped from advisory RHEA-2016:23110 by Martin Prpic (mprpic@redhat.com)

--- Additional comment from Roman Plevka on 2016-04-07 11:03:40 EDT ---

VERIFIED

on sat6.2.0 snap6.2


- modified repomd.xml to contain <revision/>


# hammer repository create --product-id=13 --content-type=yum --name=my_repo_1 --url="https://rplevka.fedorapeople.org/fakerepo02/"
[Foreman] Password for admin: 
Repository created

# hammer repository synchronize --organization-id=1 --product-id=13 --name my_repo_1
[Foreman] Password for admin: 
[...........................................................................................................................] [100%]
No new packages.

# hammer repository info --id 3
[Foreman] Password for admin: 
ID:                 3
Name:               my_repo_1
Label:              my_repo_1
Organization:       Default Organization
Red Hat Repository: no
Content Type:       yum
Checksum Type:      sha256
URL:                https://rplevka.fedorapeople.org/fakerepo02/
Publish Via HTTP:   yes
Published At:       http://sat6server/pulp/repos/Default_Organization/Library/custom/my_product_1/my_repo_1
Relative Path:      Default_Organization/Library/custom/my_product_1/my_repo_1
Download Policy:    immediate
Product:            
    ID:   13
    Name: my_product_1
GPG Key:            

Sync:               
    Status:         Success
    Last Sync Date: 2 minutes
Created:            2016/04/07 14:57:42
Updated:            2016/04/07 14:59:35
Content Counts:     
    Packages:       3
    Package Groups: 0

--- Additional comment from Eric Beaudoin on 2016-05-02 10:21:25 EDT ---

Since 6.2 is release June, my customer would like to know if is there a workaround in the meantime?

Thanks
Comment 4 Kedar Bidarkar 2016-05-11 07:50:55 EDT
VERIFIED with sat6.1.9 compose1

the repomd.xml file does have an empty revision tag

[root@xyz ~]# hammer product create --name kbidarka_prd --organization-id 1
[Foreman] Username: admin
[Foreman] Password for admin: 
Product created

[root@xyz ~]# hammer repository create --product="kbidarka_prd" --content-type=yum --name=my_repo_1 --url="https://rplevka.fedorapeople.org/fakerepo02/" --organization-id 1
[Foreman] Username: admin
[Foreman] Password for admin: 
Repository created

[root@xyz ~]#  hammer repository synchronize --organization-id=1 --product="kbidarka_prd" --name my_repo_1
[Foreman] Username: admin
[Foreman] Password for admin: 
[......................................................................] [100%]
New packages: 3 (37.1 KB).

[root@xyz ~]# hammer -u admin -p changeme repository info --organization-id 1 --product="kbidarka_prd"
ID:                 949
Name:               my_repo_1
Label:              my_repo_1
Organization:       Default Organization
Red Hat Repository: no
Content Type:       yum
Checksum Type:      sha256
URL:                https://rplevka.fedorapeople.org/fakerepo02/
Publish Via HTTP:   yes
Published At:       http://sat6server/pulp/repos/Default_Organization/Library/custom/kbidarka_prd/my_repo_1
Product:            
    ID:   843
    Name: kbidarka_prd
GPG Key:            

Sync:               
    Status:         
    Last Sync Date:
Created:            2016/05/11 11:34:27
Updated:            2016/05/11 11:35:17
Content Counts:     
    Packages: 3
    Errata:   0
Comment 5 Kedar Bidarkar 2016-05-11 07:54:41 EDT
The repomd.xml file.

<repomd><revision/><data type="filelists"><checksum type="sha256">
Comment 7 errata-xmlrpc 2016-05-17 01:33:22 EDT
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2016:1084

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