Bug 712523 - registering cds return incorrect error message w/ no auth cred
Summary: registering cds return incorrect error message w/ no auth cred
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Pulp
Classification: Retired
Component: nodes
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
low
unspecified
Target Milestone: ---
: ---
Assignee: Jay Dobies
QA Contact: dgao
URL:
Whiteboard:
Depends On:
Blocks: 673021
TreeView+ depends on / blocked
 
Reported: 2011-06-10 21:02 UTC by dgao
Modified: 2012-02-24 20:17 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2012-02-24 20:16:01 UTC
Embargoed:


Attachments (Terms of Use)

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.


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