Bug 688631

Summary:	Error messages are not consistent between different cli options for CDS
Product:	[Retired] Pulp	Reporter:	Preethi Thomas <pthomas>
Component:	nodes	Assignee:	Jay Dobies <jason.dobies>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Preethi Thomas <pthomas>
Severity:	low	Docs Contact:
Priority:	low
Version:	1.0.0	CC:	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:		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 Preethi Thomas 2011-03-17 14:58:01 UTC

Description of problem:
[root@preethi ~]# rpm -q pulp
pulp-0.0.151-1.fc14.noarch


[root@preethi ~]# pulp-admin cds  status --hostname=test
error: operation failed: Could not find CDS with hostname [test]
[root@preethi ~]# pulp-admin cds  register --hostname=test
Error attempting to register CDS [test]
Check that the CDS packages have been installed on the CDS and have been started

[root@preethi ~]# 
[root@preethi ~]# 
[root@preethi ~]# pulp-admin cds  unregister --hostname=test
error: operation failed: PulpException: u'Could not find CDS with hostname [test]'

Comment 1 Jay Dobies 2011-10-25 19:12:51 UTC

commit f8082e539b01cddcabd8a188a24d8e8a399a1a31
Author: Jay Dobies <jason.dobies>
Date:   Tue Oct 25 15:12:33 2011 -0400

    688631 - Change handling of CDS errors to raise up to the normal CLI
    handling, only reminding the user about the CDS packages and service in
    the process

src/pulp/client/admin/plugins/cds.py


I'm not a fan of the way error messages are generated on the server and displayed directly to the user, but for consistency with the rest of the CLI I'm going to leave that in place. I've changed register to remind the user to install the CDS packages in the case a register fails, but otherwise the standard CLI format is used.

-> pulp-admin cds register --hostname hulk.marvel.u
error: operation failed: A CDS with the hostname [hulk.marvel.u] already exists

-> pulp-admin cds register --hostname hulk.marvel
Check that the CDS packages have been installed on the CDS and have been started
error: operation failed: PulpException: Timeout occurred attempting to initialize CDS [hulk.marvel]

-> pulp-admin cds unregister --hostname hulk.marvel
error: operation failed: PulpException: Could not find CDS with hostname [hulk.marvel]

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

build 0.242-2

Comment 3 Preethi Thomas 2011-11-11 20:53:42 UTC

verified

[root@preethi pulp]# pulp-admin cds register --hostname=f16-cds.usersys.redhat.com
error:  error: operation failed: A CDS with the hostname [f16-cds.usersys.redhat.com] already exists
[root@preethi pulp]# pulp-admin cds  status --hostname=test
error:  error: operation failed: Could not find CDS with hostname [test]
[root@preethi pulp]# pulp-admin cds unregister --hostname=f16-cds.usersys.redhat.com
Successfully unregistered CDS [f16-cds.usersys.redhat.com]

[root@preethi pulp]# pulp-admin cds unregister --hostname=test
error:  error: operation failed: PulpException: Could not find CDS with hostname [test]
[root@preethi pulp]#

Comment 4 Preethi Thomas 2012-02-24 20:18:50 UTC

Pulp v1.0 is released
Closed Current Release.

Comment 5 Preethi Thomas 2012-02-24 20:19:43 UTC

Pulp v1.0 is released.