Bug 663453 - [RFE] Add servername attribute to pulp.conf
Summary: [RFE] Add servername attribute to pulp.conf
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Pulp
Classification: Retired
Component: z_other
Version: unspecified
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: Sprint 20
Assignee: Pradeep Kilambi
QA Contact: Preethi Thomas
URL:
Whiteboard:
Depends On:
Blocks: verified-to-close
TreeView+ depends on / blocked
 
Reported: 2010-12-15 20:32 UTC by Todd Sanders
Modified: 2011-08-15 15:14 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2011-08-15 15:14:11 UTC
Embargoed:


Attachments (Terms of Use)

Description Todd Sanders 2010-12-15 20:32:25 UTC
Description of problem:

Rather than "base_url=http://localhost", let's move to the following:

[server]
server_name=localhost

This is in place of base_url.  Also, we require ssl, so should always be https.

Also,

[messaging]
server_name=localhost /* This is a local override if you are connecting to an external broker */
server_port=5672

Comment 1 Jay Dobies 2011-01-11 19:46:13 UTC
There are three items to do as a result of discussing this bug:
1. Refactor baseurl into servername
2. Remove the "scheme" attribute; we always require https so just default it.
3. Remove [messaging] entirely, this is now held in the gofer configuration.

Comment 2 Jeff Ortel 2011-01-11 20:49:27 UTC
 [messaging] needs to stay in the pulp.conf.  It controls how the pulp server connects to qpid.

Comment 3 Jeff Ortel 2011-01-11 20:51:44 UTC
What is the base_url used for?

Comment 4 Pradeep Kilambi 2011-02-03 20:46:45 UTC
also remove local_storage as an config option. Allowing users to update this can do bad things to pulp data.

Comment 5 Pradeep Kilambi 2011-02-03 21:52:02 UTC
commit e9007e187a114d8eb82f3906a4979e285c1e0542
Author: Pradeep Kilambi <pkilambi>
Date:   Thu Feb 3 16:51:30 2011 -0500


commit 9849c900c65acefa70db52760e698868f4cac7ee
Author: Pradeep Kilambi <pkilambi>
Date:   Thu Feb 3 12:27:27 2011 -0500

Comment 6 Jay Dobies 2011-02-04 19:24:01 UTC
Fixed in 0.135.

Comment 7 Preethi Thomas 2011-03-07 20:40:27 UTC
[root@preethi ~]# rpm -q pulp
pulp-0.0.144-1.fc14.noarch

failing as step 2 in comment#1 is still not addressed.

from client.cong


[server]
host = localhost
port = 443
scheme = https
path = /pulp/api
interval = 240

Comment 8 Preethi Thomas 2011-08-15 15:14:11 UTC
Closing with current community release

pulp-0.0.223


Note You need to log in before you can comment on or make changes to this bug.