Bug 669760 - pulp-admin package install doesn't validate the consumer ID
Summary: pulp-admin package install doesn't validate the consumer ID
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Pulp
Classification: Retired
Component: z_other
Version: unspecified
Hardware: Unspecified
OS: Unspecified
low
medium
Target Milestone: ---
: Sprint 20
Assignee: Sayli Karmarkar
QA Contact: Preethi Thomas
URL:
Whiteboard:
Depends On:
Blocks: verified-to-close
TreeView+ depends on / blocked
 
Reported: 2011-01-14 16:33 UTC by Jay Dobies
Modified: 2015-03-23 01:10 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2011-02-17 16:52:13 UTC
Embargoed:


Attachments (Terms of Use)

Description Jay Dobies 2011-01-14 16:33:32 UTC
When specifying an invalid consumer ID, the task seems to spin for a while. I assume it's waiting to time out since it can't find the consumer to contact.

We should do a sanity check first to ensure the consumer actually exists before starting off a task that can't be cancelled.


$ pulp-admin package install -n emoticons --consumerid foo
Created task id: dc51ee4a-1ffb-11e0-97e0-5254003d10ee
Task is scheduled for: 2011-01-14 11:32
....................................................

Comment 1 Jay Dobies 2011-01-14 16:57:50 UTC
Followup: 20 minutes later the CLI is still rendering periods and the process hasn't ended yet.

Comment 2 Sayli Karmarkar 2011-01-28 20:44:14 UTC
Added validate_consumer and validate_consumergroup functions to ConsumerActions and ConsumerGroupActions classes.

$ sudo pulp-admin consumer list
+------------------------------------------+
            Consumer Information
+------------------------------------------+

Id                 	test-consumer            
Description        	None                     
Subscribed Repos   	[]                       
Additional info    	                 

$ sudo pulp-admin package install -n origin --consumerid=test-consumer11
error: operation failed: Consumer [test-consumer11] does not exist

Comment 3 Jay Dobies 2011-02-02 20:07:18 UTC
Fixed in 0.134.

Comment 4 Preethi Thomas 2011-02-08 18:22:53 UTC
verified
[root@preethi ~]# pulp-admin package install -n zsh --consumerid=test-consumer11 
error: operation failed: Consumer [test-consumer11] does not exist
[root@preethi ~]# rpm -q pulp
pulp-0.0.135-1.fc14.noarch

Comment 5 Preethi Thomas 2011-02-17 16:52:13 UTC
Closing with community release 

pulp-0.0.139-1.fc14.noarch


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