)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"69bb4f1ff2cdb2c42152c18c2893d57b77e7730a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cd4c6ee0_4b763010","updated":"2022-07-27 07:42:19.000000000","message":"just a nit on the code comment","commit_id":"5406081a204e03cddd38fa295e31ab4052ccce3a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"506d0fe090a2362f7e770034afc1b97a314ce13e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"6d48fae8_0bfd2f1e","updated":"2022-11-18 12:33:12.000000000","message":"as far as i can tell you have  not actully adressed the comments form the previous review","commit_id":"7a5e548f11feec3dabd4dd3cd975f701dcd300ba"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"092399fc750647c6796ca28f21d608b3b7dcf49a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"949d531f_94876b00","updated":"2022-11-21 14:31:24.000000000","message":"ok i think we can proceed with this as is.","commit_id":"185201974775bab966f4e5ca3bbdc31b8269fa4c"}],"nova/tests/functional/regressions/test_bug_1951656.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"590479f6d0a4625dc6b63cbb4dcbce7ec12a33c3","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# Copyright 2021, Canonical, Inc. All Rights Reserved."},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"# not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":2,"id":"af51a2e9_a8faf17e","line":1,"updated":"2022-07-27 08:13:29.000000000","message":"damn copy/paste, will remove it, we don\u0027t need licensing here.","commit_id":"5406081a204e03cddd38fa295e31ab4052ccce3a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f17388ac8c3009d5f8b42b550adc5c0313e5e725","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# Copyright 2021, Canonical, Inc. All Rights Reserved."},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"# not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":2,"id":"ef27e0aa_b93471f0","line":1,"in_reply_to":"af51a2e9_a8faf17e","updated":"2022-08-10 10:47:49.000000000","message":"well we need the license header we dont need copyright statements","commit_id":"5406081a204e03cddd38fa295e31ab4052ccce3a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"092399fc750647c6796ca28f21d608b3b7dcf49a","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright 2021, Canonical, Inc. All Rights Reserved."},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"# not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":2,"id":"62b166f8_5c3b1dcd","line":1,"in_reply_to":"ef27e0aa_b93471f0","updated":"2022-11-21 14:31:24.000000000","message":"Done","commit_id":"5406081a204e03cddd38fa295e31ab4052ccce3a"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"69bb4f1ff2cdb2c42152c18c2893d57b77e7730a","unresolved":true,"context_lines":[{"line_number":78,"context_line":"            networks\u003d\u0027auto\u0027, expected_state\u003d\u0027ERROR\u0027)"},{"line_number":79,"context_line":"        # The error is due to a bad mdev name parsing"},{"line_number":80,"context_line":"        self.assertIn(\u0027fault\u0027, server)"},{"line_number":81,"context_line":"        # since we forced the host, we have a RescheduledException"},{"line_number":82,"context_line":"        self.assertIn(\u0027Exceeded maximum number of retries\u0027,"},{"line_number":83,"context_line":"                      server[\u0027fault\u0027][\u0027message\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"dec07f38_fa27702f","line":81,"updated":"2022-07-27 07:42:19.000000000","message":"RescheduledException has nothing to do with the forced host. In this test you have a single compute so you don\u0027t even have to force the host and you will get the same RescheduledException.\n\nThe reason of this exception is that _get_mediated_device_information calls mdev_name2uuid and that raise a  ValueError(\u0027badly formed hexadecimal UUID string\u0027)\nand that exception bubble up to _build_and_run_instance [1] where it is translated to RescheduledException.\n\nhttps://github.com/openstack/nova/blob/bcb96f362ab12e297f125daa5189fb66345b4976/nova/compute/manager.py#L2579-L2588","commit_id":"5406081a204e03cddd38fa295e31ab4052ccce3a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"092399fc750647c6796ca28f21d608b3b7dcf49a","unresolved":false,"context_lines":[{"line_number":78,"context_line":"            networks\u003d\u0027auto\u0027, expected_state\u003d\u0027ERROR\u0027)"},{"line_number":79,"context_line":"        # The error is due to a bad mdev name parsing"},{"line_number":80,"context_line":"        self.assertIn(\u0027fault\u0027, server)"},{"line_number":81,"context_line":"        # since we forced the host, we have a RescheduledException"},{"line_number":82,"context_line":"        self.assertIn(\u0027Exceeded maximum number of retries\u0027,"},{"line_number":83,"context_line":"                      server[\u0027fault\u0027][\u0027message\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"683155fa_c72dbf4a","line":81,"in_reply_to":"38a9aff0_563718b6","updated":"2022-11-21 14:31:24.000000000","message":"Done","commit_id":"5406081a204e03cddd38fa295e31ab4052ccce3a"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"590479f6d0a4625dc6b63cbb4dcbce7ec12a33c3","unresolved":true,"context_lines":[{"line_number":78,"context_line":"            networks\u003d\u0027auto\u0027, expected_state\u003d\u0027ERROR\u0027)"},{"line_number":79,"context_line":"        # The error is due to a bad mdev name parsing"},{"line_number":80,"context_line":"        self.assertIn(\u0027fault\u0027, server)"},{"line_number":81,"context_line":"        # since we forced the host, we have a RescheduledException"},{"line_number":82,"context_line":"        self.assertIn(\u0027Exceeded maximum number of retries\u0027,"},{"line_number":83,"context_line":"                      server[\u0027fault\u0027][\u0027message\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"38a9aff0_563718b6","line":81,"in_reply_to":"dec07f38_fa27702f","updated":"2022-07-27 08:13:29.000000000","message":"Yeah, agreed, given we only have a compute, we get this exception as the first call was having the ValueError.","commit_id":"5406081a204e03cddd38fa295e31ab4052ccce3a"}]}
