)]}'
{"ironic_inspector/common/ironic.py":[{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"ba403de0c99ed15bfcb8095d26335ba7eef47832","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    else:"},{"line_number":104,"context_line":"        args[\u0027token\u0027] \u003d token"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    return client.Client(1, **args)"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"def check_provision_state(node):"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f79a3b5_e3188784","line":106,"range":{"start_line":106,"start_character":28,"end_line":106,"end_character":34},"updated":"2018-10-18 03:02:56.000000000","message":"I am not very familiar with keystone, but it looks like we need to pass an endpoint here?","commit_id":"23ac3902e9eb8ed2e3aeeaddffc1d89d5caba6d1"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"c0cba120a1dc66e0ffd56080707592f585e892aa","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    else:"},{"line_number":104,"context_line":"        args[\u0027token\u0027] \u003d token"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    return client.Client(1, **args)"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"def check_provision_state(node):"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f79a3b5_988ed978","line":106,"range":{"start_line":106,"start_character":28,"end_line":106,"end_character":34},"in_reply_to":"3f79a3b5_3816a54c","updated":"2018-10-18 08:57:35.000000000","message":"Let us see if the new variant passes","commit_id":"23ac3902e9eb8ed2e3aeeaddffc1d89d5caba6d1"},{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"6dda76e17e203604474df656707a9d084110b439","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    else:"},{"line_number":104,"context_line":"        args[\u0027token\u0027] \u003d token"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    return client.Client(1, **args)"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"def check_provision_state(node):"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f79a3b5_3816a54c","line":106,"range":{"start_line":106,"start_character":28,"end_line":106,"end_character":34},"in_reply_to":"3f79a3b5_526a5f82","updated":"2018-10-18 08:51:45.000000000","message":"Judging by current client code, it still needs an endpoint_override in the args","commit_id":"23ac3902e9eb8ed2e3aeeaddffc1d89d5caba6d1"},{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"abab395fb1ebaffd3dd5fe016682684e1b3b2da0","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    else:"},{"line_number":104,"context_line":"        args[\u0027token\u0027] \u003d token"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    return client.Client(1, **args)"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"def check_provision_state(node):"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f79a3b5_98efb9f9","line":106,"range":{"start_line":106,"start_character":28,"end_line":106,"end_character":34},"in_reply_to":"3f79a3b5_988ed978","updated":"2018-10-18 09:03:28.000000000","message":"buy me a beer if it won\u0027t, I have prepared my cup :D","commit_id":"23ac3902e9eb8ed2e3aeeaddffc1d89d5caba6d1"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"5495603336c9a652db407c5221bf208127a7fae9","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    else:"},{"line_number":104,"context_line":"        args[\u0027token\u0027] \u003d token"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    return client.Client(1, **args)"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"def check_provision_state(node):"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f79a3b5_d8269168","line":106,"range":{"start_line":106,"start_character":28,"end_line":106,"end_character":34},"in_reply_to":"3f79a3b5_98efb9f9","updated":"2018-10-18 09:04:41.000000000","message":"Come to Berlin in November, I\u0027ll get you one regardless :)","commit_id":"23ac3902e9eb8ed2e3aeeaddffc1d89d5caba6d1"},{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"cdf44a1f62e91885ba15e66d61c8774c7d092c89","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    else:"},{"line_number":104,"context_line":"        args[\u0027token\u0027] \u003d token"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    return client.Client(1, **args)"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"def check_provision_state(node):"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f79a3b5_445a88f5","line":106,"range":{"start_line":106,"start_character":28,"end_line":106,"end_character":34},"in_reply_to":"3f79a3b5_d8269168","updated":"2018-10-19 02:51:53.000000000","message":"Hope I have this opportunity :)\n\nAnyway, I don\u0027t know why the job is stuck, and it also disappeared in the zuul.openstack.org. I didn\u0027t aware the L106 is change to get_client, so indeed there is logic to get endpoint from identity there.\n\nFYI the job was failing last time I checked, the log reports catalog is missing. Maybe still something is missing there, those keystone options changed over and over every cycle, and keeps me confused *facepalm*","commit_id":"23ac3902e9eb8ed2e3aeeaddffc1d89d5caba6d1"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"01747897d0f1aecfeff30fe565f194d65876a807","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    else:"},{"line_number":104,"context_line":"        args[\u0027token\u0027] \u003d token"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    return client.Client(1, **args)"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"def check_provision_state(node):"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f79a3b5_526a5f82","line":106,"range":{"start_line":106,"start_character":28,"end_line":106,"end_character":34},"in_reply_to":"3f79a3b5_e3188784","updated":"2018-10-18 08:15:33.000000000","message":"It should not be required, ironicclient must be able to fetch it from the catalog. I need to look why it fails.","commit_id":"23ac3902e9eb8ed2e3aeeaddffc1d89d5caba6d1"}]}