Bug 887818 - a comprehensive /etc/swift/*.conf files should be included
Summary: a comprehensive /etc/swift/*.conf files should be included
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: openstack-swift
Version: 1.0 (Essex)
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: beta
: 4.0
Assignee: Pete Zaitcev
QA Contact: Dafna Ron
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-12-17 12:05 UTC by Nikola Dipanov
Modified: 2016-04-26 20:59 UTC (History)
11 users (show)

Fixed In Version: openstack-swift-1.8.0-7.el6ost
Doc Type: Bug Fix
Doc Text:
Clone Of: 887334
Environment:
Last Closed: 2013-12-19 23:53:30 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Proposed fix (3.15 KB, patch)
2013-06-06 17:21 UTC, Pete Zaitcev
ndipanov: review+
Details | Diff
Proposed fix (32.54 KB, patch)
2013-06-26 00:10 UTC, Pete Zaitcev
zaitcev: review? (ndipanov)
apevec: review+
Details | Diff


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2013:1859 0 normal SHIPPED_LIVE Red Hat Enterprise Linux OpenStack Platform Enhancement Advisory 2013-12-21 00:01:48 UTC

Description Nikola Dipanov 2012-12-17 12:05:06 UTC
+++ This bug was initially created as a clone of Bug #887334 +++

Description of problem:

A .conf file with a comprehensive set of configuration options should be shipped various nova services.  Here is a good example to work from:

https://github.com/yocum137/nova.conf/blob/master/nova.conf

--- Additional comment from Dan Yocum on 2012-12-14 12:22:34 EST ---

This one looks like it's more up-to-date:

https://github.com/openstack/nova/blob/master/etc/nova/nova.conf.sample

Comment 2 Alan Pevec 2012-12-18 14:15:38 UTC
Swift is of course special and its sample configs are not that useful as in other projects which use cfg and have autogenerated conf samples with doc comments: https://github.com/openstack/swift/tree/master/etc
So this will require some upstream work, ideally to convert Swift to use oslo-cfg, and minimally to add doc comments in sample configs manually.

Comment 4 Pete Zaitcev 2013-06-06 17:21:06 UTC
Created attachment 757780 [details]
Proposed fix

I find the complete configs for Swift rather unweildy and difficult to
edit, although they would not be without precendent. Look at sample
configs for things like Postfix and Apache.

Still, I question my own ability to maintain them in 100% coherence
with the reality. I barely have cycles to audit changes to upstream
configs that we repackage as foo-sample.

Nicola, do you think you could be satisfied with better redirection
to comprehensive sample configs that we already install? Please see
the attached patch.

Comment 5 Pete Zaitcev 2013-06-26 00:10:30 UTC
Created attachment 765343 [details]
Proposed fix

Comment 6 Alan Pevec 2013-06-26 12:38:21 UTC
Comment on attachment 765343 [details]
Proposed fix

Looks good enough, it synces what upstream currently has in etc/*-sample but ideally upstream could improve the samples and add more documentation comments.
Even more ideally, upstream would switch to oslo.config and generate samples from the code, like Nova does https://github.com/openstack/nova/tree/master/tools/conf

Comment 12 Dafna Ron 2013-10-03 08:46:43 UTC
conf files (although not hight detailed) exists: 
[root@nott-vdsa swift(keystone_admin)]# ls -l 
total 2456
-rw-r--r--. 1 root  swift 787324 Sep 27 19:02 account.builder
-rw-r--r--. 1 root  swift  37564 Sep 27 19:02 account.ring.gz
drwxr-sr-x. 2 root  swift   4096 Sep 27 19:02 backups
-rw-r--r--. 1 root  swift 787326 Sep 27 19:02 container.builder
-rw-r--r--. 1 root  swift  37730 Sep 27 19:02 container.ring.gz
-rw-r--r--. 1 root  swift 787326 Sep 27 19:02 object.builder
-rw-rw----. 1 root  swift    291 Jun 20 13:24 object-expirer.conf
-rw-r--r--. 1 root  swift  37588 Sep 27 19:02 object.ring.gz
drwxr-xr-x. 2 root  root    4096 Jun 20 13:26 proxy-server
-rw-rw----. 1 swift swift   1108 Sep 25 17:47 proxy-server.conf
-rw-rw----. 1 swift swift     58 Sep 25 17:46 swift.conf

verified for openstack-swift-1.8.0-6.el6ost.noarch

Comment 15 errata-xmlrpc 2013-12-19 23:53:30 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHEA-2013-1859.html


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