)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":37924,"name":"Abhishek Bongale","display_name":"abongale","email":"abhishekbongale@outlook.com","username":"abhibongale"},"change_message_id":"c8c773a3c296fb621dd9edabab710f1a826fd7e9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9e7ef422_4ba4601d","updated":"2025-12-10 16:43:06.000000000","message":"LGTM","commit_id":"9c864ad7a262cc41b6fdb6ce9b76b898e1ef3f4e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a32d84bb46472aa25bcd79670512a5f60a4be78a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d131b214_e6045a5d","updated":"2025-12-15 20:19:57.000000000","message":"overall lgtm, a small suggestion.","commit_id":"9c864ad7a262cc41b6fdb6ce9b76b898e1ef3f4e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f515da9a8ed7cd6920d9210ebe8412be4437dd2d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5a782f72_76f40232","updated":"2025-12-16 00:16:28.000000000","message":"thanks for updates, lgtm","commit_id":"31e13d302c371e37cc90e704cb6b2bb0079139d4"}],"tempest/api/compute/servers/test_create_server_multi_nic.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a32d84bb46472aa25bcd79670512a5f60a4be78a","unresolved":true,"context_lines":[{"line_number":68,"context_line":"        if CONF.enforce_scope.nova:"},{"line_number":69,"context_line":"            cls.reader_client \u003d cls.os_project_reader.servers_client"},{"line_number":70,"context_line":"        else:"},{"line_number":71,"context_line":"            cls.reader_client \u003d cls.client"},{"line_number":72,"context_line":"        cls.networks_client \u003d cls.os_primary.networks_client"},{"line_number":73,"context_line":"        cls.subnets_client \u003d cls.os_primary.subnets_client"},{"line_number":74,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"4ca23f0c_65f6bd11","line":71,"range":{"start_line":71,"start_character":32,"end_line":71,"end_character":42},"updated":"2025-12-15 20:19:57.000000000","message":"we do not need separate cls.client var and let\u0027s use directly here \n\ncls.reader_client \u003d cls.servers_client","commit_id":"9c864ad7a262cc41b6fdb6ce9b76b898e1ef3f4e"},{"author":{"_account_id":34373,"name":"Jason Paroly","email":"jparoly@redhat.com","username":"jparoly"},"change_message_id":"e742c60998138fb8c22d6bd31f19475478b695d9","unresolved":true,"context_lines":[{"line_number":68,"context_line":"        if CONF.enforce_scope.nova:"},{"line_number":69,"context_line":"            cls.reader_client \u003d cls.os_project_reader.servers_client"},{"line_number":70,"context_line":"        else:"},{"line_number":71,"context_line":"            cls.reader_client \u003d cls.client"},{"line_number":72,"context_line":"        cls.networks_client \u003d cls.os_primary.networks_client"},{"line_number":73,"context_line":"        cls.subnets_client \u003d cls.os_primary.subnets_client"},{"line_number":74,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9348c17f_eeda01c4","line":71,"range":{"start_line":71,"start_character":32,"end_line":71,"end_character":42},"in_reply_to":"4ca23f0c_65f6bd11","updated":"2025-12-15 22:03:19.000000000","message":"I will make that change here.  It was done this way before as well so I could go back and change that.  Thoughts?","commit_id":"9c864ad7a262cc41b6fdb6ce9b76b898e1ef3f4e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f515da9a8ed7cd6920d9210ebe8412be4437dd2d","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        if CONF.enforce_scope.nova:"},{"line_number":69,"context_line":"            cls.reader_client \u003d cls.os_project_reader.servers_client"},{"line_number":70,"context_line":"        else:"},{"line_number":71,"context_line":"            cls.reader_client \u003d cls.client"},{"line_number":72,"context_line":"        cls.networks_client \u003d cls.os_primary.networks_client"},{"line_number":73,"context_line":"        cls.subnets_client \u003d cls.os_primary.subnets_client"},{"line_number":74,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"2bbe3460_e92b5f68","line":71,"range":{"start_line":71,"start_character":32,"end_line":71,"end_character":42},"in_reply_to":"9348c17f_eeda01c4","updated":"2025-12-16 00:16:28.000000000","message":"That was needed as self.client var was used for other operation but in this test file it was not used for other operations so not needed\n\nIn other change, that was used for stop_Server also.\n- https://review.opendev.org/c/openstack/tempest/+/970130/2/tempest/api/compute/servers/test_list_server_filters.py#133","commit_id":"9c864ad7a262cc41b6fdb6ce9b76b898e1ef3f4e"}]}