)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"b7c5d1bb8bf057a282bff07260db03c394943f9c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"e6c1f562_4aba71ff","updated":"2023-08-08 12:12:22.000000000","message":"Minor opinion: I am not sure that we need separate statefulsets for nb and sb (db and raft ports are different). IMO it could be a single all-in-one statefulset that runs sb, nb and northd.","commit_id":"ec29020b32eecd81e9863b7d518f04b72a18e20b"}],"ovn/templates/daemonset-controller.yaml":[{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"b7c5d1bb8bf057a282bff07260db03c394943f9c","unresolved":true,"context_lines":[{"line_number":25,"context_line":"  name: ovn-controller"},{"line_number":26,"context_line":"  annotations:"},{"line_number":27,"context_line":"    {{ tuple $envAll | include \"helm-toolkit.snippets.release_uuid\" }}"},{"line_number":28,"context_line":"    configmap-bin-hash: {{ tuple \"configmap-bin.yaml\" . | include \"helm-toolkit.utils.hash\" }}"},{"line_number":29,"context_line":"  labels:"},{"line_number":30,"context_line":"{{ tuple $envAll \"ovn\" \"ovn-controller\" | include \"helm-toolkit.snippets.kubernetes_metadata_labels\" | indent 4 }}"},{"line_number":31,"context_line":"spec:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"b8d1d26e_8ab3c60e","line":28,"updated":"2023-08-08 12:12:22.000000000","message":"nit: Since we put this to the pod spec template, then probably there is no need to have it here.","commit_id":"ec29020b32eecd81e9863b7d518f04b72a18e20b"}]}