Bug 681804

Summary:	pulp fails when a filename contains '++'
Product:	[Retired] Pulp	Reporter:	Daniel Mach <dmach>
Component:	z_other	Assignee:	Pradeep Kilambi <pkilambi>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Preethi Thomas <pthomas>
Severity:	urgent	Docs Contact:
Priority:	urgent
Version:	unspecified	CC:	pkilambi, skarmark
Target Milestone:	---
Target Release:	Sprint 21
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2011-08-16 14:01:38 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	563609, 641987

Description Daniel Mach 2011-03-03 10:13:13 UTC

$ pulp-admin content upload compat-libstdc++-296-2.96-138.i386.rpm

it fails on '++' in the file name.
Pulp expects a regular expression, therefore ++ is invalid.

Comment 1 Pradeep Kilambi 2011-03-03 14:34:18 UTC

commit 949fee222daa61fce2c09ab78b3e80a528431502
Author: Pradeep Kilambi <pkilambi>
Date:   Thu Mar 3 09:33:05 2011 -0500


$ sudo pulp-admin content upload compat-libstdc++-296-2.96-138.i386.rpm  -v
* Starting Package Upload

* Performing Package Uploads to Pulp server
Successfully uploaded [compat-libstdc++-296-2.96-138.i386.rpm] to server

* Content Upload complete.

Comment 2 Daniel Mach 2011-03-03 15:18:50 UTC

Adding 'regex=False' works, also my upload script doesn't fail any more.

Comment 3 Jeff Ortel 2011-03-07 20:43:19 UTC

Build: 0.145

Comment 4 Preethi Thomas 2011-03-08 18:37:23 UTC

verified

[root@nec-em8 upload]# pulp-admin -u admin -p admin content upload compat-libstdc++-296-2.96-138.i386.rpm -v
* Starting Package Upload

* Performing Package Uploads to Pulp server
Successfully uploaded [compat-libstdc++-296-2.96-138.i386.rpm] to server

* Content Upload complete.
[root@nec-em8 upload]# pulp-admin -u admin -p admin content list --orphaned
grinder-0.0.83-1.fc14.noarch.rpm.1,6e83d7ab02d6d1cb4a5460465b09b8687a454639fc43f98bd9d539e62d1adc74
compat-libstdc++-296-2.96-138.i386.rpm,59317955587c4d89ef3191610b9afa346c2063aa2f1608a571d3ea52c908da84

[root@nec-em8 upload]# 
[root@nec-em8 upload]# 
[root@nec-em8 upload]# pulp-admin -u admin -p admin content upload compat-libstdc++-296-2.96-138.i386.rpm -v -r upload
* Starting Package Upload

* Performing Package Uploads to Pulp server
Package [compat-libstdc++-296-2.96-138.i386.rpm] already exists on the server with checksum [59317955587c4d89ef3191610b9afa346c2063aa2f1608a571d3ea52c908da84]

* Performing Repo Associations 
Package association Complete for Repo [upload]: 
 Packages: 
compat-libstdc++-296-2.96-138.i386.rpm 
 
 Files: 
None

* Content Upload complete.

[root@nec-em8 upload]# pulp-admin -u admin -p admin content list --repoid=upload 
grinder-0.0.83-1.fc14.noarch.rpm,6e83d7ab02d6d1cb4a5460465b09b8687a454639fc43f98bd9d539e62d1adc74
compat-libstdc++-296-2.96-138.i386.rpm,59317955587c4d89ef3191610b9afa346c2063aa2f1608a571d3ea52c908da84

Comment 5 Preethi Thomas 2011-08-16 14:01:38 UTC

Closing with Community Release 15

pulp-0.0.223-4.