Bug 795599

Summary:	RFE: more concise output on successful push
Product:	[Community] PressGang CCMS	Reporter:	Joshua Wulf <jwulf>
Component:	CSProcessor	Assignee:	Lee Newson <lnewson>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	low	Docs Contact:
Priority:	low
Version:	1.x	CC:	jwulf, lcarlon, misty
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-05-29 02:06:47 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Joshua Wulf 2012-02-21 01:19:13 UTC

Currently it looks like this:

[jwulf@radhe temp]$ csprocessor push -cn Test_Book.spec 
CSProcessor client version: 0.16.3
Loading configuration from /home/jwulf/.config/csprocessor.ini
Web Service at http://csprocessor.cloud.lab.eng.bne.redhat.com:8080/ version: 0.21.2

INFO:  The Content Specification is valid.
WARN:  Note: All descriptions, tags, source urls and writers will be ignored for existing Topics.
INFO:  The Content Specification saved successfully.
Content Specification ID: 6226
Revision: 39655


Really all it needs to be is this:

[jwulf@radhe temp]$ csprocessor push -cn Test_Book.spec 
CSProcessor client 0.16.3
Configuration file: /home/jwulf/.config/csprocessor.ini
Server: http://csprocessor.cloud.lab.eng.bne.redhat.com:8080/ version: 0.21.2

Content Specification saved successfully.
Content Specification ID: 6226
Revision: 39655


Even then, the first part is verbose. Perhaps the default should be:
[jwulf@radhe temp]$ csprocessor push -cn Test_Book.spec 

Content Specification saved successfully.
Content Specification ID: 6226
Revision: 39655


and with -v for verbose:

[jwulf@radhe temp]$ csprocessor push -cnv Test_Book.spec 
CSProcessor client 0.16.3
Configuration file: /home/jwulf/.config/csprocessor.ini
Server: http://csprocessor.cloud.lab.eng.bne.redhat.com:8080/ version: 0.21.2

Content Specification saved successfully.
Content Specification ID: 6226
Revision: 39655

In which case, a verbose switch in the csprocessor.ini would be a good idea.

Comment 1 Lee Newson 2012-02-21 06:55:06 UTC

The only part I agree with here is removing the "INFO:  The Content Specification is valid." message. The others ensure that the user knows what server/configuration they are connecting to. I guess the CSProcessor client message could be removed. 

The "WARN:  Note: All descriptions, tags, source urls and writers will be ignored
for existing Topics." is key to ensure that the user knows that those parts will be ignored in their content specification.

Anyways I'll try and catch you tomorrow to talk about it.

Comment 2 Lee Newson 2012-03-06 02:09:15 UTC

Update:

The WARN message has been removed from version 0.22.0 since tags can be added to existing topics in that version.

Comment 3 Misty Stanley-Jones 2013-05-29 02:06:47 UTC

Closing because of Lee's last comment.