)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"762e6df970f3e00ab6bd554dee8d04a46046b3a0","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Tim Burke \u003ctim.burke@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-05-10 22:51:22 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Allow unit tests to be run via pytest"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I1dfa239f9ee9ea85663b5c1f22631a97f87b4dfc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"159693e0_4065fc43","line":7,"updated":"2021-05-11 14:55:02.000000000","message":"can you provide an example commandline for how you tested this change?","commit_id":"7f26103e3a0117bc195ae1ecd757e42c00a8a5da"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"eb16b46fe8bd1ac5cc2a15bf7d92e9b3366911e2","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Tim Burke \u003ctim.burke@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-05-10 22:51:22 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Allow unit tests to be run via pytest"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I1dfa239f9ee9ea85663b5c1f22631a97f87b4dfc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"4e064133_99d07579","line":7,"in_reply_to":"159693e0_4065fc43","updated":"2021-05-11 20:43:17.000000000","message":"Done","commit_id":"7f26103e3a0117bc195ae1ecd757e42c00a8a5da"}],"test/unit/test_shell.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"762e6df970f3e00ab6bd554dee8d04a46046b3a0","unresolved":true,"context_lines":[{"line_number":3040,"context_line":"            self.defaults[\u0027auth-version\u0027] \u003d \u00272.0\u0027"},{"line_number":3041,"context_line":"            self._test_options(opts, os_opts, flags\u003dself.flags)"},{"line_number":3042,"context_line":"        finally:"},{"line_number":3043,"context_line":"            rootLogger.setLevel(orig_lvl)"},{"line_number":3044,"context_line":""},{"line_number":3045,"context_line":"    def test_catalog_options_and_flags_not_required_v3(self):"},{"line_number":3046,"context_line":"        # check that all possible command line args are passed to keystone"}],"source_content_type":"text/x-python","patch_set":1,"id":"dd411480_87ae0b8b","line":3043,"updated":"2021-05-11 14:55:02.000000000","message":"any clues you could give as to why this change was needed - or should I just revert it and run pytest?","commit_id":"7f26103e3a0117bc195ae1ecd757e42c00a8a5da"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"eb16b46fe8bd1ac5cc2a15bf7d92e9b3366911e2","unresolved":true,"context_lines":[{"line_number":3040,"context_line":"            self.defaults[\u0027auth-version\u0027] \u003d \u00272.0\u0027"},{"line_number":3041,"context_line":"            self._test_options(opts, os_opts, flags\u003dself.flags)"},{"line_number":3042,"context_line":"        finally:"},{"line_number":3043,"context_line":"            rootLogger.setLevel(orig_lvl)"},{"line_number":3044,"context_line":""},{"line_number":3045,"context_line":"    def test_catalog_options_and_flags_not_required_v3(self):"},{"line_number":3046,"context_line":"        # check that all possible command line args are passed to keystone"}],"source_content_type":"text/x-python","patch_set":1,"id":"0c9416d1_5add7a8b","line":3043,"in_reply_to":"dd411480_87ae0b8b","updated":"2021-05-11 20:43:17.000000000","message":"So the debug flag gets set based off of\n\n logging.getLogger(\"swiftclient\").isEnabledFor(logging.DEBUG)\n\n(see https://github.com/openstack/python-swiftclient/blob/3.11.1/swiftclient/client.py#L587)\n\nI *think* it\u0027s that nose would automatically set the root logger level to DEBUG for us, while pytest would leave it at the default of WARNING.","commit_id":"7f26103e3a0117bc195ae1ecd757e42c00a8a5da"}]}