openstack team mailing list archive

[Lorin Hochstein <lorin@xxxxxxxxxxxxxxxxxx>]

On Apr 13, 2012, at 12:34 PM, Adam Gandelman wrote:

> On 04/13/2012 10:50 AM, Dolph Mathews wrote:
>> 
>> While $(tenant_id)s is certainly the documented syntax, it appears that the SQL catalog backend (and *only* the SQL catalog backend, as far as I can tell) explicitly supports both $(tenant_id)s and %(tenant_id)s:
>> 
>> https://github.com/openstack/keystone/blob/master/keystone/catalog/backends/sql.py#L163
>> 
>> Perhaps Adam Gandelman has some insight?
>> 
>> -Dolph
> 
> Dolph-
> 
> No, the same is supported in the case of templated catalog as well, which is what the SQL catalog was largely based off:
> 
> https://github.com/openstack/keystone/blob/master/keystone/catalog/backends/templated.py#L115
> 
> Just tested that "sed -i 's/\$/%/g' /etc/keystone/default_catalog.templates" still produces a functional service catalog when configured to use the templated backend.
> 
> Seeing as both are supported, perhaps it would be better for docs to be updated to refer to the use of % instead of $ to avoid people running into problems with the $() sub-shell?
> 

The OpenStack Install and Deploy manual has some language about this (see last paragraph): http://docs.openstack.org/trunk/openstack-compute/install/content/elements-of-keystone-service-catalog-entry.html

This hasn't made its way into the admin docs yet, though.


Take care,

Lorin
--
Lorin Hochstein
Lead Architect - Cloud Services
Nimbis Services, Inc.
www.nimbisservices.com