)]}'
{"ironic_python_agent/cmd/agent.py":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"47f7b2e9d2307c7efb3ef3f4962e3eed96d31321","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    \"\"\"Entrypoint for IronicPythonAgent.\"\"\""},{"line_number":31,"context_line":"    # NOTE(dtantsur): this must happen very early of the files from"},{"line_number":32,"context_line":"    # /etc/ironic-python-agent.d won\u0027t be loaded"},{"line_number":33,"context_line":"    utils.copy_config_from_vmedia()"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    log.register_options(CONF)"},{"line_number":36,"context_line":"    CONF(args\u003dsys.argv[1:])"}],"source_content_type":"text/x-python","patch_set":4,"id":"fffc6b78_a3ba22db","line":33,"updated":"2020-11-19 19:20:38.000000000","message":"Makes sense, this is awfully reminiscent of the feature to load config from floppy devices, but off hand I don\u0027t remember where that code is anymore.","commit_id":"ccf44f369c30bfb981e55fdcedac1cb1cdfdc609"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"212e74651313b159543c4838dcbeb42dc6a0a638","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    \"\"\"Entrypoint for IronicPythonAgent.\"\"\""},{"line_number":31,"context_line":"    # NOTE(dtantsur): this must happen very early of the files from"},{"line_number":32,"context_line":"    # /etc/ironic-python-agent.d won\u0027t be loaded"},{"line_number":33,"context_line":"    utils.copy_config_from_vmedia()"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    log.register_options(CONF)"},{"line_number":36,"context_line":"    CONF(args\u003dsys.argv[1:])"}],"source_content_type":"text/x-python","patch_set":4,"id":"fffc6b78_83a0bea7","line":33,"in_reply_to":"fffc6b78_a3ba22db","updated":"2020-11-19 19:22:05.000000000","message":"utils.py, just above the code I\u0027m adding. I\u0027ve reused quite a few bits from it.","commit_id":"ccf44f369c30bfb981e55fdcedac1cb1cdfdc609"}],"ironic_python_agent/utils.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"61fac2e0ea6ab198e148b1090cab2bd8bb3ff662","unresolved":false,"context_lines":[{"line_number":169,"context_line":""},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"def _find_device_by_labels(labels):"},{"line_number":172,"context_line":"    for label in labels + [l.upper() for l in labels]:"},{"line_number":173,"context_line":"        path \u003d os.path.join(\u0027/dev/disk/by-label\u0027, label)"},{"line_number":174,"context_line":"        if os.path.exists(path):"},{"line_number":175,"context_line":"            return path"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f621f24_e1a0ad94","line":172,"updated":"2020-11-18 17:58:24.000000000","message":"pep8: E741 ambiguous variable name \u0027l\u0027","commit_id":"dfa0d1bef56ac6469afd403c72eb19c705a9c898"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"c1e3a38e5a5794d01681f09bff031bf6a115f768","unresolved":false,"context_lines":[{"line_number":170,"context_line":""},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"def _find_device_by_labels(labels):"},{"line_number":173,"context_line":"    for label in labels + [l.upper() for l in labels]:"},{"line_number":174,"context_line":"        path \u003d os.path.join(\u0027/dev/disk/by-label\u0027, label)"},{"line_number":175,"context_line":"        if os.path.exists(path):"},{"line_number":176,"context_line":"            return path"}],"source_content_type":"text/x-python","patch_set":2,"id":"fffc6b78_f2035d0e","line":173,"updated":"2020-11-19 13:30:25.000000000","message":"pep8: E741 ambiguous variable name \u0027l\u0027","commit_id":"0e45c106f780a531e6c6c75a343e4bf46aa234f8"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"25a6e53d01d4dc415de9073cd197c93d7608dd6a","unresolved":false,"context_lines":[{"line_number":161,"context_line":"        try:"},{"line_number":162,"context_line":"            execute(\"umount\", dest)"},{"line_number":163,"context_line":"        except processutils.ProcessExecutionError:"},{"line_number":164,"context_line":"            pass"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"        try:"},{"line_number":167,"context_line":"            shutil.rmtree(dest)"}],"source_content_type":"text/x-python","patch_set":4,"id":"fffc6b78_43c9b676","line":164,"updated":"2020-11-19 19:22:46.000000000","message":"++","commit_id":"ccf44f369c30bfb981e55fdcedac1cb1cdfdc609"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"25a6e53d01d4dc415de9073cd197c93d7608dd6a","unresolved":false,"context_lines":[{"line_number":237,"context_line":"                    src_file \u003d os.path.join(src, name)"},{"line_number":238,"context_line":"                    dst_file \u003d os.path.join(dest, name)"},{"line_number":239,"context_line":"                    shutil.copy(src_file, dst_file)"},{"line_number":240,"context_line":"            except EnvironmentError as exc:"},{"line_number":241,"context_line":"                msg \u003d (\"Unable to copy vmedia configuration %s to %s: %s\""},{"line_number":242,"context_line":"                       % (src, dest, exc))"},{"line_number":243,"context_line":"                raise errors.VirtualMediaBootError(msg)"}],"source_content_type":"text/x-python","patch_set":4,"id":"fffc6b78_0394ce43","line":240,"updated":"2020-11-19 19:22:46.000000000","message":"I might be wrong,  but I think more than EnvironmentError may be raised here... although the odds of that seem unliekly.","commit_id":"ccf44f369c30bfb981e55fdcedac1cb1cdfdc609"}]}