)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"4a9799404add6eb03b9176d133a559a4270912c5","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Adds server show test method"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adds server show in integrated_helpers"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Also fixes pep8 thing in"},{"line_number":12,"context_line":"nova/tests/unit/api/openstack/compute/test_server_topology.py"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"25ccb95a_7872593a","line":10,"updated":"2023-10-05 12:29:27.000000000","message":"you\u0027re also changing the behavior of a specific test, which is to directly return the server without waiting it to be active.\nThat said, I understand *why* you do this, since the instance is already active.\n\nCould you please explain it in the commit msg ?","commit_id":"1dbb680132cc5ebee336ce4de85ed5941dabba3d"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"105bce74f1ba63970d2bab74e3a62d0bf7b8e027","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Adds server show test method"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adds server show in integrated_helpers"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Also fixes pep8 thing in"},{"line_number":12,"context_line":"nova/tests/unit/api/openstack/compute/test_server_topology.py"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"01ec8b12_ac1bf5d0","line":10,"in_reply_to":"25ccb95a_7872593a","updated":"2023-10-12 08:37:51.000000000","message":"Done","commit_id":"1dbb680132cc5ebee336ce4de85ed5941dabba3d"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"4a9799404add6eb03b9176d133a559a4270912c5","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adds server show in integrated_helpers"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Also fixes pep8 thing in"},{"line_number":12,"context_line":"nova/tests/unit/api/openstack/compute/test_server_topology.py"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Related-Bug: #2033601"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"8060627d_aaaef011","line":12,"range":{"start_line":11,"start_character":0,"end_line":12,"end_character":61},"updated":"2023-10-05 12:29:27.000000000","message":"No, this isn\u0027t a pep8 issue, just probably a black one, so we shouldn\u0027t change it.","commit_id":"1dbb680132cc5ebee336ce4de85ed5941dabba3d"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"e8d27a7e21ba6b3d61148a74214e8ad8840c17b5","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adds server show in integrated_helpers"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Also fixes pep8 thing in"},{"line_number":12,"context_line":"nova/tests/unit/api/openstack/compute/test_server_topology.py"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Related-Bug: #2033601"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"cad60083_ab8115f2","line":12,"range":{"start_line":11,"start_character":0,"end_line":12,"end_character":61},"in_reply_to":"8060627d_aaaef011","updated":"2023-10-09 12:09:05.000000000","message":"Ack","commit_id":"1dbb680132cc5ebee336ce4de85ed5941dabba3d"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"0c2cbd8131b0be913e5d89d36a0cb06efe1753eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"893e9638_a26e5a01","updated":"2024-01-16 10:01:00.000000000","message":"lgtm","commit_id":"dabf90e0de79abbf97b1e3b2fc104eac115ecaa9"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"e8d27a7e21ba6b3d61148a74214e8ad8840c17b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"02eb9e68_85fb6fc6","updated":"2023-10-09 12:09:05.000000000","message":"recheck docs failed https://bugs.launchpad.net/nova/+bug/2026345 \nnot failing from sometime now: https://zuul.openstack.org/builds?job_name\u003dopenstack-tox-docs\u0026project\u003dopenstack%2Fnova\u0026branch\u003dmaster\u0026skip\u003d0","commit_id":"dabf90e0de79abbf97b1e3b2fc104eac115ecaa9"}]}