Bug 663453

Summary:	[RFE] Add servername attribute to pulp.conf
Product:	[Retired] Pulp	Reporter:	Todd Sanders <tsanders>
Component:	z_other	Assignee:	Pradeep Kilambi <pkilambi>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Preethi Thomas <pthomas>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	unspecified	CC:	jason.dobies, jortel, skarmark
Target Milestone:	---	Keywords:	FutureFeature, Triaged
Target Release:	Sprint 20
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Enhancement
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2011-08-15 15:14:11 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-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