Bug 845371

Summary:	Adding members to a repo group needs a success message
Product:	[Retired] Pulp	Reporter:	Jay Dobies <jason.dobies>
Component:	user-experience	Assignee:	Jay Dobies <jason.dobies>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Preethi Thomas <pthomas>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	2.0.6	CC:	skarmark
Target Milestone:	---	Keywords:	Triaged
Target Release:	Sprint 39
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-01-09 17:04:01 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Jay Dobies 2012-08-02 20:14:28 UTC

╭─[jdob]─[hinterlands]─[~]─[]─●
╰─➤ pulp-admin repo group members add --group-id comics --in "id=avengers,jla"
╭─[jdob]─[hinterlands]─[~]─[]─●
╰─➤   

There's no output indicating if it was successful or failed. There's a call in the prompt called render_success_message() that should be used to indicate the call was successful if it was.

Comment 1 Jay Dobies 2012-08-22 20:49:29 UTC

I'm refactoring these to the client package itself and will fix as part of that.

Comment 2 Jay Dobies 2012-08-23 17:15:28 UTC

commit 907a8065fd62bd9ac08330cdf8f559d2ef240a05
Author: Jay Dobies <jason.dobies>
Date:   Thu Aug 23 13:13:49 2012 -0400

    - Refactored repo and repo group commands to the client package

builtins/extensions/admin/pulp_repo/pulp_cli.py
builtins/test/unit/test_pulp_repo_extension.py
builtins/test/unit/test_repo_group_member_section.py
builtins/test/unit/test_repo_group_section.py
platform/src/pulp/client/commands/criteria.py
platform/src/pulp/client/commands/options.py
platform/src/pulp/client/commands/repo/cudl.py
platform/src/pulp/client/commands/repo/group.py
platform/src/pulp/client/extensions/core.py
platform/src/pulp/client/extensions/extensions.py
platform/test/unit/test_commands_repo_cudl.py
platform/test/unit/test_commands_repo_group.py
platform/test/unit/test_search_command.py

Comment 3 Jeff Ortel 2012-09-11 13:31:23 UTC

build: 0.327

Comment 4 Preethi Thomas 2012-09-27 15:33:58 UTC

verified

[root@pulp-master ~]# rpm -q pulp-rpm-server
pulp-rpm-server-0.0.328-1.fc17.noarch

[root@pulp-master ~]# pulp-admin repo group members add --group-id pulp --in "id=pulp,pulp-clone1"
Successfully added members to repository group [pulp]

[root@pulp-master ~]# pulp-admin repo group list
+----------------------------------------------------------------------+
                           Repository Groups
+----------------------------------------------------------------------+

Id:           pulp
Display Name: None
Description:  None
Repo Ids:     pulp, pulp-clone

Comment 5 Preethi Thomas 2013-01-09 17:04:01 UTC

Pulp v2.0 released