<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 15, 2017 at 7:25 AM, Monty Taylor <span dir="ltr"><<a href="mailto:mordred@inaugust.com" target="_blank">mordred@inaugust.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 02/15/2017 09:12 AM, Hayes, Graham wrote:<br>
> On 15/02/2017 15:00, Monty Taylor wrote:<br>
>> On 02/14/2017 07:08 PM, Qiming Teng wrote:<br>
>>> When reviewing a recent patch that adds openstacksdk support to octavia,<br>
>>> I found that octavia is using 'octavia' as its service name instead of<br>
>>> 'loadbalancing' or 'loadbalancingv2' or something similar.<br>
>><br>
>> Please not loadbalancingv2. As dean says in his email, we should be<br>
>> using service_type not service_name for this. And service type should<br>
>> not contain a version (please ignore what cinder did for v2 and v3<br>
>> entries in the service catalog, it is a pattern that should not happen)<br>
><br>
> +1000<br>
><br>
>> All the services should have a version discovery endpoint on their<br>
>> unversioned endpoint. If there is a v1 and a v2, then a user looking for<br>
>> the loadbalancing service, if they want v2, should be able to get there<br>
>> through version discovery.<br>
>><br>
>> Also, if you haven't used loadbalancing anywhere yet, can I suggest<br>
>> load-balancing instead to match object-store and key-manager?<br>
>><br>
>>> The overall suggestion is to use a word/phrase that indicates what a<br>
>>> service do instead of the name of the project providing that service.<br>
>>><br>
>>> Below is the list of the service types currently supported by<br>
>>> openstacksdk:<br>
>>><br>
>>>     'alarming',    # aodh<br>
>>>     'baremetal',   # ironic<br>
>>>     'clustering',  # senlin<br>
>>>     'compute',     # nova<br>
>>>     'database',    # trove<br>
>>>     'identity',    # keystone<br>
>>>     'image',       # glance<br>
>>>     'key-manager', # barbican<br>
>>>     'messaging',   # zaqar<br>
>>>     'metering',    # ceilometer<br>
>>>     'network',     # neutron<br>
>>>     'object-store',   # swift<br>
>>>     'octavia',        # <--- this is an exception<br>
>>>     'orchestration',  # heat<br>
>>>     'volume',         # cinder<br>
>>>     'workflowv2',     # mistral<br>
<br>
</div></div>Also - while we're on the topic - can we fix that to just be workflow ^^ ?<br>
<div class="HOEnZb"><div class="h5"><br>
<br></div></div></blockquote><div>++</div><div>Please change that to workflow if possible. </div></div></div></div>