)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"0a416ca526772735e003ac5dde8db940463801c0","unresolved":false,"context_lines":[{"line_number":11,"context_line":"In the first two cases conductor is not handling"},{"line_number":12,"context_line":"the InstanceExists exception raised from the cell"},{"line_number":13,"context_line":"DB API when trying to create an instance so the"},{"line_number":14,"context_line":"instance is stuck in with the BUILD status. In the"},{"line_number":15,"context_line":"third test, osapi_compute_unique_server_name_scope\u003dglobal"},{"line_number":16,"context_line":"is not honered across multiple cells because the"},{"line_number":17,"context_line":"validation of that option happens in the per-cell"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_dc6b7fed","line":14,"range":{"start_line":14,"start_character":18,"end_line":14,"end_character":20},"updated":"2019-12-23 22:03:40.000000000","message":"nix","commit_id":"a49bd9b00166ba37a4f9a66535dceca79a5579fa"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"125f655b44924546ba622b820b19f3c0f8f328f7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"266d711a_0c366ac8","updated":"2022-07-12 09:30:28.000000000","message":"@amit @Rajesh since this is a functional reproducer for the api bug I think this could be a nice next bug for one of you to tackle.\n\nits a bit more complicated of a fix then some of the other bugs you have looked at but I think it would be good for one of ye to try working on a fix for this.","commit_id":"a49bd9b00166ba37a4f9a66535dceca79a5579fa"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c9b979e2cd2cfc0b04970fb1087e59cbf7f6edd8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0dd3d969_c90e8fcc","updated":"2022-07-14 11:50:25.000000000","message":"while there are no code conflict since this is a standalone test the shared code it depens on has evolved and this need to be reworked.","commit_id":"a49bd9b00166ba37a4f9a66535dceca79a5579fa"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"636ca2f9bd043299ebe54d5edf40954aadaf9e76","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"38369e58_2bfefd50","updated":"2023-03-30 08:16:36.000000000","message":"I have three details here but I\u0027d prefer to see how we correctly verify the test.","commit_id":"f61065cbd34a376a6dd90c5abd34870f89f26f87"},{"author":{"_account_id":19234,"name":"Alexey Stupnikov","email":"aleksey.stupnikov@gmail.com","username":"astupnikov"},"change_message_id":"4f866a11b3168ab6f1096ae27dd95dc2a6e70ff7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ae6e5005_7a5daea6","updated":"2023-03-23 13:53:56.000000000","message":"Wh","commit_id":"f61065cbd34a376a6dd90c5abd34870f89f26f87"}],"nova/tests/functional/regressions/test_bug_1857306.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"636ca2f9bd043299ebe54d5edf40954aadaf9e76","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# Copyright 2022 Red Hat, Inc."},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":5,"id":"6bb7e6a3_cf2e752f","line":1,"updated":"2023-03-30 08:16:36.000000000","message":"please remove this line, we no longer need copyrights.","commit_id":"f61065cbd34a376a6dd90c5abd34870f89f26f87"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"636ca2f9bd043299ebe54d5edf40954aadaf9e76","unresolved":true,"context_lines":[{"line_number":11,"context_line":"# distributed under the License is distributed on an \"AS IS\" BASIS, WITHOUT"},{"line_number":12,"context_line":"# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":13,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"# under the License."},{"line_number":15,"context_line":"from nova import test"},{"line_number":16,"context_line":"from nova.tests import fixtures as nova_fixtures"},{"line_number":17,"context_line":"from nova.tests.functional import fixtures as func_fixtures"}],"source_content_type":"text/x-python","patch_set":5,"id":"20fd0e3f_15a3b8a7","line":14,"updated":"2023-03-30 08:16:36.000000000","message":"usually, we leave an empty line here.","commit_id":"f61065cbd34a376a6dd90c5abd34870f89f26f87"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"636ca2f9bd043299ebe54d5edf40954aadaf9e76","unresolved":true,"context_lines":[{"line_number":60,"context_line":"        server2 \u003d self._create_server(name\u003d\u0027test_unique_server_name_cell0\u0027,"},{"line_number":61,"context_line":"                                      networks\u003d[],"},{"line_number":62,"context_line":"                                      expected_state\u003d\u0027BUILD\u0027)"},{"line_number":63,"context_line":"        self._wait_for_state_change(server2, \u0027BUILD\u0027)"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    def test_unique_server_name_cell1(self):"},{"line_number":66,"context_line":"        \"\"\"Tests handling InstanceExists in schedule_and_build_instances.\"\"\""}],"source_content_type":"text/x-python","patch_set":5,"id":"45df5856_2a6d2cd5","line":63,"updated":"2023-03-30 08:16:36.000000000","message":"I\u0027m confused here. What do we want to prove ? The fact that since we hande InstanceExists in the API, we don\u0027t bubble it the exception ? If so, why can\u0027t we just await for the expected_state to be ERROR ?","commit_id":"f61065cbd34a376a6dd90c5abd34870f89f26f87"}]}