)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"6a62b9768aa578756e43bf2b4c16802d82c2a12f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"3ec1e44d_fa849f02","updated":"2022-01-27 17:08:04.000000000","message":"Doesn\u0027t this need a line in projects.yaml so that the new job gets run, even if in the experimental queue?","commit_id":"e88032e156981dc7d935fbe7aafa41884fe7d433"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"d127bc6b5ea154631b27d62fc960bbe7be04612f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"be7bd4f0_7c93125e","in_reply_to":"3ec1e44d_fa849f02","updated":"2022-01-27 17:13:17.000000000","message":"Yeah you\u0027re right, I\u0027ll add it as a non-voting job (it won\u0027t work because it requires the octavia patches to be merged first)","commit_id":"e88032e156981dc7d935fbe7aafa41884fe7d433"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"afdec196200428a903de7a7dd5a50e790278e022","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"995b757f_bfbb527e","updated":"2022-02-01 15:34:05.000000000","message":"This new job is failing due to missing pip. Is there a required devstack patch for this?","commit_id":"fbf51a7633cbd68d9bb6fcec403362b28c31376b"},{"author":{"_account_id":32458,"name":"Ananya","display_name":"Ananya Banerjee","email":"anbanerj@redhat.com","username":"frenzyfriday"},"change_message_id":"c9e7b3ec6bc0cf2c81cdb315197cf8f27b40c6ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"44fcc32c_68919c80","updated":"2022-01-31 12:34:39.000000000","message":"testproject passed: https://code.engineering.redhat.com/gerrit/c/testproject/+/307244","commit_id":"fbf51a7633cbd68d9bb6fcec403362b28c31376b"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"cb9822dfb9c9606eb87a5942d7efb73f64fccfbe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"8a87cf73_b831a820","in_reply_to":"995b757f_bfbb527e","updated":"2022-02-01 16:37:08.000000000","message":"sorry about that, there\u0027s a required patch in octavia (the octavia patched depended on this one, I swapped the depends-on)","commit_id":"fbf51a7633cbd68d9bb6fcec403362b28c31376b"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"58f7b502088e0adf6ce74ca87ad0f69c9f4e975b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"c67a3cf0_fa9fb659","updated":"2022-02-02 17:36:22.000000000","message":"The C9s job passed, so LGTM","commit_id":"8ba04aa262ac9ca785359b5e63ab6ec47f7aec95"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"3c8fe9f7b4e1e44eec0315b8a74c4181fd578c3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"b3a524b3_cf3bdb93","updated":"2022-02-07 19:04:25.000000000","message":"recheck","commit_id":"8ba04aa262ac9ca785359b5e63ab6ec47f7aec95"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"48f682c20d15856ab216842100325ed84d9586a6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"b80f26e6_dba40572","updated":"2022-02-02 05:51:11.000000000","message":"recheck","commit_id":"8ba04aa262ac9ca785359b5e63ab6ec47f7aec95"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"d8b4bcffd811dbf6c6e65e044ed3e47ebf2f8153","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"d5d50960_dfa90050","updated":"2022-02-03 12:43:41.000000000","message":"recheck","commit_id":"8ba04aa262ac9ca785359b5e63ab6ec47f7aec95"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"1afd55790d0aee90eefbd49e8711a8d5ec96da93","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"85e43f79_62315a22","updated":"2022-02-04 14:59:04.000000000","message":"recheck\n\ntesting new depends-on patchset","commit_id":"8ba04aa262ac9ca785359b5e63ab6ec47f7aec95"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"252cdd7802151e4530042e4f3d70a7e36cc2eadc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"2c2399ce_862e7d35","updated":"2022-02-07 19:04:43.000000000","message":"waiting for c9s job results","commit_id":"8ba04aa262ac9ca785359b5e63ab6ec47f7aec95"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"a345feaa24e863ddf12b358ccfee64bbb10c9d61","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"b41e4764_e906df4d","updated":"2022-02-07 20:30:27.000000000","message":"Still LGTM, just a dependency change.","commit_id":"a2c234ea225bae8f979daf3a9e835d864723bbff"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"5cca2584845f4370c2530b496ad8d840965866b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"af256d86_830c28ec","updated":"2022-02-08 06:47:15.000000000","message":"it looks good for centos jobs:\n\noctavia-v2-dsvm-scenario-centos-8-stream https://zuul.opendev.org/t/openstack/build/952291da630a4c58b7369c8b9d896aea : SUCCESS in 2h 08m 31s (non-voting)\noctavia-v2-dsvm-scenario-centos-9-stream https://zuul.opendev.org/t/openstack/build/984fe68225e64e8a92d3587bb5746c71 : SUCCESS in 1h 56m 53s (non-voting)\n","commit_id":"a2c234ea225bae8f979daf3a9e835d864723bbff"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"854e9199c2bedb43f2bf5565e08b9983dccf8da3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"219b9adb_70f473ac","updated":"2022-02-08 18:08:10.000000000","message":"recheck\n\ndepends-on is merged","commit_id":"a2c234ea225bae8f979daf3a9e835d864723bbff"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"1c44836176cf17a5fb6d632fda365799fe0b555b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"2613e676_9a0aa6af","updated":"2022-02-08 18:47:23.000000000","message":"recheck\n\nipv6 issue","commit_id":"a2c234ea225bae8f979daf3a9e835d864723bbff"}],"octavia_tempest_plugin/tests/test_base.py":[{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"27c06d6fe89e3bfad932be7273c5d7f2c100693b","unresolved":true,"context_lines":[{"line_number":1017,"context_line":"                \"-o PubkeyAcceptedKeyTypes\u003d+ssh-rsa\")"},{"line_number":1018,"context_line":"            if cls._need_scp_protocol():"},{"line_number":1019,"context_line":"                ssh_extra_args +\u003d \" -O\""},{"line_number":1020,"context_line":"            cmd \u003d (\"scp -v -o UserKnownHostsFile\u003d/dev/null \""},{"line_number":1021,"context_line":"                   \"{7} \""},{"line_number":1022,"context_line":"                   \"-o StrictHostKeyChecking\u003dno \""},{"line_number":1023,"context_line":"                   \"-o ConnectTimeout\u003d{0} -o ConnectionAttempts\u003d{1} \""},{"line_number":1024,"context_line":"                   \"-i {2} {3} {4}@{5}:{6}\").format("},{"line_number":1025,"context_line":"                CONF.load_balancer.scp_connection_timeout,"},{"line_number":1026,"context_line":"                CONF.load_balancer.scp_connection_attempts,"},{"line_number":1027,"context_line":"                key.name, local_file, CONF.validation.image_ssh_user,"},{"line_number":1028,"context_line":"                ip_address, const.TEST_SERVER_BINARY,"},{"line_number":1029,"context_line":"                ssh_extra_args)"},{"line_number":1030,"context_line":"            args \u003d shlex.split(cmd)"},{"line_number":1031,"context_line":"            subprocess_args \u003d {\u0027stdout\u0027: subprocess.PIPE,"},{"line_number":1032,"context_line":"                               \u0027stderr\u0027: subprocess.STDOUT,"}],"source_content_type":"text/x-python","patch_set":12,"id":"e682320a_f5f3763e","line":1029,"range":{"start_line":1020,"start_character":19,"end_line":1029,"end_character":31},"updated":"2022-01-28 11:13:07.000000000","message":"Just a suggestion: Since Python 2 is no longer supported, we should no longer need to use str.format(). Using f-strings would make things a lot more readable IMO.","commit_id":"fbf51a7633cbd68d9bb6fcec403362b28c31376b"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"a7b9d080b3b7e4f9f76311cd9f448a8bbdb2151d","unresolved":true,"context_lines":[{"line_number":1017,"context_line":"                \"-o PubkeyAcceptedKeyTypes\u003d+ssh-rsa\")"},{"line_number":1018,"context_line":"            if cls._need_scp_protocol():"},{"line_number":1019,"context_line":"                ssh_extra_args +\u003d \" -O\""},{"line_number":1020,"context_line":"            cmd \u003d (\"scp -v -o UserKnownHostsFile\u003d/dev/null \""},{"line_number":1021,"context_line":"                   \"{7} \""},{"line_number":1022,"context_line":"                   \"-o StrictHostKeyChecking\u003dno \""},{"line_number":1023,"context_line":"                   \"-o ConnectTimeout\u003d{0} -o ConnectionAttempts\u003d{1} \""},{"line_number":1024,"context_line":"                   \"-i {2} {3} {4}@{5}:{6}\").format("},{"line_number":1025,"context_line":"                CONF.load_balancer.scp_connection_timeout,"},{"line_number":1026,"context_line":"                CONF.load_balancer.scp_connection_attempts,"},{"line_number":1027,"context_line":"                key.name, local_file, CONF.validation.image_ssh_user,"},{"line_number":1028,"context_line":"                ip_address, const.TEST_SERVER_BINARY,"},{"line_number":1029,"context_line":"                ssh_extra_args)"},{"line_number":1030,"context_line":"            args \u003d shlex.split(cmd)"},{"line_number":1031,"context_line":"            subprocess_args \u003d {\u0027stdout\u0027: subprocess.PIPE,"},{"line_number":1032,"context_line":"                               \u0027stderr\u0027: subprocess.STDOUT,"}],"source_content_type":"text/x-python","patch_set":12,"id":"e940ac84_75a61474","line":1029,"range":{"start_line":1020,"start_character":19,"end_line":1029,"end_character":31},"in_reply_to":"ac164298_46b67e46","updated":"2022-02-03 13:57:59.000000000","message":"recommended way I\u0027m not sure, but in this particular case, they would be make it more readable, using named parameters with format would be also a solution","commit_id":"fbf51a7633cbd68d9bb6fcec403362b28c31376b"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"d8b4bcffd811dbf6c6e65e044ed3e47ebf2f8153","unresolved":true,"context_lines":[{"line_number":1017,"context_line":"                \"-o PubkeyAcceptedKeyTypes\u003d+ssh-rsa\")"},{"line_number":1018,"context_line":"            if cls._need_scp_protocol():"},{"line_number":1019,"context_line":"                ssh_extra_args +\u003d \" -O\""},{"line_number":1020,"context_line":"            cmd \u003d (\"scp -v -o UserKnownHostsFile\u003d/dev/null \""},{"line_number":1021,"context_line":"                   \"{7} \""},{"line_number":1022,"context_line":"                   \"-o StrictHostKeyChecking\u003dno \""},{"line_number":1023,"context_line":"                   \"-o ConnectTimeout\u003d{0} -o ConnectionAttempts\u003d{1} \""},{"line_number":1024,"context_line":"                   \"-i {2} {3} {4}@{5}:{6}\").format("},{"line_number":1025,"context_line":"                CONF.load_balancer.scp_connection_timeout,"},{"line_number":1026,"context_line":"                CONF.load_balancer.scp_connection_attempts,"},{"line_number":1027,"context_line":"                key.name, local_file, CONF.validation.image_ssh_user,"},{"line_number":1028,"context_line":"                ip_address, const.TEST_SERVER_BINARY,"},{"line_number":1029,"context_line":"                ssh_extra_args)"},{"line_number":1030,"context_line":"            args \u003d shlex.split(cmd)"},{"line_number":1031,"context_line":"            subprocess_args \u003d {\u0027stdout\u0027: subprocess.PIPE,"},{"line_number":1032,"context_line":"                               \u0027stderr\u0027: subprocess.STDOUT,"}],"source_content_type":"text/x-python","patch_set":12,"id":"ac164298_46b67e46","line":1029,"range":{"start_line":1020,"start_character":19,"end_line":1029,"end_character":31},"in_reply_to":"c4c0c5af_d3b7c330","updated":"2022-02-03 12:43:41.000000000","message":"So f-strings is the new recommended way to do this type of thing everywhere? We should generally convert / stop using .format()?","commit_id":"fbf51a7633cbd68d9bb6fcec403362b28c31376b"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"44ff56028f2ab4339476c2b82120ac299ed28f55","unresolved":true,"context_lines":[{"line_number":1017,"context_line":"                \"-o PubkeyAcceptedKeyTypes\u003d+ssh-rsa\")"},{"line_number":1018,"context_line":"            if cls._need_scp_protocol():"},{"line_number":1019,"context_line":"                ssh_extra_args +\u003d \" -O\""},{"line_number":1020,"context_line":"            cmd \u003d (\"scp -v -o UserKnownHostsFile\u003d/dev/null \""},{"line_number":1021,"context_line":"                   \"{7} \""},{"line_number":1022,"context_line":"                   \"-o StrictHostKeyChecking\u003dno \""},{"line_number":1023,"context_line":"                   \"-o ConnectTimeout\u003d{0} -o ConnectionAttempts\u003d{1} \""},{"line_number":1024,"context_line":"                   \"-i {2} {3} {4}@{5}:{6}\").format("},{"line_number":1025,"context_line":"                CONF.load_balancer.scp_connection_timeout,"},{"line_number":1026,"context_line":"                CONF.load_balancer.scp_connection_attempts,"},{"line_number":1027,"context_line":"                key.name, local_file, CONF.validation.image_ssh_user,"},{"line_number":1028,"context_line":"                ip_address, const.TEST_SERVER_BINARY,"},{"line_number":1029,"context_line":"                ssh_extra_args)"},{"line_number":1030,"context_line":"            args \u003d shlex.split(cmd)"},{"line_number":1031,"context_line":"            subprocess_args \u003d {\u0027stdout\u0027: subprocess.PIPE,"},{"line_number":1032,"context_line":"                               \u0027stderr\u0027: subprocess.STDOUT,"}],"source_content_type":"text/x-python","patch_set":12,"id":"c4c0c5af_d3b7c330","line":1029,"range":{"start_line":1020,"start_character":19,"end_line":1029,"end_character":31},"in_reply_to":"e682320a_f5f3763e","updated":"2022-01-28 13:23:30.000000000","message":"Yes, I\u0027m sorry for adding a new parameter to this monster","commit_id":"fbf51a7633cbd68d9bb6fcec403362b28c31376b"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"69f0ce27a75f9304e0756c8470785a514d1de8a7","unresolved":false,"context_lines":[{"line_number":1017,"context_line":"                \"-o PubkeyAcceptedKeyTypes\u003d+ssh-rsa\")"},{"line_number":1018,"context_line":"            if cls._need_scp_protocol():"},{"line_number":1019,"context_line":"                ssh_extra_args +\u003d \" -O\""},{"line_number":1020,"context_line":"            cmd \u003d (\"scp -v -o UserKnownHostsFile\u003d/dev/null \""},{"line_number":1021,"context_line":"                   \"{7} \""},{"line_number":1022,"context_line":"                   \"-o StrictHostKeyChecking\u003dno \""},{"line_number":1023,"context_line":"                   \"-o ConnectTimeout\u003d{0} -o ConnectionAttempts\u003d{1} \""},{"line_number":1024,"context_line":"                   \"-i {2} {3} {4}@{5}:{6}\").format("},{"line_number":1025,"context_line":"                CONF.load_balancer.scp_connection_timeout,"},{"line_number":1026,"context_line":"                CONF.load_balancer.scp_connection_attempts,"},{"line_number":1027,"context_line":"                key.name, local_file, CONF.validation.image_ssh_user,"},{"line_number":1028,"context_line":"                ip_address, const.TEST_SERVER_BINARY,"},{"line_number":1029,"context_line":"                ssh_extra_args)"},{"line_number":1030,"context_line":"            args \u003d shlex.split(cmd)"},{"line_number":1031,"context_line":"            subprocess_args \u003d {\u0027stdout\u0027: subprocess.PIPE,"},{"line_number":1032,"context_line":"                               \u0027stderr\u0027: subprocess.STDOUT,"}],"source_content_type":"text/x-python","patch_set":12,"id":"eea42a55_4015c20c","line":1029,"range":{"start_line":1020,"start_character":19,"end_line":1029,"end_character":31},"in_reply_to":"e940ac84_75a61474","updated":"2022-02-08 08:02:29.000000000","message":"Ack","commit_id":"fbf51a7633cbd68d9bb6fcec403362b28c31376b"}]}