Bug 662586 - removing non-existing package ends with success
Summary: removing non-existing package ends with success
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Pulp
Classification: Retired
Component: z_other
Version: unspecified
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: ---
Assignee: Pradeep Kilambi
QA Contact: Preethi Thomas
URL:
Whiteboard:
Depends On:
Blocks: 563609 verified-to-close
TreeView+ depends on / blocked
 
Reported: 2010-12-13 09:32 UTC by Daniel Mach
Modified: 2011-08-16 14:00 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-08-16 14:00:35 UTC
Embargoed:


Attachments (Terms of Use)

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.


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