)]}'
{"shade/__init__.py":[{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"7fe05c748ee96fb5004103e2198fa755a95c1c7a","unresolved":false,"context_lines":[{"line_number":1237,"context_line":""},{"line_number":1238,"context_line":"    @property"},{"line_number":1239,"context_line":"    def auth_token(self):"},{"line_number":1240,"context_line":"        if self.auth_plugin in (None, \"None\", \u0027\u0027, \u0027ironic\u0027):"},{"line_number":1241,"context_line":"            return self._auth_token"},{"line_number":1242,"context_line":"        if not self._auth_token:"},{"line_number":1243,"context_line":"            self._auth_token \u003d self.keystone_session.get_token()"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_a3f7b1f4","line":1240,"updated":"2015-02-27 00:21:06.000000000","message":"I would prefer to remove \"ironic\" as an option here. Ironic does not have its own auth model, so this appears misleading to me.","commit_id":"6a7dbfbc9b68a060c5f849f1a47ff3dfcd443c6c"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"cc46985b13b383f2e59e5fd08d76e5ada55d46bb","unresolved":false,"context_lines":[{"line_number":1237,"context_line":""},{"line_number":1238,"context_line":"    @property"},{"line_number":1239,"context_line":"    def auth_token(self):"},{"line_number":1240,"context_line":"        if self.auth_plugin in (None, \"None\", \u0027\u0027, \u0027ironic\u0027):"},{"line_number":1241,"context_line":"            return self._auth_token"},{"line_number":1242,"context_line":"        if not self._auth_token:"},{"line_number":1243,"context_line":"            self._auth_token \u003d self.keystone_session.get_token()"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_f475ddfa","line":1240,"in_reply_to":"ba7be1f8_a3f7b1f4","updated":"2015-02-27 02:18:47.000000000","message":"In retrospect, I agree.  I\u0027ll change it in the next revision.","commit_id":"6a7dbfbc9b68a060c5f849f1a47ff3dfcd443c6c"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"7fe05c748ee96fb5004103e2198fa755a95c1c7a","unresolved":false,"context_lines":[{"line_number":1249,"context_line":"            ironic_logging \u003d logging.getLogger(\u0027ironicclient\u0027)"},{"line_number":1250,"context_line":"            ironic_logging.addHandler(logging.NullHandler())"},{"line_number":1251,"context_line":"            token \u003d self.auth_token"},{"line_number":1252,"context_line":"            if self.auth_plugin in (None, \"None\", \u0027\u0027, \u0027ironic\u0027):"},{"line_number":1253,"context_line":"                # TODO: This needs to be improved logic wise, perhaps a list,"},{"line_number":1254,"context_line":"                # or enhancement of the data stuctures with-in the library"},{"line_number":1255,"context_line":"                # to allow for things aside password authentication, or no"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_c3dbc57b","line":1252,"updated":"2015-02-27 00:21:06.000000000","message":"ditto","commit_id":"6a7dbfbc9b68a060c5f849f1a47ff3dfcd443c6c"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"cc46985b13b383f2e59e5fd08d76e5ada55d46bb","unresolved":false,"context_lines":[{"line_number":1249,"context_line":"            ironic_logging \u003d logging.getLogger(\u0027ironicclient\u0027)"},{"line_number":1250,"context_line":"            ironic_logging.addHandler(logging.NullHandler())"},{"line_number":1251,"context_line":"            token \u003d self.auth_token"},{"line_number":1252,"context_line":"            if self.auth_plugin in (None, \"None\", \u0027\u0027, \u0027ironic\u0027):"},{"line_number":1253,"context_line":"                # TODO: This needs to be improved logic wise, perhaps a list,"},{"line_number":1254,"context_line":"                # or enhancement of the data stuctures with-in the library"},{"line_number":1255,"context_line":"                # to allow for things aside password authentication, or no"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_14735116","line":1252,"in_reply_to":"ba7be1f8_c3dbc57b","updated":"2015-02-27 02:18:47.000000000","message":"Done","commit_id":"6a7dbfbc9b68a060c5f849f1a47ff3dfcd443c6c"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"657f3c2ccf542d111ff5d754fead7156156783e3","unresolved":false,"context_lines":[{"line_number":1256,"context_line":"                # authentiation if so desired by the user."},{"line_number":1257,"context_line":"                #"},{"line_number":1258,"context_line":"                # Attempt to utilize what is stored in auth as an endpoint URL"},{"line_number":1259,"context_line":"                endpoint \u003d self.auth"},{"line_number":1260,"context_line":"            else:"},{"line_number":1261,"context_line":"                endpoint \u003d self.get_endpoint(service_type\u003d\u0027baremetal\u0027)"},{"line_number":1262,"context_line":"            try:"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba7be1f8_be624d14","line":1259,"updated":"2015-02-27 13:34:40.000000000","message":"I think this should be:\nendpoint \u003d self.auth[\u0027endpoint\u0027]\nor something. That auth was not coming across as a dict is ... disturbing...","commit_id":"4d883823403605ac4e45f82d97c2eafa04d82ce1"}]}