Devstack does not deal well will quantum lowercase section names

[Gary Kotton]

devstack has hard coded upper case section names. After the patches:

https://review.openstack.org/gitweb?p=openstack%2Fquantum.git;a=commitdiff;h=fd7223bcd4709fcaaf103fb5f0f14e90ca3f00d7
https://review.openstack.org/33429

and

https://review.openstack.org/gitweb?p=openstack%2Fquantum.git;a=commitdiff;h=8a6143e145f3ed46ef3d723e83c9f4f4a5447729
https://review.openstack.org/33432

quantum will have upper and lower case section names.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/34177

[Mark McLoughlin (markmc)]

Ok, we've dug into this bug in some detail and the summary is that if you have a file with:

  [database]
  connection = sqlite:///...

  [database]
  sql_connection = mysql:///...

then the value returned by cfg will be the 'sqlite:///' value

This is arguably a bug in cfg and is seen in https://gist.github.com/markmc/5858373 ... basically the behaviour with t1.conf is not what you might expect. However, after some discussion we think is fairly reasonable behaviour and I'm going to mark it as WontFix in oslo.

Now, there is something we can do to help the situation in quantum - if we have the default values commented out everywhere, then even if the user supplies a value using the deprecated name it won't be overwritten by a default value using the new name

i.e. we would have:

  [database]
  #connection = sqlite:///...

  [database]
  sql_connection = mysql:///...

So, I'm opening a task against quantum to comment out default values

[Dean Troyer (dtroyer)]

Is this going to be an upgrade issue? I'm hoping Mark's suggestion about commenting out defaults will handle that.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/34177
Committed: http://github.com/openstack-dev/devstack/commit/d42634ff631e5faa5e3a91f581c861e59279e6f7
Submitter: Jenkins
Branch: master

commit d42634ff631e5faa5e3a91f581c861e59279e6f7
Author: Gary Kotton <email address hidden>
Date: Mon Jun 24 09:26:55 2013 +0000

    Use lowercase section names in Neutron (aka Quantum)

    Fixes bug 1194064

    The DEFAULT section remains the same.

    In addition to this the sql_connection has been changed to
    connection - this is for OSLO DB support.

    Change-Id: I56175146941a9cae966033b557e44097e29a5d43

[Mate Lakat (mate-lakat)]

Added permalinks

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/40164

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/40164
Committed: http://github.com/openstack/neutron/commit/3313448f2de1ce288944af101a794d706fd43abb
Submitter: Jenkins
Branch: master

commit 3313448f2de1ce288944af101a794d706fd43abb
Author: ZhiQiang Fan <email address hidden>
Date: Mon Jul 29 18:40:46 2013 +0800

    Avoid overwrite value of deprecated name in conf.

    See bug/1194064 #2 Mark McLoughlin's comment

    Fixes: Bug #1194064

    Change-Id: I421d2decfa8829796fe7b6c14f954f3cccc17215