)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":37313,"name":"Alma MAILLET-CONTOZ","display_name":"Alma MAILLET-CONTOZ","email":"alma.maillet-contoz@infomaniak.com","username":"nicolasmc","status":"Intern SRE at Infomaniak"},"change_message_id":"1d090a1ff9b7d6909270507102768ebec1ca160c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"072086b3_733dbe78","updated":"2026-09-02 13:27:04.000000000","message":"LGTM","commit_id":"ca291d19e78ed3792ebe5ba1f734354169d2781e"},{"author":{"_account_id":34034,"name":"Franciszek Przewoźny","display_name":"Franciszek Przewozny","email":"przewozny.franciszek@gmail.com","username":"fprzewozn","status":"OpenStack Engineer / SRE @ Opera Software"},"change_message_id":"ed1ced37f3df15388de74e1c4e1895c549ccbd59","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"87b93d0f_22fcf56d","in_reply_to":"072086b3_733dbe78","updated":"2026-09-03 05:41:51.000000000","message":"I\u0027ve updated commit message, please recheck.","commit_id":"ca291d19e78ed3792ebe5ba1f734354169d2781e"}],"ansible/roles/prometheus/tasks/precheck.yml":[{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"6340e5ad4873053b06495a62560f98b88f02b6e0","unresolved":true,"context_lines":[{"line_number":29,"context_line":"    that:"},{"line_number":30,"context_line":"      - prometheus_bcrypt_salt is defined"},{"line_number":31,"context_line":"      - prometheus_bcrypt_salt is string or prometheus_bcrypt_salt is vault_encrypted"},{"line_number":32,"context_line":"      - prometheus_bcrypt_salt | length \u003d\u003d 22"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"- name: Check that prometheus_password is correctly set"},{"line_number":35,"context_line":"  ansible.builtin.assert:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"46cf66df_4cb72cbd","line":32,"range":{"start_line":32,"start_character":33,"end_line":32,"end_character":45},"updated":"2026-09-01 23:42:07.000000000","message":"if prometheus_bcrypt_salt is vault_encrypted the length would be \u003e 22","commit_id":"ca291d19e78ed3792ebe5ba1f734354169d2781e"},{"author":{"_account_id":34034,"name":"Franciszek Przewoźny","display_name":"Franciszek Przewozny","email":"przewozny.franciszek@gmail.com","username":"fprzewozn","status":"OpenStack Engineer / SRE @ Opera Software"},"change_message_id":"58330d2abc28caceb27e81069a7059e43b4a786b","unresolved":true,"context_lines":[{"line_number":29,"context_line":"    that:"},{"line_number":30,"context_line":"      - prometheus_bcrypt_salt is defined"},{"line_number":31,"context_line":"      - prometheus_bcrypt_salt is string or prometheus_bcrypt_salt is vault_encrypted"},{"line_number":32,"context_line":"      - prometheus_bcrypt_salt | length \u003d\u003d 22"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"- name: Check that prometheus_password is correctly set"},{"line_number":35,"context_line":"  ansible.builtin.assert:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bebaf24b_64b41558","line":32,"range":{"start_line":32,"start_character":33,"end_line":32,"end_character":45},"in_reply_to":"46cf66df_4cb72cbd","updated":"2026-09-02 05:23:19.000000000","message":"Since https://github.com/ansible/ansible/pull/67823 has been implemented, Ansible is detecting correct length of those types of secrets (method itself even trims the trailing \\n if present). Also I have tested this change on 3 different environments that are using Ansible Vault secrets in passwords.yml","commit_id":"ca291d19e78ed3792ebe5ba1f734354169d2781e"}]}
