Bug 662247

Summary: pulp-admin repo clone should *require* that --clone_id be passed.
Product: [Retired] Pulp Reporter: Todd Sanders <tsanders>
Component: z_otherAssignee: Pradeep Kilambi <pkilambi>
Status: CLOSED CURRENTRELEASE QA Contact: Preethi Thomas <pthomas>
Severity: medium Docs Contact:
Priority: low    
Version: unspecified   
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: 2011-02-17 16:52:15 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: 647488    

Description Todd Sanders 2010-12-11 00:30:01 UTC
Description of problem:

*Fedora14
*Pulp v0.109

[ec2-user@ip-10-203-21-224 ~]$ sudo pulp-admin repo clone --id=rhel5-i386
Repository [rhel5-i386] is being cloned as [None]

*Both --help and functionality need to be updated to *require* id of cloned repo -> --clone_id

We should check this on the pulp-client as well.

Comment 1 Pradeep Kilambi 2010-12-22 19:58:50 UTC
commit 2e295b7656e5a4b4f7c72135da2184a37fd20876
Author: Pradeep Kilambi <pkilambi>
Date:   Wed Dec 22 14:58:03 2010 -0500

    662247 - making clone_id a required option

$ sudo pulp-admin -u admin -p admin repo clone  --help
Usage: pulp-admin <options> repo clone <options>

Options:
  -h, --help            show this help message and exit
  --id=ID               repository id (required)
  --clone_id=CLONE_ID   id of cloned repo (required)

$ sudo pulp-admin -u admin -p admin repo clone --id=foo123 
Usage: pulp-admin <options> repo clone <options>

pulp-admin: error: Option --clone_id is required; please see --help

Comment 2 Jay Dobies 2010-12-22 20:56:29 UTC
Fixed in 0.119.

Comment 3 Preethi Thomas 2010-12-23 15:22:13 UTC
verified


[root@pulp ~]# rpm -q pulp
pulp-0.0.119-1.fc14.noarch
[root@pulp ~]# 
[root@pulp ~]# 
[root@pulp ~]# 
[root@pulp ~]# pulp-admin repo clone
Usage: pulp-admin <options> repo clone <options>

pulp-admin: error: Option --id is required; please see --help
[root@pulp ~]# pulp-admin repo clone --help
Usage: pulp-admin <options> repo clone <options>

Options:
  -h, --help            show this help message and exit
  --id=ID               repository id (required)
  --clone_id=CLONE_ID   id of cloned repo (required)
  --clone_name=CLONE_NAME
                        common repository name for cloned repo
  --feed=FEED           feed of cloned_repo: parent/origin/none
  --groupid=GROUPID     a group to which the repository belongs; this is just
                        a string identifier
  --timeout=TIMEOUT     repository clone timeout
  -F, --foreground      clone repository in the foreground
[root@pulp ~]# pulp-admin repo clone --clone_id=test 
Usage: pulp-admin <options> repo clone <options>

pulp-admin: error: Option --id is required; please see --help
[root@pulp ~]# pulp-admin repo clone --id=f13test --clone_name=test --feed=none
Usage: pulp-admin <options> repo clone <options>

pulp-admin: error: Option --clone_id is required; please see --help
[root@pulp ~]#

Comment 4 Preethi Thomas 2011-02-17 16:52:15 UTC
Closing with community release 

pulp-0.0.139-1.fc14.noarch