)]}'
{"specs/pike/approved/use-service-catalog-for-endpoints.rst":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"dd3ff9c58de31dd0ff4b85cde8e62e18b34de0ae","unresolved":false,"context_lines":[{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Currently, Nova uses configuration settings from nova.conf file to get"},{"line_number":53,"context_line":"endpoint URL. Each service has url param in nova.conf which represents"},{"line_number":54,"context_line":"service endpoint. For Example, In nova.conf, there are many endpoints"},{"line_number":55,"context_line":"like:"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"[glance]"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7ffa3b31_3659f759","line":54,"range":{"start_line":54,"start_character":31,"end_line":54,"end_character":33},"updated":"2017-04-14 20:42:15.000000000","message":"in","commit_id":"50a8cc4cc3ce815e836c07b863c3370996c28e34"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"dd3ff9c58de31dd0ff4b85cde8e62e18b34de0ae","unresolved":false,"context_lines":[{"line_number":74,"context_line":"endpoint instead of fetching from configuration file."},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"This method finds specified API endpoint from the service catalog for a"},{"line_number":77,"context_line":"perticular endpoint attribute using keystoneauth. The attributes can be:"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"``service_type``: Service type of the endpoint."},{"line_number":80,"context_line":"``interface``: Type of endpoint."}],"source_content_type":"text/x-rst","patch_set":3,"id":"7ffa3b31_d63533f9","line":77,"range":{"start_line":77,"start_character":0,"end_line":77,"end_character":10},"updated":"2017-04-14 20:42:15.000000000","message":"particular","commit_id":"50a8cc4cc3ce815e836c07b863c3370996c28e34"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"dd3ff9c58de31dd0ff4b85cde8e62e18b34de0ae","unresolved":false,"context_lines":[{"line_number":77,"context_line":"perticular endpoint attribute using keystoneauth. The attributes can be:"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"``service_type``: Service type of the endpoint."},{"line_number":80,"context_line":"``interface``: Type of endpoint."},{"line_number":81,"context_line":"``region_name``: Region of the endpoint."},{"line_number":82,"context_line":"``service_name``: The assigned name of the service."},{"line_number":83,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"7ffa3b31_b63aa7e4","line":80,"range":{"start_line":80,"start_character":0,"end_line":80,"end_character":32},"updated":"2017-04-14 20:42:15.000000000","message":"generally public, internal, admin","commit_id":"50a8cc4cc3ce815e836c07b863c3370996c28e34"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"dd3ff9c58de31dd0ff4b85cde8e62e18b34de0ae","unresolved":false,"context_lines":[{"line_number":79,"context_line":"``service_type``: Service type of the endpoint."},{"line_number":80,"context_line":"``interface``: Type of endpoint."},{"line_number":81,"context_line":"``region_name``: Region of the endpoint."},{"line_number":82,"context_line":"``service_name``: The assigned name of the service."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"If this method fails to get API endpoint from service catalog then, by default"},{"line_number":85,"context_line":"we will find it using ``endpoint_override`` configuration. That is, we will"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7ffa3b31_76241fc7","line":82,"range":{"start_line":82,"start_character":0,"end_line":82,"end_character":51},"updated":"2017-04-14 20:42:15.000000000","message":"I don\u0027t know that we ever care about service name.","commit_id":"50a8cc4cc3ce815e836c07b863c3370996c28e34"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"dd3ff9c58de31dd0ff4b85cde8e62e18b34de0ae","unresolved":false,"context_lines":[{"line_number":81,"context_line":"``region_name``: Region of the endpoint."},{"line_number":82,"context_line":"``service_name``: The assigned name of the service."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"If this method fails to get API endpoint from service catalog then, by default"},{"line_number":85,"context_line":"we will find it using ``endpoint_override`` configuration. That is, we will"},{"line_number":86,"context_line":"override service catalog lookup and get required URL from configuration."},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"By adding this change, we will have consistent way to connect with other"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7ffa3b31_b613c766","line":85,"range":{"start_line":84,"start_character":68,"end_line":85,"end_character":57},"updated":"2017-04-14 20:42:15.000000000","message":"Does keystoneauth provide this option or does nova have to? I feel like this was already a backdoor option KSA provided.","commit_id":"50a8cc4cc3ce815e836c07b863c3370996c28e34"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"dd3ff9c58de31dd0ff4b85cde8e62e18b34de0ae","unresolved":false,"context_lines":[{"line_number":91,"context_line":""},{"line_number":92,"context_line":"Alternatives"},{"line_number":93,"context_line":"------------"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"Data model impact"},{"line_number":97,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7ffa3b31_560a4326","line":94,"updated":"2017-04-14 20:42:15.000000000","message":"None","commit_id":"50a8cc4cc3ce815e836c07b863c3370996c28e34"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"dd3ff9c58de31dd0ff4b85cde8e62e18b34de0ae","unresolved":false,"context_lines":[{"line_number":110,"context_line":""},{"line_number":111,"context_line":"Notifications impact"},{"line_number":112,"context_line":"--------------------"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"Other end user impact"},{"line_number":116,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7ffa3b31_3607b71f","line":113,"updated":"2017-04-14 20:42:15.000000000","message":"None","commit_id":"50a8cc4cc3ce815e836c07b863c3370996c28e34"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"dd3ff9c58de31dd0ff4b85cde8e62e18b34de0ae","unresolved":false,"context_lines":[{"line_number":113,"context_line":""},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"Other end user impact"},{"line_number":116,"context_line":"---------------------"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Performance Impact"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7ffa3b31_1604bb16","line":116,"updated":"2017-04-14 20:42:15.000000000","message":"None","commit_id":"50a8cc4cc3ce815e836c07b863c3370996c28e34"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"dd3ff9c58de31dd0ff4b85cde8e62e18b34de0ae","unresolved":false,"context_lines":[{"line_number":119,"context_line":"Performance Impact"},{"line_number":120,"context_line":"------------------"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"* Request to get endpoint will now passed to the keystoneauth instead of"},{"line_number":123,"context_line":"  reading from configuration file."},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"Other deployer impact"},{"line_number":126,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7ffa3b31_e9fe1004","line":123,"range":{"start_line":122,"start_character":2,"end_line":123,"end_character":34},"updated":"2017-04-14 20:42:15.000000000","message":"None","commit_id":"50a8cc4cc3ce815e836c07b863c3370996c28e34"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"dd3ff9c58de31dd0ff4b85cde8e62e18b34de0ae","unresolved":false,"context_lines":[{"line_number":171,"context_line":"History"},{"line_number":172,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"Optional section intended to be used each time the spec is updated to describe"},{"line_number":175,"context_line":"new design, API or any database schema updated. Useful to let reader understand"},{"line_number":176,"context_line":"what\u0027s happened along the time."},{"line_number":177,"context_line":""},{"line_number":178,"context_line":".. list-table:: Revisions"},{"line_number":179,"context_line":"   :header-rows: 1"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7ffa3b31_291cf868","line":176,"range":{"start_line":174,"start_character":0,"end_line":176,"end_character":31},"updated":"2017-04-14 20:42:15.000000000","message":"Remove this template text.","commit_id":"50a8cc4cc3ce815e836c07b863c3370996c28e34"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"5b601712ae2ce1356a609ab7bcf0091e52a2b77e","unresolved":false,"context_lines":[{"line_number":29,"context_line":"interacts with Cinder, Glance and Neutron."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"The service catalog is composed of list of services which represents"},{"line_number":32,"context_line":"web services in OpenStack deployment. Each service has zero or more endpoints"},{"line_number":33,"context_line":"associated with it."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Use Cases"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7ffa3b31_096a5c7f","line":32,"updated":"2017-04-14 21:02:13.000000000","message":"well, really *One or more*. A service with no endpoints is, well, not a service :)","commit_id":"2a57986851e8b8ee335a480969d7d1183e076856"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"5b601712ae2ce1356a609ab7bcf0091e52a2b77e","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Nova uses API endpoints to establish communication with other services like"},{"line_number":39,"context_line":"Cinder, Glance and Neutron. These endpoints are currently part of nova"},{"line_number":40,"context_line":"configuration file. Ideally, rather than hardcoding these URL\u0027s values in"},{"line_number":41,"context_line":"configuration file, use the service catalog to provide discoverability"},{"line_number":42,"context_line":"for those."},{"line_number":43,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"7ffa3b31_e904d0c8","line":40,"updated":"2017-04-14 21:02:13.000000000","message":"s/URL\u0027s/URLs/","commit_id":"2a57986851e8b8ee335a480969d7d1183e076856"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"5b601712ae2ce1356a609ab7bcf0091e52a2b77e","unresolved":false,"context_lines":[{"line_number":44,"context_line":"Proposed change"},{"line_number":45,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"The service catalog offers list of all available services along with"},{"line_number":48,"context_line":"additional information about regions, API endpoints and API versions."},{"line_number":49,"context_line":"It is useful to efficiently find information about services such as how to"},{"line_number":50,"context_line":"configure communication between services."}],"source_content_type":"text/x-rst","patch_set":4,"id":"7ffa3b31_c90914de","line":47,"updated":"2017-04-14 21:02:13.000000000","message":"s/offers list of/offers a list of/","commit_id":"2a57986851e8b8ee335a480969d7d1183e076856"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"5b601712ae2ce1356a609ab7bcf0091e52a2b77e","unresolved":false,"context_lines":[{"line_number":59,"context_line":"   [neutron]"},{"line_number":60,"context_line":"   url \u003d http://127.0.0.1:9696"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"Keystoneauth provides simple and consistent way to get API endpoints from"},{"line_number":63,"context_line":"service catalog using API instead of configuring it in conf file."},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"For example, the ``catalog_info \u003d volume:cinder:publicURL`` in nova.conf"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7ffa3b31_a90ec8e2","line":62,"updated":"2017-04-14 21:02:13.000000000","message":"s/providers simple/provides a simple/","commit_id":"2a57986851e8b8ee335a480969d7d1183e076856"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"5b601712ae2ce1356a609ab7bcf0091e52a2b77e","unresolved":false,"context_lines":[{"line_number":60,"context_line":"   url \u003d http://127.0.0.1:9696"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"Keystoneauth provides simple and consistent way to get API endpoints from"},{"line_number":63,"context_line":"service catalog using API instead of configuring it in conf file."},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"For example, the ``catalog_info \u003d volume:cinder:publicURL`` in nova.conf"},{"line_number":66,"context_line":"is a configuration setting to set the info to match when looking for cinder"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7ffa3b31_89130c0e","line":63,"updated":"2017-04-14 21:02:13.000000000","message":"s/service catalog/the service catalog/","commit_id":"2a57986851e8b8ee335a480969d7d1183e076856"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"5b601712ae2ce1356a609ab7bcf0091e52a2b77e","unresolved":false,"context_lines":[{"line_number":79,"context_line":"   * ``service_type``: Service type of the endpoint."},{"line_number":80,"context_line":"   * ``interface``: Type of endpoint, e.g. public, admin, internal."},{"line_number":81,"context_line":"   * ``region_name``: Region of the endpoint."},{"line_number":82,"context_line":"   * ``service_name``: The assigned name of the service."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"If this method fails to get API endpoint from service catalog then, by default"},{"line_number":85,"context_line":"we will find it using ``endpoint_override`` configuration. That is, we will"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7ffa3b31_69f8c0c3","line":82,"updated":"2017-04-14 21:02:13.000000000","message":"Remove this. Only service_type should ever be used, period.","commit_id":"2a57986851e8b8ee335a480969d7d1183e076856"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"5b601712ae2ce1356a609ab7bcf0091e52a2b77e","unresolved":false,"context_lines":[{"line_number":82,"context_line":"   * ``service_name``: The assigned name of the service."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"If this method fails to get API endpoint from service catalog then, by default"},{"line_number":85,"context_line":"we will find it using ``endpoint_override`` configuration. That is, we will"},{"line_number":86,"context_line":"override service catalog lookup and get required URL from configuration."},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"By adding this change, we will have consistent way to connect with other"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7ffa3b31_49f504cb","line":85,"updated":"2017-04-14 21:02:13.000000000","message":"Wait, so we\u0027re adding yet another override configuration option for each service? I thought we wanted to get rid of those options?\n\nWhy not just use the existing configuration options like [glance]api_servers and [neutron]url as the override options? If those exist, they override the ksauth service catalog lookup, which will provide perfect backwards compatibility for upgrade operations. The operator would then set those override options to \u0027\u0027 (or remove the option from the configuration file entirely).","commit_id":"2a57986851e8b8ee335a480969d7d1183e076856"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"1174a80b051225fe0dae81d4f0fa564d6e1bfb03","unresolved":false,"context_lines":[{"line_number":82,"context_line":"   * ``service_name``: The assigned name of the service."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"If this method fails to get API endpoint from service catalog then, by default"},{"line_number":85,"context_line":"we will find it using ``endpoint_override`` configuration. That is, we will"},{"line_number":86,"context_line":"override service catalog lookup and get required URL from configuration."},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"By adding this change, we will have consistent way to connect with other"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7ffa3b31_49918452","line":85,"in_reply_to":"7ffa3b31_49f504cb","updated":"2017-04-14 21:11:32.000000000","message":"As noted in PS1, I thought KSA provided the override URL option, not nova. However, looking at the [ironic] config options in our sample conf file it doesn\u0027t:\n\nhttps://docs.openstack.org/developer/nova/sample_config.html\n\nWe have a \"api_endpoint\" option for Ironic which is like endpoint_override. There was debate about whether or not we should remove that when we moved the internal ironic stuff to ksa though.\n\nAlso, I think once we use the service catalog for these things, we deprecate the old options for removal.\n\nFor [placement] we allow overriding the region and interface, so I think those are OK to leave in here for all others (cinder/glance/neutron).\n\nAnd unfortunately we need service_type because we have cases like cinderv2 and cinderv3 :(.\n\nBut yeah I\u0027m OK with not supporting a backdoor endpoint_override and also for deprecating the old options once we start using ksa.","commit_id":"2a57986851e8b8ee335a480969d7d1183e076856"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"49440fc2681345b4d5a7fe15af4963e8b3a3f285","unresolved":false,"context_lines":[{"line_number":25,"context_line":"Ideally, these API endpoints should not be hardcoded. We should have a simple"},{"line_number":26,"context_line":"and consistent way to retrieve these endpoint URLs in Nova."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"This spec focuses on using the Keystone service catalog to get API endpoint"},{"line_number":29,"context_line":"when Nova interacts with Cinder, Glance and Neutron."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"The service catalog is composed of a list of services which represent services"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7ffa3b31_6ccb2e53","line":28,"range":{"start_line":28,"start_character":67,"end_line":28,"end_character":75},"updated":"2017-04-14 23:20:24.000000000","message":"an endpoint / endpoints","commit_id":"b2ab72bceed0ade841340bd12cd704287b190b2d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"49440fc2681345b4d5a7fe15af4963e8b3a3f285","unresolved":false,"context_lines":[{"line_number":35,"context_line":"Use Cases"},{"line_number":36,"context_line":"---------"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Nova uses API endpoints to establish communication with other services like"},{"line_number":39,"context_line":"Cinder, Glance and Neutron. These endpoints are currently configured using"},{"line_number":40,"context_line":"options in the ``nova.conf`` configuration file. Ideally, rather than"},{"line_number":41,"context_line":"hardcoding these URL values in a configuration file, use the service catalog to"},{"line_number":42,"context_line":"provide discoverability for those."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Proposed change"},{"line_number":45,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7ffa3b31_4cc87254","line":42,"range":{"start_line":38,"start_character":0,"end_line":42,"end_character":34},"updated":"2017-04-14 23:20:24.000000000","message":"This isn\u0027t really a use case. Would be something more like:\n\n\"As an operator, I want to configure the service catalog and have my services use that to interact with each other so I don\u0027t have to individually manage inter-project connections separately.\"","commit_id":"b2ab72bceed0ade841340bd12cd704287b190b2d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"49440fc2681345b4d5a7fe15af4963e8b3a3f285","unresolved":false,"context_lines":[{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Currently, Nova uses configuration settings from ``nova.conf`` file to get"},{"line_number":53,"context_line":"endpoint URL. Each service has url param in ``nova.conf`` which represents"},{"line_number":54,"context_line":"service endpoint. For Example, in ``nova.conf``, there are many endpoints"},{"line_number":55,"context_line":"like::"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"   [glance]"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7ffa3b31_0cc26a34","line":54,"range":{"start_line":54,"start_character":22,"end_line":54,"end_character":29},"updated":"2017-04-14 23:20:24.000000000","message":"example","commit_id":"b2ab72bceed0ade841340bd12cd704287b190b2d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"49440fc2681345b4d5a7fe15af4963e8b3a3f285","unresolved":false,"context_lines":[{"line_number":73,"context_line":"communication with any other service, Nova will call this method to find API"},{"line_number":74,"context_line":"endpoints."},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"The method will first look at the existing configuration options such as"},{"line_number":77,"context_line":"``[glance]api_servers`` and use these options if they exist in the"},{"line_number":78,"context_line":"configuration file. This is to ensure backwards compatibility and a smooth"},{"line_number":79,"context_line":"upgrade experience. If the existing configuration option is *not* found or has"},{"line_number":80,"context_line":"no value, then the method will look up a specified API endpoint from the"},{"line_number":81,"context_line":"Keystone service catalog for a particular service type using ``keystoneauth``."},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"By adding this change, we will have consistent way to connect with other"},{"line_number":84,"context_line":"services from Nova."}],"source_content_type":"text/x-rst","patch_set":5,"id":"7ffa3b31_8ceb1aaf","line":81,"range":{"start_line":76,"start_character":0,"end_line":81,"end_character":78},"updated":"2017-04-14 23:20:24.000000000","message":"I\u0027m OK with the fallback for Pike, but I\u0027d like to say that we\u0027re going to deprecate the old options in favor of the standard keystoneauth options. That\u0027s for the upgrade experience. We\u0027ll drop the deprecated config options in Queens.","commit_id":"b2ab72bceed0ade841340bd12cd704287b190b2d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6d24983514485b79235af7a5241d3588d2e1a347","unresolved":false,"context_lines":[{"line_number":132,"context_line":"Implementation"},{"line_number":133,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"Assignee(s)"},{"line_number":136,"context_line":"-----------"},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"Primary assignee:"},{"line_number":139,"context_line":"  Pushkar Umaranikar (pumaranikar)"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"Other contributors:"},{"line_number":142,"context_line":"  Sarafraj Singh (raj_singh)"},{"line_number":143,"context_line":"  OSIC team"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"Work Items"},{"line_number":146,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7ffa3b31_06ea2574","line":143,"range":{"start_line":135,"start_character":0,"end_line":143,"end_character":11},"updated":"2017-04-17 18:18:50.000000000","message":"Is this true anymore with the OSIC news? I\u0027d be OK with approving the spec and putting this as TBD or something until we know who can/wants to work on it.","commit_id":"b2ab72bceed0ade841340bd12cd704287b190b2d"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"d2927aa0eab3f48fc62014c55b54a41aa88a0720","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"`\u003chttps://blueprints.launchpad.net/nova/+spec/use-service-catalog-for-endpoints\u003e`_"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Make use of service catalog to find endpoint URL\u0027s instead of reading"},{"line_number":14,"context_line":"from nova configuration file."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"7ffa3b31_fc48e9cc","line":13,"range":{"start_line":13,"start_character":45,"end_line":13,"end_character":50},"updated":"2017-04-17 21:35:10.000000000","message":"URLs","commit_id":"abc259da4bbc3c5a8ad6111d6f8a6931540cce0d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f932fac2196ff46c9770fecb30c6e4f5af359b08","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"`\u003chttps://blueprints.launchpad.net/nova/+spec/use-service-catalog-for-endpoints\u003e`_"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Make use of service catalog to find endpoint URL\u0027s instead of reading"},{"line_number":14,"context_line":"from nova configuration file."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"7ffa3b31_9f3af795","line":13,"range":{"start_line":13,"start_character":45,"end_line":13,"end_character":50},"in_reply_to":"7ffa3b31_fc48e9cc","updated":"2017-04-17 22:00:36.000000000","message":"Done","commit_id":"abc259da4bbc3c5a8ad6111d6f8a6931540cce0d"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"d2927aa0eab3f48fc62014c55b54a41aa88a0720","unresolved":false,"context_lines":[{"line_number":67,"context_line":"   \u003cservice_type\u003e:\u003cservice_name\u003e:\u003cendpoint_type\u003e"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"To make retrieving API endpoint consistent, we can add a new method"},{"line_number":70,"context_line":"``get_url_from_service_catalog_from_conf()`` in Nova. To estabilsh"},{"line_number":71,"context_line":"communication with any other service, Nova will call this method to find API"},{"line_number":72,"context_line":"endpoints."},{"line_number":73,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"7ffa3b31_bc2b81e9","line":70,"range":{"start_line":70,"start_character":2,"end_line":70,"end_character":40},"updated":"2017-04-17 21:35:10.000000000","message":"If I\u0027m understanding the functionality, `get_url_from_conf_or_service_catalog` might be more appropriate.  Or better yet, `get_service_url`, and hide the implementation details (doc them in comments/docstring) so that we can later deprecate the conf bits without changing this method signature.","commit_id":"abc259da4bbc3c5a8ad6111d6f8a6931540cce0d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f932fac2196ff46c9770fecb30c6e4f5af359b08","unresolved":false,"context_lines":[{"line_number":67,"context_line":"   \u003cservice_type\u003e:\u003cservice_name\u003e:\u003cendpoint_type\u003e"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"To make retrieving API endpoint consistent, we can add a new method"},{"line_number":70,"context_line":"``get_url_from_service_catalog_from_conf()`` in Nova. To estabilsh"},{"line_number":71,"context_line":"communication with any other service, Nova will call this method to find API"},{"line_number":72,"context_line":"endpoints."},{"line_number":73,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"7ffa3b31_7f416b0b","line":70,"range":{"start_line":70,"start_character":57,"end_line":70,"end_character":66},"updated":"2017-04-17 22:00:36.000000000","message":"establish","commit_id":"abc259da4bbc3c5a8ad6111d6f8a6931540cce0d"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"13c17492cf70555777922141e2fce65da8e7b9f1","unresolved":false,"context_lines":[{"line_number":67,"context_line":"   \u003cservice_type\u003e:\u003cservice_name\u003e:\u003cendpoint_type\u003e"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"To make retrieving API endpoint consistent, we can add a new method"},{"line_number":70,"context_line":"``get_url_from_service_catalog_from_conf()`` in Nova. To estabilsh"},{"line_number":71,"context_line":"communication with any other service, Nova will call this method to find API"},{"line_number":72,"context_line":"endpoints."},{"line_number":73,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"7ffa3b31_1cf5d583","line":70,"range":{"start_line":70,"start_character":2,"end_line":70,"end_character":40},"updated":"2017-04-17 21:34:19.000000000","message":"might be a better name for the above :) but whatevs, we can address that in the patch reviews.","commit_id":"abc259da4bbc3c5a8ad6111d6f8a6931540cce0d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f932fac2196ff46c9770fecb30c6e4f5af359b08","unresolved":false,"context_lines":[{"line_number":67,"context_line":"   \u003cservice_type\u003e:\u003cservice_name\u003e:\u003cendpoint_type\u003e"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"To make retrieving API endpoint consistent, we can add a new method"},{"line_number":70,"context_line":"``get_url_from_service_catalog_from_conf()`` in Nova. To estabilsh"},{"line_number":71,"context_line":"communication with any other service, Nova will call this method to find API"},{"line_number":72,"context_line":"endpoints."},{"line_number":73,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"7ffa3b31_bf37f3b0","line":70,"range":{"start_line":70,"start_character":2,"end_line":70,"end_character":40},"in_reply_to":"7ffa3b31_7cb85975","updated":"2017-04-17 22:00:36.000000000","message":"Done","commit_id":"abc259da4bbc3c5a8ad6111d6f8a6931540cce0d"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"7f07d92acde3b10dfaa69a23e2f3d682bfe005cd","unresolved":false,"context_lines":[{"line_number":67,"context_line":"   \u003cservice_type\u003e:\u003cservice_name\u003e:\u003cendpoint_type\u003e"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"To make retrieving API endpoint consistent, we can add a new method"},{"line_number":70,"context_line":"``get_url_from_service_catalog_from_conf()`` in Nova. To estabilsh"},{"line_number":71,"context_line":"communication with any other service, Nova will call this method to find API"},{"line_number":72,"context_line":"endpoints."},{"line_number":73,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"7ffa3b31_fc2109fd","line":70,"range":{"start_line":70,"start_character":2,"end_line":70,"end_character":40},"in_reply_to":"7ffa3b31_bc2b81e9","updated":"2017-04-17 21:36:04.000000000","message":"yeah, but meh, we can handle that in patch reviews. :)","commit_id":"abc259da4bbc3c5a8ad6111d6f8a6931540cce0d"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"fd29e7a28601938818355455b1bee3975a2036f1","unresolved":false,"context_lines":[{"line_number":67,"context_line":"   \u003cservice_type\u003e:\u003cservice_name\u003e:\u003cendpoint_type\u003e"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"To make retrieving API endpoint consistent, we can add a new method"},{"line_number":70,"context_line":"``get_url_from_service_catalog_from_conf()`` in Nova. To estabilsh"},{"line_number":71,"context_line":"communication with any other service, Nova will call this method to find API"},{"line_number":72,"context_line":"endpoints."},{"line_number":73,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"7ffa3b31_7cb85975","line":70,"range":{"start_line":70,"start_character":2,"end_line":70,"end_character":40},"in_reply_to":"7ffa3b31_fc2109fd","updated":"2017-04-17 21:41:55.000000000","message":"ight","commit_id":"abc259da4bbc3c5a8ad6111d6f8a6931540cce0d"}]}