Project

Profile

Help

Issue #376

closed

Not able to update relative-url to default

Added by pthomas@redhat.com about 9 years ago. Updated about 5 years ago.

Status:
CLOSED - WONTFIX
Priority:
Low
Assignee:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Master
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

Description of problem:

If you have repo with relative url specified, it wont change it back to default with repo update

Version-Release number of selected component (if applicable):

[root@pulp-v2-server ~]# rpm -qa pulp-server
pulp-server-2.3.0-0.16.alpha.el6.noarch

How reproducible:

Steps to Reproduce:
1. create a repo with feed and --reltive-url
2. see repo list details and notice that --relatove-url is set
3. run repo update with --relative-url ""
4. See repo list details again and notice that relative url is still the one from step 2

Actual results:

[root@pulp-v2-server ~]# pulp-admin rpm repo create --repo-id demo --feed http://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/pulp_unittest/ --relative-url /demo/url
Successfully created repository [demo]

pulp-admin repo list --details
+----------------------------------------------------------------------+
                             Repositories
+----------------------------------------------------------------------+

Id: demo
Display Name: demo
Description: None
Content Unit Counts:
Notes:
Importers:
Config:
Feed: http://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/pulp_unittest
/
Id: yum_importer
Importer Type Id: yum_importer
Last Sync: None
Repo Id: demo
Scheduled Syncs:
Distributors:
Auto Publish: True
Config:
Http: False
Https: True
Relative URL: /demo/url
Distributor Type Id: yum_distributor
Id: yum_distributor
Last Publish: None
Repo Id: demo
Scheduled Publishes:
Auto Publish: False
Config:
Http: False
Https: True
Distributor Type Id: export_distributor
Id: export_distributor
Last Publish: None
Repo Id: demo
Scheduled Publishes:

[root@pulp-v2-server ~]#
[root@pulp-v2-server ~]# pulp-admin rpm repo update --repo-id demo --relative-url ""
Repository [demo] successfully updated

Id: demo
Display Name: demo
Description: None
Content Unit Counts:
Notes:
Importers:
Config:
Feed: http://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/pulp_unittest
/
Id: yum_importer
Importer Type Id: yum_importer
Last Sync: None
Repo Id: demo
Scheduled Syncs:
Distributors:
Auto Publish: True
Config:
Http: False
Https: True
Relative URL: /demo/url
Distributor Type Id: yum_distributor
Id: yum_distributor
Last Publish: None
Repo Id: demo
Scheduled Publishes:
Auto Publish: False
Config:
Http: False
Https: True
Distributor Type Id: export_distributor
Id: export_distributor
Last Publish: None
Repo Id: demo
Scheduled Publishes:

Expected results:

Additional info:

+ This bug was cloned from Bugzilla Bug #1014649 +

Actions #1

Updated by mhrivnak about 9 years ago

Part of what makes this challenging is that we're deriving a distributor config value based on an importer config value (the feed). It's possible that repo creation time is the only reasonable time (if any) for us to make an assumption that a missing distributor config value can be generated based on the supplied importer config value (the feed). Generally speaking, importers and distributors should be as independent as possible.

+ This comment was cloned from Bugzilla #1014649 comment 1 +

Actions #2

Updated by bmbouter about 5 years ago

  • Status changed from NEW to CLOSED - WONTFIX
  • Severity set to 2. Medium

Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.

Actions #3

Updated by bmbouter about 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF