)]}'
{"tempest/scenario/utils.py":[{"author":{"_account_id":8576,"name":"Yair Fried","email":"yfried@redhat.com","username":"yfried"},"change_message_id":"a0727fa51360b3558f36b95f7b3208c63e66f6ab","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        self.flavors_client \u003d os.flavors_client"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def ssh_user(self, image_id):"},{"line_number":48,"context_line":"        _, _image \u003d self.images_client.get_image(image_id)"},{"line_number":49,"context_line":"        for regex, user in self.ssh_users:"},{"line_number":50,"context_line":"            # First match wins"},{"line_number":51,"context_line":"            if re.match(regex, _image[\u0027name\u0027]) is not None:"}],"source_content_type":"text/x-python","patch_set":4,"id":"da9df570_f949b031","line":48,"updated":"2014-09-28 06:19:43.000000000","message":"nit: why \"_image\"?\nAFAIK, the \"_\" prefix indicates a variable that won\u0027t be used.","commit_id":"c2ab36ec2182a943be67e8edec200373a6c6aff7"},{"author":{"_account_id":8576,"name":"Yair Fried","email":"yfried@redhat.com","username":"yfried"},"change_message_id":"a0727fa51360b3558f36b95f7b3208c63e66f6ab","unresolved":false,"context_lines":[{"line_number":58,"context_line":"                             string\u003dstr(image[\u0027name\u0027]))"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    def is_sshable_image(self, image_id):"},{"line_number":61,"context_line":"        _, _image \u003d self.images_client.get_image(image_id)"},{"line_number":62,"context_line":"        return self._is_sshable_image(_image)"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    def _is_flavor_enough(self, flavor, image):"}],"source_content_type":"text/x-python","patch_set":4,"id":"da9df570_1953d4e2","line":61,"updated":"2014-09-28 06:19:43.000000000","message":"same here","commit_id":"c2ab36ec2182a943be67e8edec200373a6c6aff7"}]}