)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"f5be76caa985f4e08c72dfa2a97c0bebe007de0b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9ab3f032_2bd4e090","updated":"2023-10-05 18:07:01.000000000","message":"One comment to allow this to run downstream as well.  Whenever you are ok with upstream implementation I could update/test downstream also.","commit_id":"df84402face5522a068937fe66bb92a02850d28b"}],"whitebox_tempest_plugin/services/clients.py":[{"author":{"_account_id":31033,"name":"James Parker","email":"jparker@redhat.com","username":"jparker"},"change_message_id":"f5be76caa985f4e08c72dfa2a97c0bebe007de0b","unresolved":true,"context_lines":[{"line_number":162,"context_line":""},{"line_number":163,"context_line":"        self.stop()"},{"line_number":164,"context_line":"        self.start(wait\u003dFalse)"},{"line_number":165,"context_line":"        command \u003d (\u0027journalctl -u devstack@n-cpu -S \"1 min ago\" \u0027"},{"line_number":166,"context_line":"                   \u0027-g \\\u0027%s\\\u0027\u0027 % error_msg)"},{"line_number":167,"context_line":"        for i in range(60):"},{"line_number":168,"context_line":"            try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"8fe15e44_7b696f52","line":165,"updated":"2023-10-05 18:07:01.000000000","message":"Could you do something similar to this [1] which is defined by [2]. The unit is not defined downstream so we have to parse the log directory mounted into nova-compute \u0027/var/log/nova\u0027. This would allow the test to run in downstream CI as well.\n\n[1] https://opendev.org/openstack/whitebox-tempest-plugin/src/branch/master/whitebox_tempest_plugin/services/clients.py#L85\n[2] https://opendev.org/openstack/whitebox-tempest-plugin/src/branch/master/whitebox_tempest_plugin/config.py#L147","commit_id":"df84402face5522a068937fe66bb92a02850d28b"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"78e3f7c567930a0964aff1ac9dd16208bb30bde9","unresolved":true,"context_lines":[{"line_number":162,"context_line":""},{"line_number":163,"context_line":"        self.stop()"},{"line_number":164,"context_line":"        self.start(wait\u003dFalse)"},{"line_number":165,"context_line":"        command \u003d (\u0027journalctl -u devstack@n-cpu -S \"1 min ago\" \u0027"},{"line_number":166,"context_line":"                   \u0027-g \\\u0027%s\\\u0027\u0027 % error_msg)"},{"line_number":167,"context_line":"        for i in range(60):"},{"line_number":168,"context_line":"            try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"35da4c40_e77f5873","line":165,"in_reply_to":"8fe15e44_7b696f52","updated":"2023-10-05 19:01:08.000000000","message":"Yeah, sorry I should have commented here after lots of discussion with artom. This\u0027ll need plenty of cleanup to make it generic for sure.","commit_id":"df84402face5522a068937fe66bb92a02850d28b"}]}