)]}'
{"container_config_scripts/nova_wait_for_api_service.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"1c4cc5069667aff8494e5cc6193e16511b527082","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":14,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":15,"context_line":"# under the License."},{"line_number":16,"context_line":"import logging"},{"line_number":17,"context_line":"import argparse"},{"line_number":18,"context_line":"import os"},{"line_number":19,"context_line":"import six"},{"line_number":20,"context_line":"import sys"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_85161cf2","line":17,"range":{"start_line":16,"start_character":0,"end_line":17,"end_character":15},"updated":"2020-01-18 13:03:49.000000000","message":"pep8 test is failing because this causes un-alphabetical import order.","commit_id":"8c4fb06ff5979d362c1cfcbf85e98912f80d5faf"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"2bdbc86b7c2bfaf038c6ee16e39c299921c6df41","unresolved":false,"context_lines":[{"line_number":56,"context_line":"if __name__ \u003d\u003d \u0027__main__\u0027:"},{"line_number":57,"context_line":"    parser \u003d argparse.ArgumentParser(usage\u003d\u0027%(prog)s [options]\u0027)"},{"line_number":58,"context_line":"    parser.add_argument(\u0027-k\u0027, \u0027--insecure\u0027,"},{"line_number":59,"context_line":"                      action\u003d\"store_false\","},{"line_number":60,"context_line":"                      dest\u003d\u0027insecure\u0027,"},{"line_number":61,"context_line":"                      default\u003dTrue,"},{"line_number":62,"context_line":"                      help\u003d\u0027Allow insecure connection when using SSL\u0027)"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    args \u003d parser.parse_args()"},{"line_number":65,"context_line":"    LOG.debug(\u0027Running with parameter insecure \u003d %s\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_9a675653","line":62,"range":{"start_line":59,"start_character":0,"end_line":62,"end_character":70},"updated":"2020-01-19 02:39:09.000000000","message":"Can you fix the indents for these lines so that these lines start at the same position as \u0027-k\u0027 in the above line ?\n\n    parser.add_argument(\u0027-k\u0027, \u0027--insecure\u0027,\n                        action\u003d\"store_false\",\n                        ...","commit_id":"56ffccb8e06dae8dfd17654578aaba408286fe35"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"c57845365db24d2d50eaf68d8baf42eff71509c5","unresolved":false,"context_lines":[{"line_number":56,"context_line":"if __name__ \u003d\u003d \u0027__main__\u0027:"},{"line_number":57,"context_line":"    parser \u003d argparse.ArgumentParser(usage\u003d\u0027%(prog)s [options]\u0027)"},{"line_number":58,"context_line":"    parser.add_argument(\u0027-k\u0027, \u0027--insecure\u0027,"},{"line_number":59,"context_line":"                      action\u003d\"store_false\","},{"line_number":60,"context_line":"                      dest\u003d\u0027insecure\u0027,"},{"line_number":61,"context_line":"                      default\u003dTrue,"},{"line_number":62,"context_line":"                      help\u003d\u0027Allow insecure connection when using SSL\u0027)"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    args \u003d parser.parse_args()"},{"line_number":65,"context_line":"    LOG.debug(\u0027Running with parameter insecure \u003d %s\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_ed8016c1","line":62,"range":{"start_line":59,"start_character":0,"end_line":62,"end_character":70},"in_reply_to":"3fa7e38b_9a675653","updated":"2020-01-20 06:09:56.000000000","message":"Done","commit_id":"56ffccb8e06dae8dfd17654578aaba408286fe35"}],"container_config_scripts/nova_wait_for_compute_service.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"2bdbc86b7c2bfaf038c6ee16e39c299921c6df41","unresolved":false,"context_lines":[{"line_number":56,"context_line":"if __name__ \u003d\u003d \u0027__main__\u0027:"},{"line_number":57,"context_line":"    parser \u003d argparse.ArgumentParser(usage\u003d\u0027%(prog)s [options]\u0027)"},{"line_number":58,"context_line":"    parser.add_argument(\u0027-k\u0027, \u0027--insecure\u0027,"},{"line_number":59,"context_line":"                      action\u003d\"store_false\","},{"line_number":60,"context_line":"                      dest\u003d\u0027insecure\u0027,"},{"line_number":61,"context_line":"                      default\u003dTrue,"},{"line_number":62,"context_line":"                      help\u003d\u0027Allow insecure connection when using SSL\u0027)"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    args \u003d parser.parse_args()"},{"line_number":65,"context_line":"    LOG.debug(\u0027Running with parameter insecure \u003d %s\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_7a6cda32","line":62,"range":{"start_line":59,"start_character":22,"end_line":62,"end_character":70},"updated":"2020-01-19 02:39:09.000000000","message":"ditto","commit_id":"56ffccb8e06dae8dfd17654578aaba408286fe35"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"c57845365db24d2d50eaf68d8baf42eff71509c5","unresolved":false,"context_lines":[{"line_number":56,"context_line":"if __name__ \u003d\u003d \u0027__main__\u0027:"},{"line_number":57,"context_line":"    parser \u003d argparse.ArgumentParser(usage\u003d\u0027%(prog)s [options]\u0027)"},{"line_number":58,"context_line":"    parser.add_argument(\u0027-k\u0027, \u0027--insecure\u0027,"},{"line_number":59,"context_line":"                      action\u003d\"store_false\","},{"line_number":60,"context_line":"                      dest\u003d\u0027insecure\u0027,"},{"line_number":61,"context_line":"                      default\u003dTrue,"},{"line_number":62,"context_line":"                      help\u003d\u0027Allow insecure connection when using SSL\u0027)"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    args \u003d parser.parse_args()"},{"line_number":65,"context_line":"    LOG.debug(\u0027Running with parameter insecure \u003d %s\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_8d672253","line":62,"range":{"start_line":59,"start_character":22,"end_line":62,"end_character":70},"in_reply_to":"3fa7e38b_7a6cda32","updated":"2020-01-20 06:09:56.000000000","message":"Done","commit_id":"56ffccb8e06dae8dfd17654578aaba408286fe35"}],"network/endpoints/build_endpoint_map.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"80f4e75236e9de53d3551846f7c4cd9d68b6fbc2","unresolved":false,"context_lines":[{"line_number":288,"context_line":""},{"line_number":289,"context_line":""},{"line_number":290,"context_line":"def main():"},{"line_number":291,"context_line":"    options, args \u003d get_options()"},{"line_number":292,"context_line":"    if args:"},{"line_number":293,"context_line":"        print(\u0027Warning: ignoring positional args: %s\u0027 % \u0027 \u0027.join(args),"},{"line_number":294,"context_line":"              file\u003dsys.stderr)"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"    try:"},{"line_number":297,"context_line":"        if options.check:"}],"source_content_type":"text/x-python","patch_set":4,"id":"3fa7e38b_286e032d","side":"PARENT","line":294,"range":{"start_line":291,"start_character":4,"end_line":294,"end_character":30},"updated":"2020-01-21 11:32:04.000000000","message":"This change makes build_endpoint_map.py fail if we have additional args, while currently it doesn\u0027t but shows warn log.\n\nTo me that \"hard fail\" makes more sense than only logging warn because theoretically we don\u0027t expect to have unexpected args passed here, so I agree with the change in behavior.","commit_id":"fe56b682c3442f945f170cc35397e96539ceca18"}]}