Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 707295

Summary: relativepath option in repo update
Product: [Retired] Pulp Reporter: Preethi Thomas <pthomas>
Component: z_otherAssignee: Pradeep Kilambi <pkilambi>
Status: CLOSED CURRENTRELEASE QA Contact: Preethi Thomas <pthomas>
Severity: unspecified Docs Contact:
Priority: medium    
Version: unspecifiedCC: jortel, skarmark
Target Milestone: ---Keywords: Triaged
Target Release: Sprint 25   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-08-16 12:06:56 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 Preethi Thomas 2011-05-24 16:03:03 UTC
Description of problem.

Running repo update with --relativepath option always fails now since createrepo is being run at repo create time.  So the option needs to be removed from repo update.

Comment 1 Jeff Ortel 2011-05-25 16:03:28 UTC
Need to remove the --relativepath option from the CLI and the API class.  While we're there, let's make sure the --feed check is correct.  It needs to make sure the relative path is not changed.  We can also remove the code that checks to see if the repo has content.  Since the repo always has metadata, it's probably broken anyway.

Comment 2 Pradeep Kilambi 2011-06-17 14:42:30 UTC
commit 58b2e4aa2d8d95dcab7c33c867be00c5b7d0a473
Author: Pradeep Kilambi <pkilambi>
Date:   Fri Jun 17 10:40:45 2011 -0400

    707295 - removed relativepath from repo update; updated feed update logic to check if relative path matches before allowing update



$ sudo pulp-admin repo create --id=testrepo --feed=http://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/test_bandwidth_repo/
Successfully created repository [ testrepo ]

$ sudo pulp-admin repo update --feed=http://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/test_bandwidth_repo/ --id=testrepo
Successfully updated repository [ testrepo ]

$ sudo pulp-admin repo update --feed=http://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/test_bandwidth_repo/fake/ --id=testrepo
error: operation failed: PulpException: u'Relativepath of the new feed [repos/pulp/pulp/demo_repos/test_bandwidth_repo/fake] does not match existing feed [repos/pulp/pulp/demo_repos/test_bandwidth_repo]; cannot perform update'

Comment 3 Jeff Ortel 2011-06-17 21:10:11 UTC
build: 0.192

Comment 4 Preethi Thomas 2011-07-07 16:20:33 UTC
[root@preethi ~]# rpm -q pulp
pulp-0.0.202-1.fc14.noarch

[root@preethi ~]# pulp-admin repo create --id=testrepo --feed=http://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/test_bandwidth_repo/
Successfully created repository [ testrepo ]

[root@preethi ~]# pulp-admin repo update --id=testrepo --feed=http://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/test_bandwidth_repo/
Successfully updated repository [ testrepo ]

[root@preethi ~]# pulp-admin repo update --id=testrepo --feed=http://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/test_bandwidth_repo/fake/
error: operation failed: PulpException: u'Relativepath of the new feed [repos/pulp/pulp/demo_repos/test_bandwidth_repo/fake] does not match existing feed [repos/pulp/pulp/demo_repos/test_bandwidth_repo]; cannot perform update'
[root@preethi ~]#

Comment 5 Preethi Thomas 2011-08-16 12:06:56 UTC
Closing with Community Release 15

pulp-0.0.223-4.

Comment 6 Preethi Thomas 2011-08-16 12:20:49 UTC
Closing with Community Release 15

pulp-0.0.223-4.