)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3c3b1b71f81aa88f9ec1b7ed60e27e6f3e64f86a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"abfe50b6_2116d1fa","updated":"2024-03-05 17:55:32.000000000","message":"actully i would prefer if you added some unit test coverage for this or modified the extisting test to assert the relevent log was emitted.","commit_id":"8856a6db302ad742cdd9194c053510f733d62646"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"2036375025ff627da7222695e049cbc2a14a1c59","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"02d85212_a793a52d","updated":"2024-03-05 17:54:46.000000000","message":"this might be a little noisy but its at debug level so that is generally ok.\n\nwe have some operators that deploy with debug logging enabeld by default and we intend that to be a vaild configuration to run in production.\n\nso while im conservative with adding too much logging to this type of thing i agree this is a good improvement.","commit_id":"8856a6db302ad742cdd9194c053510f733d62646"}]}