)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"ba49ad34de7e23c1ed7fab4d72e8f891abe7a116","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f3b2423c_b9ae4ff6","updated":"2024-11-20 01:22:11.000000000","message":"Thanks James - I normally wouldn\u0027t +2+A my own patch, but I\u0027m interpreting your upload as tacit approval of everything else :)","commit_id":"bee34dbb867dc3c107f1262f68a997ef7ccff55a"}],"whitebox_tempest_plugin/api/compute/base.py":[{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"33a167d383869256b7ca5ea06c00a4bd6f3c24da","unresolved":true,"context_lines":[{"line_number":144,"context_line":"        timeout \u003d CONF.compute.build_timeout"},{"line_number":145,"context_line":"        interval \u003d CONF.compute.build_interval"},{"line_number":146,"context_line":"        while int(time.time()) - start_time \u003c\u003d timeout:"},{"line_number":147,"context_line":"            domains \u003d ssh_client.execute(\u0027virsh list --name\u0027)"},{"line_number":148,"context_line":"            if domain_name in domains:"},{"line_number":149,"context_line":"                continue"},{"line_number":150,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"52494b0e_16d94d2a","line":147,"updated":"2024-10-28 16:42:28.000000000","message":"Does this need to be sudo?  Looking at the recent logs, the stack user is being used and it is not showing any guests when listing.","commit_id":"0558ad7a4c90a955e4fc00f3facf12e2b396d06f"},{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"26142fcb32bf1b4947aa0d9d955077d9fbf453d2","unresolved":true,"context_lines":[{"line_number":144,"context_line":"        timeout \u003d CONF.compute.build_timeout"},{"line_number":145,"context_line":"        interval \u003d CONF.compute.build_interval"},{"line_number":146,"context_line":"        while int(time.time()) - start_time \u003c\u003d timeout:"},{"line_number":147,"context_line":"            domains \u003d ssh_client.execute(\u0027virsh list --name\u0027)"},{"line_number":148,"context_line":"            if domain_name in domains:"},{"line_number":149,"context_line":"                continue"},{"line_number":150,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"3ec8ca6c_c930ac7e","line":147,"in_reply_to":"52494b0e_16d94d2a","updated":"2024-10-28 17:00:59.000000000","message":"Good catch!","commit_id":"0558ad7a4c90a955e4fc00f3facf12e2b396d06f"},{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"596b5816f5041ba7f0b37c7691ba55e00f0b2b64","unresolved":true,"context_lines":[{"line_number":134,"context_line":"        return ET.fromstring(xml)"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"    def shutdown_server_domain(self, server, host):"},{"line_number":137,"context_line":"        domain_name \u003d server[\u0027OS-EXT-SRV-ATTR:instance_name\u0027]"},{"line_number":138,"context_line":"        ssh_client \u003d clients.SSHClient(host)"},{"line_number":139,"context_line":"        ssh_client.execute(\u0027virsh shutdown %s\u0027 % domain_name, sudo\u003dTrue)"},{"line_number":140,"context_line":"        self._wait_for_domain_shutdown(ssh_client, domain_name)"}],"source_content_type":"text/x-python","patch_set":4,"id":"896d504c_3659213e","line":137,"updated":"2024-10-29 17:20:57.000000000","message":"The guests are being created by primary client so this attribute isn\u0027t showing up when checking. I think either update the guest creations to use the admin client or update here to show server as admin e.g.:\n```\nserver_attributes \u003d self.admin_servers_client.show_server(server[\u0027id\u0027])[\u0027server\u0027]\ndomain_name \u003d server_attributes[\u0027OS-EXT-SRV-ATTR:instance_name\u0027]\n```\n\nwould allow gathering of the domain name.","commit_id":"9c10f3ec43e8bdd7c41683979dd911a42c78f833"}]}