)]}'
{"helm-toolkit/templates/snippets/_kubernetes_metadata_labels.tpl":[{"author":{"_account_id":17440,"name":"Marlin Cremers","display_name":"Marlin Cremers","email":"marlin@cbws.nl","username":"mcremers"},"change_message_id":"bc2e319e93d4baf909db426f8a5f007e9a267feb","unresolved":true,"context_lines":[{"line_number":32,"context_line":"release_group: {{ $envAll.Values.release_group | default $envAll.Release.Name }}"},{"line_number":33,"context_line":"application: {{ $application }}"},{"line_number":34,"context_line":"component: {{ $component }}"},{"line_number":35,"context_line":"{{- if $envAll.Values.pod.labels }}"},{"line_number":36,"context_line":"{{ $envAll.Values.pod.labels | toYaml }}"},{"line_number":37,"context_line":"{{- end -}}"},{"line_number":38,"context_line":"{{- end -}}"}],"source_content_type":"text/x-smarty","patch_set":1,"id":"ac607b2d_ade4f9fe","line":35,"updated":"2021-09-23 14:18:51.000000000","message":"I\u0027m not sure if this is the right place in the values tree to put the configuration, if anyone knows a better place just let me know.","commit_id":"422c5723d2615d3798461b89ef49d58d4cd0ffb2"},{"author":{"_account_id":17440,"name":"Marlin Cremers","display_name":"Marlin Cremers","email":"marlin@cbws.nl","username":"mcremers"},"change_message_id":"2425f68d7ad02a4448011653b7cfeef96e151362","unresolved":false,"context_lines":[{"line_number":32,"context_line":"release_group: {{ $envAll.Values.release_group | default $envAll.Release.Name }}"},{"line_number":33,"context_line":"application: {{ $application }}"},{"line_number":34,"context_line":"component: {{ $component }}"},{"line_number":35,"context_line":"{{- if $envAll.Values.pod.labels }}"},{"line_number":36,"context_line":"{{ $envAll.Values.pod.labels | toYaml }}"},{"line_number":37,"context_line":"{{- end -}}"},{"line_number":38,"context_line":"{{- end -}}"}],"source_content_type":"text/x-smarty","patch_set":1,"id":"8e0216b4_2be54d00","line":35,"in_reply_to":"ac607b2d_ade4f9fe","updated":"2021-09-23 16:48:44.000000000","message":"Done","commit_id":"422c5723d2615d3798461b89ef49d58d4cd0ffb2"},{"author":{"_account_id":33519,"name":"Tin Lam","email":"tinlam@outlook.com","username":"tlam"},"change_message_id":"a3b310522f6843b477908dd90c38fbb39bd3a590","unresolved":true,"context_lines":[{"line_number":19,"context_line":"  release_group: null"},{"line_number":20,"context_line":"  pod:"},{"line_number":21,"context_line":"    labels:"},{"line_number":22,"context_line":"      default:"},{"line_number":23,"context_line":"        label1.example.com: value"},{"line_number":24,"context_line":"      bar:"},{"line_number":25,"context_line":"        label2.example.com: bar"},{"line_number":26,"context_line":"usage: |"},{"line_number":27,"context_line":"  {{ tuple . \"foo\" \"bar\" | include \"helm-toolkit.snippets.kubernetes_metadata_labels\" }}"}],"source_content_type":"text/x-smarty","patch_set":8,"id":"080c91b3_3efb8fe2","line":24,"range":{"start_line":22,"start_character":0,"end_line":24,"end_character":10},"updated":"2021-09-27 17:50:54.000000000","message":"do we need the default: and bar: keys. I don\u0027t see the need to differentiate them as they just all get consolidated to labels","commit_id":"4340e272d7c83abce0b5c597999e4bab758cad45"},{"author":{"_account_id":17440,"name":"Marlin Cremers","display_name":"Marlin Cremers","email":"marlin@cbws.nl","username":"mcremers"},"change_message_id":"0473bc59ef3037b24d28cf73559b40c38b2c6496","unresolved":true,"context_lines":[{"line_number":19,"context_line":"  release_group: null"},{"line_number":20,"context_line":"  pod:"},{"line_number":21,"context_line":"    labels:"},{"line_number":22,"context_line":"      default:"},{"line_number":23,"context_line":"        label1.example.com: value"},{"line_number":24,"context_line":"      bar:"},{"line_number":25,"context_line":"        label2.example.com: bar"},{"line_number":26,"context_line":"usage: |"},{"line_number":27,"context_line":"  {{ tuple . \"foo\" \"bar\" | include \"helm-toolkit.snippets.kubernetes_metadata_labels\" }}"}],"source_content_type":"text/x-smarty","patch_set":8,"id":"4d5b9a04_02d1231d","line":24,"range":{"start_line":22,"start_character":0,"end_line":24,"end_character":10},"in_reply_to":"080c91b3_3efb8fe2","updated":"2021-09-27 17:56:24.000000000","message":"The difference between them is that default (maybe I should instead name it something like common?) get applied to all resources that use this snippet while bar only to the bar component.\n\nSo practically you could for example allow Istio sidecar ingestion on only api pods (and not jobs). While adding the Istio app label to all resources.","commit_id":"4340e272d7c83abce0b5c597999e4bab758cad45"},{"author":{"_account_id":33519,"name":"Tin Lam","email":"tinlam@outlook.com","username":"tlam"},"change_message_id":"b38de66e17b5d0530cdbe231ddb9c47e574a6973","unresolved":true,"context_lines":[{"line_number":19,"context_line":"  release_group: null"},{"line_number":20,"context_line":"  pod:"},{"line_number":21,"context_line":"    labels:"},{"line_number":22,"context_line":"      default:"},{"line_number":23,"context_line":"        label1.example.com: value"},{"line_number":24,"context_line":"      bar:"},{"line_number":25,"context_line":"        label2.example.com: bar"},{"line_number":26,"context_line":"usage: |"},{"line_number":27,"context_line":"  {{ tuple . \"foo\" \"bar\" | include \"helm-toolkit.snippets.kubernetes_metadata_labels\" }}"}],"source_content_type":"text/x-smarty","patch_set":8,"id":"912aedd8_5c755377","line":24,"range":{"start_line":22,"start_character":0,"end_line":24,"end_character":10},"in_reply_to":"4d5b9a04_02d1231d","updated":"2021-09-28 04:14:30.000000000","message":"Ah, I understand now. Would this make sense to use the top level key labels: in the Values.yaml?","commit_id":"4340e272d7c83abce0b5c597999e4bab758cad45"},{"author":{"_account_id":17440,"name":"Marlin Cremers","display_name":"Marlin Cremers","email":"marlin@cbws.nl","username":"mcremers"},"change_message_id":"730721b71de402f7c7390a091068676f0a5733cb","unresolved":true,"context_lines":[{"line_number":19,"context_line":"  release_group: null"},{"line_number":20,"context_line":"  pod:"},{"line_number":21,"context_line":"    labels:"},{"line_number":22,"context_line":"      default:"},{"line_number":23,"context_line":"        label1.example.com: value"},{"line_number":24,"context_line":"      bar:"},{"line_number":25,"context_line":"        label2.example.com: bar"},{"line_number":26,"context_line":"usage: |"},{"line_number":27,"context_line":"  {{ tuple . \"foo\" \"bar\" | include \"helm-toolkit.snippets.kubernetes_metadata_labels\" }}"}],"source_content_type":"text/x-smarty","patch_set":8,"id":"12e9f9cd_aff7525a","line":24,"range":{"start_line":22,"start_character":0,"end_line":24,"end_character":10},"in_reply_to":"912aedd8_5c755377","updated":"2021-09-28 09:03:28.000000000","message":"In some way, I think it makes more sense than using pod.labels, however right now the labels section is used to define node selector labels so it could become confusing once we start combining the two.","commit_id":"4340e272d7c83abce0b5c597999e4bab758cad45"}],"releasenotes/notes/helm-toolkit.yaml":[{"author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"change_message_id":"07e588fbc261e38ae28a3002379369ea77a1cd0d","unresolved":true,"context_lines":[{"line_number":27,"context_line":"  - 0.2.18 Make Rabbit-init job more robust"},{"line_number":28,"context_line":"  - 0.2.19 Revoke all privileges for PUBLIC role in postgres dbs"},{"line_number":29,"context_line":"  - 0.2.20 Modify the template of rbac_role to make secrets accessible"},{"line_number":30,"context_line":"  - 0.2.11 Add ability to set common labels to add to all objects"},{"line_number":31,"context_line":"..."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"2000501d_f7ec599b","line":30,"range":{"start_line":30,"start_character":8,"end_line":30,"end_character":11},"updated":"2021-09-23 16:22:46.000000000","message":"Change this to 21, should fix the lint failure","commit_id":"5fff2cf9ed482a62663980de489bfdad181a9760"},{"author":{"_account_id":17440,"name":"Marlin Cremers","display_name":"Marlin Cremers","email":"marlin@cbws.nl","username":"mcremers"},"change_message_id":"2425f68d7ad02a4448011653b7cfeef96e151362","unresolved":false,"context_lines":[{"line_number":27,"context_line":"  - 0.2.18 Make Rabbit-init job more robust"},{"line_number":28,"context_line":"  - 0.2.19 Revoke all privileges for PUBLIC role in postgres dbs"},{"line_number":29,"context_line":"  - 0.2.20 Modify the template of rbac_role to make secrets accessible"},{"line_number":30,"context_line":"  - 0.2.11 Add ability to set common labels to add to all objects"},{"line_number":31,"context_line":"..."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5510e3ca_da869ff3","line":30,"range":{"start_line":30,"start_character":8,"end_line":30,"end_character":11},"in_reply_to":"2000501d_f7ec599b","updated":"2021-09-23 16:48:44.000000000","message":"Done","commit_id":"5fff2cf9ed482a62663980de489bfdad181a9760"}]}