)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"df3d4ae3bea11e2398680b16049792a52e0ad61b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"afd42de7_33744bd6","updated":"2026-02-17 11:24:07.000000000","message":"Recheck unrelated failure","commit_id":"78d6fbc58ff45b85174278b4eb3d761d118281b0"},{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"aad058eb6a423929b9485a521c7a608cfadf68a6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fb436409_3d2ba5df","updated":"2026-02-20 00:09:25.000000000","message":"There no volume mounts in three charts designate, octavia and tacker like in most of other templates. Is it what you meant?","commit_id":"78d6fbc58ff45b85174278b4eb3d761d118281b0"},{"author":{"_account_id":7156,"name":"Mathieu Gagné","email":"mgagne@calavera.ca","username":"mgagne"},"change_message_id":"3d56afe547c43615ec7dc2398ba9eba395c1eec2","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ebf0e3e7_4bcc38cd","updated":"2026-02-20 23:10:31.000000000","message":"I didn\u0027t inspect every files but found those were missed:\n./gnocchi/templates/daemonset-metricd.yaml\n./gnocchi/templates/daemonset-statsd.yaml","commit_id":"6210215ed827a4efd1fac2a87cf02897e366f40e"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"32b6de186a89d72e01792379f143df5e209ad3eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9bfcc6a6_10c76dee","in_reply_to":"ebf0e3e7_4bcc38cd","updated":"2026-02-21 03:18:57.000000000","message":"Done","commit_id":"6210215ed827a4efd1fac2a87cf02897e366f40e"},{"author":{"_account_id":7156,"name":"Mathieu Gagné","email":"mgagne@calavera.ca","username":"mgagne"},"change_message_id":"a49856de2b51a1d37e51fd9e99208c6c756f2361","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3c7cc9d7_1275b0fa","updated":"2026-02-23 18:37:42.000000000","message":"Considering some services seem to use /var/lib/service/tmp for other purposes than lock files, would it make sense to use a different directory for locks? So an operator can override one path without impacting the other.\n\nExamples:\n- /var/lib/glance/tmp\n- /var/lib/cinder/tmp\n- /var/lib/manila (state-tmp)","commit_id":"c707830adc700a2b2cf5d99a7dd8ea7fc6bdba6b"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"4be21444e0d0eca0272ff0b3a1fad85816609552","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e17962c5_5bdf3ac7","in_reply_to":"3c7cc9d7_1275b0fa","updated":"2026-02-24 05:16:01.000000000","message":"Yeah I was originally targetting /var/lock which is what we\u0027re using locally in our charts. I\u0027ll use that.","commit_id":"c707830adc700a2b2cf5d99a7dd8ea7fc6bdba6b"},{"author":{"_account_id":7156,"name":"Mathieu Gagné","email":"mgagne@calavera.ca","username":"mgagne"},"change_message_id":"81c22bec2e03c1765bc50ebf776fc4ca9ce395d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c3cf0f1d_c685165a","in_reply_to":"e17962c5_5bdf3ac7","updated":"2026-02-24 14:40:39.000000000","message":"Done","commit_id":"c707830adc700a2b2cf5d99a7dd8ea7fc6bdba6b"}],"aodh/templates/deployment-api.yaml":[{"author":{"_account_id":7156,"name":"Mathieu Gagné","email":"mgagne@calavera.ca","username":"mgagne"},"change_message_id":"7c78b355549ac818517fb142ec716240ded0997e","unresolved":true,"context_lines":[{"line_number":83,"context_line":"            - name: pod-tmp"},{"line_number":84,"context_line":"              mountPath: /tmp"},{"line_number":85,"context_line":"            - name: oslo-lock-path"},{"line_number":86,"context_line":"              mountPath: {{ .Values.conf.oslo_concurrency.lock_path }}"},{"line_number":87,"context_line":"            - name: wsgi-aodh"},{"line_number":88,"context_line":"              mountPath: /var/www/cgi-bin/aodh"},{"line_number":89,"context_line":"            - name: pod-etc-aodh"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"b2e60dc6_bebb7a6d","line":86,"updated":"2026-02-20 23:30:44.000000000","message":"Isn\u0027t the path to the config `.Values.aodh.conf.oslo_concurrency.lock_path` ? It seems to be the issue for all charts where service name is missing for the config path.","commit_id":"6210215ed827a4efd1fac2a87cf02897e366f40e"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"32b6de186a89d72e01792379f143df5e209ad3eb","unresolved":false,"context_lines":[{"line_number":83,"context_line":"            - name: pod-tmp"},{"line_number":84,"context_line":"              mountPath: /tmp"},{"line_number":85,"context_line":"            - name: oslo-lock-path"},{"line_number":86,"context_line":"              mountPath: {{ .Values.conf.oslo_concurrency.lock_path }}"},{"line_number":87,"context_line":"            - name: wsgi-aodh"},{"line_number":88,"context_line":"              mountPath: /var/www/cgi-bin/aodh"},{"line_number":89,"context_line":"            - name: pod-etc-aodh"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"33c94dae_57ce6730","line":86,"in_reply_to":"b2e60dc6_bebb7a6d","updated":"2026-02-21 03:18:57.000000000","message":"Done","commit_id":"6210215ed827a4efd1fac2a87cf02897e366f40e"}],"cinder/templates/deployment-backup.yaml":[{"author":{"_account_id":7156,"name":"Mathieu Gagné","email":"mgagne@calavera.ca","username":"mgagne"},"change_message_id":"223c812f67d29e15b7168a998caa56559e56044a","unresolved":true,"context_lines":[{"line_number":171,"context_line":"            - name: oslo-lock-path"},{"line_number":172,"context_line":"              mountPath: {{ .Values.conf.cinder.oslo_concurrency.lock_path }}"},{"line_number":173,"context_line":"            - name: cinder-tmp"},{"line_number":174,"context_line":"              mountPath: /var/lib/cinder/tmp"},{"line_number":175,"context_line":"            - name: cinder-bin"},{"line_number":176,"context_line":"              mountPath: /tmp/cinder-backup.sh"},{"line_number":177,"context_line":"              subPath: cinder-backup.sh"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1ab6599c_f5ee1d12","line":174,"updated":"2026-02-23 18:24:42.000000000","message":"This is a duplicate of the above.","commit_id":"c707830adc700a2b2cf5d99a7dd8ea7fc6bdba6b"},{"author":{"_account_id":7156,"name":"Mathieu Gagné","email":"mgagne@calavera.ca","username":"mgagne"},"change_message_id":"81c22bec2e03c1765bc50ebf776fc4ca9ce395d0","unresolved":false,"context_lines":[{"line_number":171,"context_line":"            - name: oslo-lock-path"},{"line_number":172,"context_line":"              mountPath: {{ .Values.conf.cinder.oslo_concurrency.lock_path }}"},{"line_number":173,"context_line":"            - name: cinder-tmp"},{"line_number":174,"context_line":"              mountPath: /var/lib/cinder/tmp"},{"line_number":175,"context_line":"            - name: cinder-bin"},{"line_number":176,"context_line":"              mountPath: /tmp/cinder-backup.sh"},{"line_number":177,"context_line":"              subPath: cinder-backup.sh"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"6567da17_af030ebf","line":174,"in_reply_to":"1ab6599c_f5ee1d12","updated":"2026-02-24 14:40:39.000000000","message":"Done","commit_id":"c707830adc700a2b2cf5d99a7dd8ea7fc6bdba6b"}],"designate/templates/deployment-api.yaml":[{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"aad058eb6a423929b9485a521c7a608cfadf68a6","unresolved":true,"context_lines":[{"line_number":77,"context_line":"              scheme: {{ tuple \"dns\" \"internal\" \"api\" . | include \"helm-toolkit.endpoints.keystone_endpoint_scheme_lookup\" | upper }}"},{"line_number":78,"context_line":"              path: /"},{"line_number":79,"context_line":"              port: {{ tuple \"dns\" \"internal\" \"api\" . | include \"helm-toolkit.endpoints.endpoint_port_lookup\" }}"},{"line_number":80,"context_line":"          volumeMounts:"},{"line_number":81,"context_line":"            - name: designate-bin"},{"line_number":82,"context_line":"              mountPath: /tmp/designate-api.sh"},{"line_number":83,"context_line":"              subPath: designate-api.sh"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"d6a4d33a_acb33316","line":80,"updated":"2026-02-20 00:09:25.000000000","message":"Why didn\u0027t you add volume mount here like in other templates?","commit_id":"78d6fbc58ff45b85174278b4eb3d761d118281b0"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"d3ccf54a58e99402a112bf8d210e984a1cd457bb","unresolved":true,"context_lines":[{"line_number":77,"context_line":"              scheme: {{ tuple \"dns\" \"internal\" \"api\" . | include \"helm-toolkit.endpoints.keystone_endpoint_scheme_lookup\" | upper }}"},{"line_number":78,"context_line":"              path: /"},{"line_number":79,"context_line":"              port: {{ tuple \"dns\" \"internal\" \"api\" . | include \"helm-toolkit.endpoints.endpoint_port_lookup\" }}"},{"line_number":80,"context_line":"          volumeMounts:"},{"line_number":81,"context_line":"            - name: designate-bin"},{"line_number":82,"context_line":"              mountPath: /tmp/designate-api.sh"},{"line_number":83,"context_line":"              subPath: designate-api.sh"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"f8f61f25_884953ee","line":80,"in_reply_to":"d6a4d33a_acb33316","updated":"2026-02-20 04:39:23.000000000","message":"Well it got really mechanical and I asked AI to double check me after I wrote a script and clearly I missed stuff and so did AI.","commit_id":"78d6fbc58ff45b85174278b4eb3d761d118281b0"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"b6b4543cef02aad3e1efbbbb289d2f03dde8df02","unresolved":false,"context_lines":[{"line_number":77,"context_line":"              scheme: {{ tuple \"dns\" \"internal\" \"api\" . | include \"helm-toolkit.endpoints.keystone_endpoint_scheme_lookup\" | upper }}"},{"line_number":78,"context_line":"              path: /"},{"line_number":79,"context_line":"              port: {{ tuple \"dns\" \"internal\" \"api\" . | include \"helm-toolkit.endpoints.endpoint_port_lookup\" }}"},{"line_number":80,"context_line":"          volumeMounts:"},{"line_number":81,"context_line":"            - name: designate-bin"},{"line_number":82,"context_line":"              mountPath: /tmp/designate-api.sh"},{"line_number":83,"context_line":"              subPath: designate-api.sh"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3072483a_bee6f240","line":80,"in_reply_to":"f8f61f25_884953ee","updated":"2026-02-20 23:16:00.000000000","message":"Done","commit_id":"78d6fbc58ff45b85174278b4eb3d761d118281b0"}],"designate/templates/deployment-central.yaml":[{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"aad058eb6a423929b9485a521c7a608cfadf68a6","unresolved":true,"context_lines":[{"line_number":62,"context_line":"          command:"},{"line_number":63,"context_line":"            - bash"},{"line_number":64,"context_line":"            - /tmp/designate-central.sh"},{"line_number":65,"context_line":"          volumeMounts:"},{"line_number":66,"context_line":"            - name: designate-bin"},{"line_number":67,"context_line":"              mountPath: /tmp/designate-central.sh"},{"line_number":68,"context_line":"              subPath: designate-central.sh"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"872abab9_10433b35","line":65,"updated":"2026-02-20 00:09:25.000000000","message":"ditto. there is no empty dir volume mount like in other templates.","commit_id":"78d6fbc58ff45b85174278b4eb3d761d118281b0"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"b6b4543cef02aad3e1efbbbb289d2f03dde8df02","unresolved":false,"context_lines":[{"line_number":62,"context_line":"          command:"},{"line_number":63,"context_line":"            - bash"},{"line_number":64,"context_line":"            - /tmp/designate-central.sh"},{"line_number":65,"context_line":"          volumeMounts:"},{"line_number":66,"context_line":"            - name: designate-bin"},{"line_number":67,"context_line":"              mountPath: /tmp/designate-central.sh"},{"line_number":68,"context_line":"              subPath: designate-central.sh"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"76ac8767_9aed01fb","line":65,"in_reply_to":"872abab9_10433b35","updated":"2026-02-20 23:16:00.000000000","message":"Done","commit_id":"78d6fbc58ff45b85174278b4eb3d761d118281b0"}],"glance/templates/deployment-api.yaml":[{"author":{"_account_id":7156,"name":"Mathieu Gagné","email":"mgagne@calavera.ca","username":"mgagne"},"change_message_id":"223c812f67d29e15b7168a998caa56559e56044a","unresolved":true,"context_lines":[{"line_number":194,"context_line":"            - name: oslo-lock-path"},{"line_number":195,"context_line":"              mountPath: {{ .Values.conf.glance.oslo_concurrency.lock_path }}"},{"line_number":196,"context_line":"            - name: glance-tmp"},{"line_number":197,"context_line":"              mountPath: /var/lib/glance/tmp"},{"line_number":198,"context_line":"            - name: etcglance"},{"line_number":199,"context_line":"              mountPath: /etc/glance"},{"line_number":200,"context_line":"            - name: glance-bin"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"0aa8a19a_6c398486","line":197,"updated":"2026-02-23 18:24:42.000000000","message":"This is a duplicate of the above.","commit_id":"c707830adc700a2b2cf5d99a7dd8ea7fc6bdba6b"},{"author":{"_account_id":7156,"name":"Mathieu Gagné","email":"mgagne@calavera.ca","username":"mgagne"},"change_message_id":"81c22bec2e03c1765bc50ebf776fc4ca9ce395d0","unresolved":false,"context_lines":[{"line_number":194,"context_line":"            - name: oslo-lock-path"},{"line_number":195,"context_line":"              mountPath: {{ .Values.conf.glance.oslo_concurrency.lock_path }}"},{"line_number":196,"context_line":"            - name: glance-tmp"},{"line_number":197,"context_line":"              mountPath: /var/lib/glance/tmp"},{"line_number":198,"context_line":"            - name: etcglance"},{"line_number":199,"context_line":"              mountPath: /etc/glance"},{"line_number":200,"context_line":"            - name: glance-bin"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"dcf306d0_568a8ac2","line":197,"in_reply_to":"0aa8a19a_6c398486","updated":"2026-02-24 14:40:39.000000000","message":"Done","commit_id":"c707830adc700a2b2cf5d99a7dd8ea7fc6bdba6b"}],"octavia/templates/daemonset-health-manager.yaml":[{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"aad058eb6a423929b9485a521c7a608cfadf68a6","unresolved":true,"context_lines":[{"line_number":105,"context_line":"                command:"},{"line_number":106,"context_line":"                  - /tmp/octavia-health-manager.sh"},{"line_number":107,"context_line":"                  - stop"},{"line_number":108,"context_line":"          volumeMounts:"},{"line_number":109,"context_line":"            - name: pod-etc-octavia"},{"line_number":110,"context_line":"              mountPath: /etc/octavia"},{"line_number":111,"context_line":"            - name: octavia-bin"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"387ad909_b5cbed41","line":108,"updated":"2026-02-20 00:09:25.000000000","message":"ditto. there is no empty dir volume mount like in other templates","commit_id":"78d6fbc58ff45b85174278b4eb3d761d118281b0"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"b6b4543cef02aad3e1efbbbb289d2f03dde8df02","unresolved":false,"context_lines":[{"line_number":105,"context_line":"                command:"},{"line_number":106,"context_line":"                  - /tmp/octavia-health-manager.sh"},{"line_number":107,"context_line":"                  - stop"},{"line_number":108,"context_line":"          volumeMounts:"},{"line_number":109,"context_line":"            - name: pod-etc-octavia"},{"line_number":110,"context_line":"              mountPath: /etc/octavia"},{"line_number":111,"context_line":"            - name: octavia-bin"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"a06f8105_55f5c929","line":108,"in_reply_to":"387ad909_b5cbed41","updated":"2026-02-20 23:16:00.000000000","message":"Done","commit_id":"78d6fbc58ff45b85174278b4eb3d761d118281b0"}],"tacker/templates/deployment-conductor.yaml":[{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"aad058eb6a423929b9485a521c7a608cfadf68a6","unresolved":true,"context_lines":[{"line_number":63,"context_line":"            - /bin/bash"},{"line_number":64,"context_line":"            - -c"},{"line_number":65,"context_line":"            - /tmp/tacker-conductor.sh"},{"line_number":66,"context_line":"          volumeMounts:"},{"line_number":67,"context_line":"          - name: localtime"},{"line_number":68,"context_line":"            mountPath: \"/etc/localtime\""},{"line_number":69,"context_line":"            readOnly: yes"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9f188396_a76129cc","line":66,"updated":"2026-02-20 00:09:25.000000000","message":"there is no volume mount like in most templates","commit_id":"78d6fbc58ff45b85174278b4eb3d761d118281b0"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"b6b4543cef02aad3e1efbbbb289d2f03dde8df02","unresolved":false,"context_lines":[{"line_number":63,"context_line":"            - /bin/bash"},{"line_number":64,"context_line":"            - -c"},{"line_number":65,"context_line":"            - /tmp/tacker-conductor.sh"},{"line_number":66,"context_line":"          volumeMounts:"},{"line_number":67,"context_line":"          - name: localtime"},{"line_number":68,"context_line":"            mountPath: \"/etc/localtime\""},{"line_number":69,"context_line":"            readOnly: yes"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"430c6883_efc646cb","line":66,"in_reply_to":"9f188396_a76129cc","updated":"2026-02-20 23:16:00.000000000","message":"Done","commit_id":"78d6fbc58ff45b85174278b4eb3d761d118281b0"}]}
