)]}'
{"shade/openstackcloud.py":[{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"064c2ae5c26e09276134f48955cf79d513a3e619","unresolved":false,"context_lines":[{"line_number":446,"context_line":"        2.4) If config_version is specified and the endpoint in the adapter is"},{"line_number":447,"context_line":"        versioned, remove the last part from the versioned endpoint and do the"},{"line_number":448,"context_line":"        discovery again. Pick the URL from the discovery document that matches"},{"line_number":449,"context_line":"        the config_version. If it does not exist, pick the latest version from"},{"line_number":450,"context_line":"        the discovery document."},{"line_number":451,"context_line":""},{"line_number":452,"context_line":"        \"\"\""},{"line_number":453,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ffa3b31_1a8ece2d","line":450,"range":{"start_line":449,"start_character":50,"end_line":450,"end_character":31},"updated":"2017-04-19 01:32:00.000000000","message":"Why not just do as 2.2) and use the versioned endpoint from the client adapter?\n\nIf we want to keep with this, shouldn\u0027t we follow the same approach there then?","commit_id":"d5d78e9e980e61f6ca93199de3513c026ea743fa"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"3cc523b55b5066db0e824bd212f134267f201e6d","unresolved":false,"context_lines":[{"line_number":446,"context_line":"        2.4) If config_version is specified and the endpoint in the adapter is"},{"line_number":447,"context_line":"        versioned, remove the last part from the versioned endpoint and do the"},{"line_number":448,"context_line":"        discovery again. Pick the URL from the discovery document that matches"},{"line_number":449,"context_line":"        the config_version. If it does not exist, pick the latest version from"},{"line_number":450,"context_line":"        the discovery document."},{"line_number":451,"context_line":""},{"line_number":452,"context_line":"        \"\"\""},{"line_number":453,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ffa3b31_a88662da","line":450,"range":{"start_line":449,"start_character":50,"end_line":450,"end_character":31},"in_reply_to":"7ffa3b31_1a8ece2d","updated":"2017-04-19 12:01:35.000000000","message":"I think there are two cases:\n\n\u0027If the config version is specified and the endpoint in the adapter is versioned and it matches, we\u0027re good\u0027\n\n\u0027If the config version is specified and the endpoint in the adapter is versioned and it doesn\u0027t match, then go through discovery.\u0027\n\nI agree though, and I think you\u0027re right. In that second case, rather than picking a different versioned endpoint, we should just use the catalog entry.","commit_id":"d5d78e9e980e61f6ca93199de3513c026ea743fa"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"106cb21a1830bc6ac3cbe9609f93b902911b0b08","unresolved":false,"context_lines":[{"line_number":442,"context_line":"        4) If config_version is specified and the endpoint in the adapter is"},{"line_number":443,"context_line":"        versioned, remove the last part from the versioned endpoint and do the"},{"line_number":444,"context_line":"        discovery again. Pick the URL from the discovery document that matches"},{"line_number":445,"context_line":"        the config_version. If it does not exist, warn the user and pick the"},{"line_number":446,"context_line":"        latest version from the discovery document."},{"line_number":447,"context_line":""},{"line_number":448,"context_line":"        \"\"\""},{"line_number":449,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":3,"id":"7ffa3b31_75184374","line":446,"range":{"start_line":445,"start_character":68,"end_line":446,"end_character":51},"updated":"2017-04-19 01:59:37.000000000","message":"Same question still stands: Why not just do as 2) and use the versioned endpoint from the client adapter?\n\nIf we want to keep with this, shouldn\u0027t we follow the same approach there then?","commit_id":"6228e1278cb6d7fb4a85d43830899a081e555287"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"d57c19d97489930f5fc38fce42ec42cb9bb497ae","unresolved":false,"context_lines":[{"line_number":442,"context_line":"        4) If config_version is specified and the endpoint in the adapter is"},{"line_number":443,"context_line":"        versioned, remove the last part from the versioned endpoint and do the"},{"line_number":444,"context_line":"        discovery again. Pick the URL from the discovery document that matches"},{"line_number":445,"context_line":"        the config_version. If it does not exist, warn the user and pick the"},{"line_number":446,"context_line":"        latest version from the discovery document."},{"line_number":447,"context_line":""},{"line_number":448,"context_line":"        \"\"\""},{"line_number":449,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":3,"id":"7ffa3b31_fcdadb48","line":446,"range":{"start_line":445,"start_character":68,"end_line":446,"end_character":51},"in_reply_to":"7ffa3b31_284b72b0","updated":"2017-04-19 23:34:46.000000000","message":"Agreed! However as discussed in IRC let\u0027s proceed with implementing this in ksa with Jamie.","commit_id":"6228e1278cb6d7fb4a85d43830899a081e555287"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"460fae84b58fd9120d80cea21167313361e9c491","unresolved":false,"context_lines":[{"line_number":442,"context_line":"        4) If config_version is specified and the endpoint in the adapter is"},{"line_number":443,"context_line":"        versioned, remove the last part from the versioned endpoint and do the"},{"line_number":444,"context_line":"        discovery again. Pick the URL from the discovery document that matches"},{"line_number":445,"context_line":"        the config_version. If it does not exist, warn the user and pick the"},{"line_number":446,"context_line":"        latest version from the discovery document."},{"line_number":447,"context_line":""},{"line_number":448,"context_line":"        \"\"\""},{"line_number":449,"context_line":"        pass"}],"source_content_type":"text/x-python","patch_set":3,"id":"7ffa3b31_284b72b0","line":446,"range":{"start_line":445,"start_character":68,"end_line":446,"end_character":51},"in_reply_to":"7ffa3b31_75184374","updated":"2017-04-19 12:22:49.000000000","message":"Whoops, replied to PS1 before:\n\nI think there are two cases:\n\n\u0027If the config version is specified and the endpoint in the adapter is versioned and it matches, we\u0027re good\u0027\n\n\u0027If the config version is specified and the endpoint in the adapter is versioned and it doesn\u0027t match, then go through discovery.\u0027\n\nI agree though, and I think you\u0027re right. In that second case, rather than picking a different versioned endpoint, we should just use the catalog entry.","commit_id":"6228e1278cb6d7fb4a85d43830899a081e555287"}]}