Project

Profile

Help

Issue #598

Pulp cannot publish rpms with non-ascii characters. Falling due gzip

Added by jluza over 6 years ago. Updated over 2 years ago.

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

Description

patch for plugins/pulp_rpm/plugins/distributors/yum/metadata/primary.py

reproducer:
------------

vim dummy.spec
================================================
Summary: Dummy Package to provide tomcat5
Name: dummy-tomcat
Version: 1.0
Release: 1
Group: System Environment/Base
License: Beerware
BuildArch: noarch
Provides: tomcat5, tomcat5-admin-webapps

%description
řšěřčšřšěřě (or some other non-ascii chars)

%files
===============================================

rpmbuild -bb dummy.spec

pulp-admin rpm repo create --repo-id unicode_problem
pulp-admin rpm repo uploads rpm --file dummy* --repo-id unicode_problem
pulp-admin -u admin -p admin rpm repo publish run --repo-id unicode_problem

OUTPUT:
--------------------------------------------------------------------
Publishing Repository [unicode_problem]
--------------------------------------------------------------------

This command may be exited via ctrl+c without affecting the request.

Initializing repo metadata
[-]
... completed

Publishing Distribution files
[-]
... completed

Publishing RPMs
[-]
... failed
Incorrect length of data produced

Task Failed

Incorrect length of data produced

Affected files:
plugins/pulp_rpm/plugins/distributors/yum/metadata/primary.py
plugins/pulp_rpm/plugins/distributors/yum/metadata/other.py
plugins/pulp_rpm/plugins/distributors/yum/metadata/filelists.py

patch for primary.py in attachement

+ This bug was cloned from Bugzilla Bug #1158945 +

History

#1 Updated by jortel@redhat.com over 6 years ago

Reproduce using UTF-8 description.

+ This comment was cloned from Bugzilla #1158945 comment 1 +

#3 Updated by cduryee over 6 years ago

fixed in pulp 2.6.0-0.2.beta

+ This comment was cloned from Bugzilla #1158945 comment 3 +

#4 Updated by jiri.tyr@gmail.com over 6 years ago

There is a problem with the proposed patch:
https://github.com/pulp/pulp_rpm/pull/605#issuecomment-68809841

+ This comment was cloned from Bugzilla #1158945 comment 4 +

#5 Updated by igulina@redhat.com over 6 years ago

rpm -qa pulp-server

pulp-server-2.6.0-0.5.beta.el6.noarch

rpm -qpil /home/ec2-user/rpmbuild/RPMS/x86_64/dummy-client-1.0-el6.x86_64.rpm

Name : dummy-client Relocations: /usr
Version : 1.0 Vendor: (none)
Release : el6 Build Date: Thu 22 Jan 2015 09:57:34 AM EST
Install Date: (not installed) Build Host:
Group : Development/Tools Source RPM: dummy-client-1.0-el6.src.rpm
Size : 0 License: GPL
Signature : (none)
Summary : dummy
Description :
sample řšěřčšřšěřě (or some other non-ascii chars)
/usr/local/dummy-client
/usr/local/dummy-client/test.txt

pulp-admin rpm repo create --repo-id unicode_problem

Successfully created repository [unicode_problem]

pulp-admin rpm repo uploads rpm --file /home/ec2-user/rpmbuild/RPMS/x86_64/dummy-client-1.0-el6.x86_64.rpm --repo-id unicode_problem

--------------------------------------------------------------------
Unit Upload
--------------------------------------------------------------------

Extracting necessary metadata for each request...
[==================================================] 100%
Analyzing: dummy-client-1.0-el6.x86_64.rpm
... completed

Creating upload requests on the server...
[==================================================] 100%
Initializing: dummy-client-1.0-el6.x86_64.rpm
... completed

Starting upload of selected units. If this process is stopped through ctrl+c,
the uploads will be paused and may be resumed later using the resume command or
cancelled entirely using the cancel command.

Uploading: dummy-client-1.0-el6.x86_64.rpm
[==================================================] 100%
2106/2106 bytes
... completed

Importing into the repository...
This command may be exited via ctrl+c without affecting the request.

[\]
Running...

Task Succeeded

Deleting the upload request...
... completed

pulp-admin rpm repo list

--------------------------------------------------------------------
RPM Repositories
--------------------------------------------------------------------

Id: unicode_problem
Display Name: unicode_problem
Description: None
Content Unit Counts:
Rpm: 1

pulp-admin -u admin -p admin rpm repo publish run --repo-id unicode_problem

--------------------------------------------------------------------
Publishing Repository [unicode_problem]
--------------------------------------------------------------------

This command may be exited via ctrl+c without affecting the request.

Initializing repo metadata
[-]
... completed

Publishing Distribution files
[-]
... completed

Publishing RPMs
[-]
... completed

Publishing Delta RPMs
... skipped

Publishing Errata
[-]
... completed

Publishing Comps file
[-]
... completed

Publishing Metadata.
[-]
... completed

Closing repo metadata
[-]
... completed

Generating sqlite files
... skipped

Publishing files to web
[-]
... completed

Writing Listings File
[-]
... completed

Task Succeeded

ls -l /var/lib/pulp/published/yum/master/yum_distributor/unicode_problem/1421938997.07/

total 8
lrwxrwxrwx. 1 apache apache 152 Jan 22 10:03 dummy-client-1.0-el6.x86_64.rpm > /var/lib/pulp/content/rpm/dummy-client/1.0/el6/x86_64/b322e2027529911bb7b5d524ba808b70a8e01e73b1a5f2381ce45130b35a142e/dummy-client-1.0-el6.x86_64.rpm
drwxr-x--
. 2 apache apache 4096 Jan 22 10:03 repodata

+ This comment was cloned from Bugzilla #1158945 comment 5 +

#6 Updated by bmbouter over 6 years ago

  • Severity changed from Medium to 2. Medium

#7 Updated by rbarlow over 6 years ago

  • Status changed from 6 to CLOSED - CURRENTRELEASE

#9 Updated by bmbouter over 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF