keystone endpoint registration uses class variable named `service_type` for what Keystone describes as "Service Name"

Bug #1796910 reported by Frode Nordahl on 2018-10-09
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack API Layer
Low
Unassigned
charms.openstack
Low
Unassigned

Bug Description

Example charm implementation:
https://github.com/openstack-charmers/charm-octavia/blob/975e80d99f810c1230cc23e8c2ce2afb3c3736b7/src/lib/charm/openstack/octavia.py#L47

Example keystone charm endpoint implementation:
https://github.com/openstack/charm-keystone/blob/1c1c41c7f1cb43a795acbece6ef8b50c4e91b88d/hooks/keystone_utils.py#L417

$ openstack endpoint list
+----------------------------------+-----------+--------------+---------------+---------+-----------+-------------------------------+
| ID | Region | Service Name | Service Type | Enabled | Interface | URL |
+----------------------------------+-----------+--------------+---------------+---------+-----------+-------------------------------+
| 49ccebaebf8a476489e5ed3cef813f24 | RegionOne | keystone | identity | True | admin | http://10.130.236.12:35357/v3 |
| 76453a022199459db0d96b182e7e810a | RegionOne | octavia | load-balancer | True | internal | http://10.130.236.167:9876 |
| 7e4b1787708e41de91941a038e855f5a | RegionOne | keystone | identity | True | internal | http://10.130.236.12:5000/v3 |
| a7897ad7936d4cb08ad1d83322b2f693 | RegionOne | octavia | load-balancer | True | public | http://10.130.236.167:9876 |
| b4dd97462e0748758fef913647ebfbe4 | RegionOne | keystone | identity | True | public | http://10.130.236.12:5000/v3 |
| b8b283a7fba34ba0ac141dd793290f3b | RegionOne | octavia | load-balancer | True | admin | http://10.130.236.167:9876 |
+----------------------------------+-----------+--------------+---------------+---------+-----------+-------------------------------+

Frode Nordahl (fnordahl) on 2018-10-09
description: updated
James Page (james-page) wrote :

From the interface:

    def register_endpoints(self, service, region, public_url, internal_url,
                           admin_url):
        """
        Register this service with keystone
        """
        relation_info = {
            'service': service,
            'public_url': public_url,
            'internal_url': internal_url,
            'admin_url': admin_url,
            'region': region,
        }
        self.set_local(**relation_info)
        self.set_remote(**relation_info)

Agree service_type is a bit misleading - maybe switch to service name with a backwards compat accessor?

Changed in layer-openstack-api:
status: New → Triaged
Changed in charms.openstack:
status: New → Triaged
Changed in layer-openstack-api:
importance: Undecided → Low
Changed in charms.openstack:
importance: Undecided → Low
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers