)]}'
{"refstack_client/refstack_client.py":[{"author":{"_account_id":9377,"name":"Sergey Slipushenko","email":"slipushenko@gmail.com","username":"sslypushenko"},"change_message_id":"f808a2a77288a6ff3a07e2d2d0640bc37fc6a56f","unresolved":false,"context_lines":[{"line_number":219,"context_line":"                                     \u0027\u0027 % auth_version)"},{"line_number":220,"context_line":"            # If a Key or Index Error was raised, one of the expected keys or"},{"line_number":221,"context_line":"            # indices for retrieving the identity service ID was not found."},{"line_number":222,"context_line":"            except (KeyError, IndexError) as e:"},{"line_number":223,"context_line":"                raise RuntimeError(\u0027Unable to retrieve CPID from Keystone %s \u0027"},{"line_number":224,"context_line":"                                   \u0027catalog. The catalog or the identity \u0027"},{"line_number":225,"context_line":"                                   \u0027service endpoint was not \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba8a016a_7c16c9f2","line":222,"updated":"2015-11-17 11:13:07.000000000","message":"It looks like over complication. Lets do it simpler\n\nfor v3:\nfor service in token[\u0027catalog\u0027]:\n   if service[\u0027type\u0027] \u003d\u003d \u0027identity\u0027 and service[\u0027id\u0027]:\n      return service[\u0027id\u0027]\nelse:\n  msg \u003d \u0027Unable to retrieve CPID from Keystone %s \u0027...\n  self.logger.error(msg)\n  RuntimeError(msg)\nand same for v2","commit_id":"e04a06b106c8f093add77faee8c5ea544160a45e"},{"author":{"_account_id":9626,"name":"Paul Van Eck","email":"pvaneck@us.ibm.com","username":"pvaneck"},"change_message_id":"adcd8a1aaa3925323981353afc2da51226644cf6","unresolved":false,"context_lines":[{"line_number":219,"context_line":"                                     \u0027\u0027 % auth_version)"},{"line_number":220,"context_line":"            # If a Key or Index Error was raised, one of the expected keys or"},{"line_number":221,"context_line":"            # indices for retrieving the identity service ID was not found."},{"line_number":222,"context_line":"            except (KeyError, IndexError) as e:"},{"line_number":223,"context_line":"                raise RuntimeError(\u0027Unable to retrieve CPID from Keystone %s \u0027"},{"line_number":224,"context_line":"                                   \u0027catalog. The catalog or the identity \u0027"},{"line_number":225,"context_line":"                                   \u0027service endpoint was not \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba8a016a_d893214e","line":222,"in_reply_to":"ba8a016a_7c16c9f2","updated":"2015-11-17 22:38:49.000000000","message":"My reasoning for using try/excepts is for preparation for initiating the fallback mechanism for generating a cpid. If for whatever reason the \u0027catalog\u0027 or \u0027serviceCatalog\u0027 key does not exist or it is in some format we do not expect, then we want to be able to initiate the fallback cpid mechanism instead of having the script terminate with a key or index error.\n\nThere was some discussion of just using a hash of the endpoint URL for the cpid if it can\u0027t be retrieved from keystone.","commit_id":"e04a06b106c8f093add77faee8c5ea544160a45e"},{"author":{"_account_id":9377,"name":"Sergey Slipushenko","email":"slipushenko@gmail.com","username":"sslypushenko"},"change_message_id":"aedf02aa98e58766644fb0c6f8ff1f0e8bed6d8b","unresolved":false,"context_lines":[{"line_number":219,"context_line":"                                     \u0027\u0027 % auth_version)"},{"line_number":220,"context_line":"            # If a Key or Index Error was raised, one of the expected keys or"},{"line_number":221,"context_line":"            # indices for retrieving the identity service ID was not found."},{"line_number":222,"context_line":"            except (KeyError, IndexError) as e:"},{"line_number":223,"context_line":"                raise RuntimeError(\u0027Unable to retrieve CPID from Keystone %s \u0027"},{"line_number":224,"context_line":"                                   \u0027catalog. The catalog or the identity \u0027"},{"line_number":225,"context_line":"                                   \u0027service endpoint was not \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba8a016a_7ae1c2fd","line":222,"in_reply_to":"ba8a016a_d893214e","updated":"2015-11-17 23:54:51.000000000","message":"I don\u0027t see any kind of fallback here and don\u0027t really get a point how nested try/except will help to do fallback. I don\u0027t want to block things and don\u0027t have a change to provide my version.\nLet it be, so.","commit_id":"e04a06b106c8f093add77faee8c5ea544160a45e"}]}