Bug 662586

Summary:	removing non-existing package ends with success
Product:	[Retired] Pulp	Reporter:	Daniel Mach <dmach>
Component:	z_other	Assignee:	Pradeep Kilambi <pkilambi>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Preethi Thomas <pthomas>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	unspecified
Target Milestone:	---
Target Release:	---
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:00:35 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, 647488

Description Daniel Mach 2010-12-13 09:32:03 UTC

$ pulp-admin repo delete_package --id=rhel-i386-client-5 --id=rhel-i386-server-5 -p foo
Package [foo] is not part of the source repository [rhel-i386-server-5]
Successfully removed package(s) ['foo'] to repo [rhel-i386-server-5].

The first message doesn't make sense.
What's the source repo? Does it contain sources or does it mean it's the repo we're deleting from?
I'd also avoid 'is not part of' part of the message. Sounds like is not on some kind of whitelist or so. 'does not exist in' would be more clear to me.

The second message is obviously wrong.
There was no package -> nothing was deleted.

Comment 1 Pradeep Kilambi 2010-12-13 19:52:16 UTC

fixed

$ sudo pulp-admin repo delete_package --id=pk01 -p foo -p patb-0.1-2.x86_64.rpm -p emoticons-0.1-2.x86_64.rpm
Package [foo] does not exist in repository [pk01]
Successfully removed package patb-0.1-2.x86_64.rpm from repo [pk01].
Successfully removed package emoticons-0.1-2.x86_64.rpm from repo [pk01].

Comment 2 Jay Dobies 2010-12-20 19:46:33 UTC

Fixed in build 0.117.

Comment 3 Preethi Thomas 2010-12-23 15:18:53 UTC

[root@pulp ~]# rpm -q pulp
pulp-0.0.119-1.fc14.noarch
[root@pulp ~]# 


[root@pulp ~]# pulp-admin repo delete_package --id=package_group_test_repo -p foo -p patb
Package [foo] does not exist in repository [package_group_test_repo]
Package [patb] does not exist in repository [package_group_test_repo]

Comment 4 Preethi Thomas 2011-08-16 14:00:35 UTC

Closing with Community Release 15

pulp-0.0.223-4.