)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35201,"name":"Michael Kelly","display_name":"Michael Kelly","email":"mkelly@arista.com","username":"michael_kelly"},"change_message_id":"da39dd39974b327e94d33ffb06566ef3f396a947","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"dc7c5057_117c3d08","updated":"2022-11-04 03:10:21.000000000","message":"recheck","commit_id":"bee39b03663e738c3d2263c19605b8b1bfbba3d3"}],"zuul_operator/utils.py":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"8227220bfc9e9a70e599e995dc2508b4ca81fa80","unresolved":false,"context_lines":[{"line_number":82,"context_line":""},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"def pod_exec(namespace, name, command):"},{"line_number":85,"context_line":"    kubernetes.config.load_kube_config()"},{"line_number":86,"context_line":"    try:"},{"line_number":87,"context_line":"        c \u003d Configuration().get_default_copy()"},{"line_number":88,"context_line":"    except AttributeError:"}],"source_content_type":"text/x-python","patch_set":3,"id":"fedf9e8f_7da9955a","side":"PARENT","line":85,"updated":"2022-10-27 19:56:25.000000000","message":"The fact that we weren\u0027t catching the error makes me realize we really should have testing for this.  Especially since there has been an alternate suggestion about how to fix this: https://review.opendev.org/c/zuul/zuul-operator/+/810498/\n\nIt would be great if we could:\n* Identify the best fix (ideally we should ensure running the operator from the CLI and in the cluster both continue to wkr)\n* Add a config update to trigger this in the functional test (functional/run.yaml)\n\nProbably the easiest way to do that would be to add a tenant to main.yaml, then wait for that tenant to show up in the tenant list in the web api.","commit_id":"8b05b31a067b386d16c9e925ed1214e30d12529e"}]}
