)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32657,"name":"Piotr Parczewski","email":"piotr@stackhpc.com","username":"piotrp"},"change_message_id":"02545ab035000222cfd1a191de72194178e325b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9e39f412_6277e4b6","updated":"2022-05-17 13:11:02.000000000","message":"Missing role part","commit_id":"998300eba47d636d6b53175f9fb4e7153c1e2723"},{"author":{"_account_id":32657,"name":"Piotr Parczewski","email":"piotr@stackhpc.com","username":"piotrp"},"change_message_id":"541e7c01ed5fa04ab2ebb77a719f58b8dc73438e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3277a2e3_6e852c0c","updated":"2022-05-17 13:01:56.000000000","message":"check experimental","commit_id":"998300eba47d636d6b53175f9fb4e7153c1e2723"},{"author":{"_account_id":32657,"name":"Piotr Parczewski","email":"piotr@stackhpc.com","username":"piotrp"},"change_message_id":"2aa6a494167b4806e162febad7d996873c257b42","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"09f04c1b_91505514","updated":"2022-05-18 10:17:47.000000000","message":"check experimental","commit_id":"77e811e25c38f460aad21bd9c6428954fd64ddd5"}],"ansible/roles/prometheus/templates/prometheus-alertmanager.yml.j2":[{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"7127e5349389f59a38c89dc873ab01b108b8d013","unresolved":true,"context_lines":[{"line_number":18,"context_line":"            password: \u0027{{ keystone_admin_password }}\u0027"},{"line_number":19,"context_line":"{% endif %}"},{"line_number":20,"context_line":"templates:"},{"line_number":21,"context_line":"  - \u0027/etc/prometheus/*.tmpl\u0027"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"add7ce31_e47694ce","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":28},"updated":"2022-05-31 15:33:52.000000000","message":"I think this should be rendered as real paths to templates, it\u0027s usefull if user remove some template from custom config, so prometheus-alertmanager will be restarted and template will be ignored.\n\nNo it will include all. What do you think ?","commit_id":"77e811e25c38f460aad21bd9c6428954fd64ddd5"},{"author":{"_account_id":32657,"name":"Piotr Parczewski","email":"piotr@stackhpc.com","username":"piotrp"},"change_message_id":"6d7d66f3e2f9e64169c49449aa95e6e2ffc4fecd","unresolved":true,"context_lines":[{"line_number":18,"context_line":"            password: \u0027{{ keystone_admin_password }}\u0027"},{"line_number":19,"context_line":"{% endif %}"},{"line_number":20,"context_line":"templates:"},{"line_number":21,"context_line":"  - \u0027/etc/prometheus/*.tmpl\u0027"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"ab741583_758b4065","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":28},"in_reply_to":"add7ce31_e47694ce","updated":"2022-06-01 09:30:29.000000000","message":"I\u0027m not sure if I follow; it\u0027s more or less upstream example config https://github.com/prometheus/alertmanager/blob/main/doc/examples/simple.yml#L8-L10\nGives operators flexibility on the number and filenames for the templates. Could add a \"known issues\" reno about templates removed from config but remaining on the control plane after initial deployment. This is broader and known K-A issue though","commit_id":"77e811e25c38f460aad21bd9c6428954fd64ddd5"}]}