Bug 712523

Summary:	registering cds return incorrect error message w/ no auth cred
Product:	[Retired] Pulp	Reporter:	dgao
Component:	nodes	Assignee:	Jay Dobies <jason.dobies>
Status:	CLOSED CURRENTRELEASE	QA Contact:	dgao
Severity:	unspecified	Docs Contact:
Priority:	low
Version:	1.0.0	CC:	pthomas, skarmark
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-02-24 20:16:01 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:	673021

Description dgao 2011-06-10 21:02:32 UTC

[root@pulp-nightly ~]# ls ~/.pulp/
upload
[root@pulp-nightly ~]# pulp-admin cds register --hostname pulp-cds-f14.usersys.redhat.com --name "QE F14 CDS"
Error attempting to register CDS [pulp-cds-f14.usersys.redhat.com]
Check that the CDS packages have been installed on the CDS and have been started

[root@pulp-nightly ~]# pulp-admin -u admin -p admin cds register --hostname pulp-cds-f14.usersys.redhat.com --name "QE F14 CDS"
Successfully registered CDS [pulp-cds-f14.usersys.redhat.com]

[root@pulp-nightly ~]# 


Ironically enough, unregistering cds return the expected error message when auth credential is not provided.

Comment 1 Jay Dobies 2011-10-24 20:07:21 UTC

commit ad0d311484d9c2175aa031239e3a9320686fe99b
Author: Jay Dobies <jason.dobies>
Date:   Mon Oct 24 15:55:15 2011 -0400

    712523 - Split missing auth into its own exception (previously it was
    bundled in with server-side errors). Changed the CDS register code to
    only catch server exceptions and let the auth ones bubble up.

src/pulp/client/admin/plugins/cds.py
src/pulp/client/api/server.py
src/pulp/client/pluginlib/command.py

Comment 2 Jeff Ortel 2011-10-27 03:22:21 UTC

build 0.242-2

Comment 3 Preethi Thomas 2011-12-22 15:22:29 UTC

verified
[root@preethi ~]# rpm -q pulp
pulp-0.0.255-1.fc15.noarch

[root@preethi ~]# pulp-admin cds register --hostname f16-cds.usersys.redhat.com --name "QE F16 CDS"
error:  error: operation failed: No valid authorization credentials found, please see: pulp-admin --help
[root@preethi ~]# 
[root@preethi ~]# 
[root@preethi ~]# pulp-admin -u admin -p admin cds register --hostname f16-cds.usersys.redhat.com --name "QE F16 CDS"
Successfully registered CDS [f16-cds.usersys.redhat.com]

[root@preethi ~]# pulp-admin cds unregister --hostname f16-cds.usersys.redhat.com
error:  error: operation failed: No valid authorization credentials found, please see: pulp-admin --help
[root@preethi ~]# pulp-admin -u admin -p admin cds unregister --hostname f16-cds.usersys.redhat.com
Successfully unregistered CDS [f16-cds.usersys.redhat.com]

Comment 4 Preethi Thomas 2012-02-24 20:16:01 UTC

Pulp v1.0 is released
Closed Current Release.

Comment 5 Preethi Thomas 2012-02-24 20:17:29 UTC

Pulp v1.0 is released.