)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},"change_message_id":"d3e9b51e97172a036f62292043676d0f1fa09a36","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d5800c82_fad6f589","updated":"2021-10-11 07:48:41.000000000","message":"Thanks, LGTM now","commit_id":"fdca31a2009a0aaf3f3ee9c5e30083ac59bf067f"}],"manifests/profile/pacemaker/rabbitmq_bundle.pp":[{"author":{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},"change_message_id":"ac084ca11c5c1362fbd53836a972b00fd98b6581","unresolved":true,"context_lines":[{"line_number":328,"context_line":"          },"},{"line_number":329,"context_line":"          \u0027rabbitmq-pki-cafile\u0027 \u003d\u003e {"},{"line_number":330,"context_line":"            \u0027source-dir\u0027 \u003d\u003e $rabbitmq_cacert,"},{"line_number":331,"context_line":"            \u0027target-dir\u0027 \u003d\u003e \"/var/lib/kolla/config_files/src-tls${rabbitmq_cacert}\","},{"line_number":332,"context_line":"            \u0027options\u0027    \u003d\u003e \u0027ro\u0027,"},{"line_number":333,"context_line":"          },"},{"line_number":334,"context_line":"        }"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"a604c80e_fa71276f","line":331,"updated":"2021-10-11 07:32:02.000000000","message":"What happens if rabbitmq_cacert falls back to /etc/ipa/ca.crt and this file doesn\u0027t exist?","commit_id":"fdca31a2009a0aaf3f3ee9c5e30083ac59bf067f"},{"author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"change_message_id":"7d8bb35089e8283ede55a037813d07253f79aa42","unresolved":true,"context_lines":[{"line_number":328,"context_line":"          },"},{"line_number":329,"context_line":"          \u0027rabbitmq-pki-cafile\u0027 \u003d\u003e {"},{"line_number":330,"context_line":"            \u0027source-dir\u0027 \u003d\u003e $rabbitmq_cacert,"},{"line_number":331,"context_line":"            \u0027target-dir\u0027 \u003d\u003e \"/var/lib/kolla/config_files/src-tls${rabbitmq_cacert}\","},{"line_number":332,"context_line":"            \u0027options\u0027    \u003d\u003e \u0027ro\u0027,"},{"line_number":333,"context_line":"          },"},{"line_number":334,"context_line":"        }"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"b144b446_5d977f65","line":331,"in_reply_to":"a604c80e_fa71276f","updated":"2021-10-11 07:38:24.000000000","message":"So I have to set this default to be able to pass CI because /etc/ipa/ca.crt will be defined in the tht patch of this patchset. Once THT change lands I could add a few more guards in https://review.opendev.org/c/openstack/puppet-tripleo/+/812401\nNote that all of this is under the if internal tls condition so it\u0027s unset in non tls-e cases","commit_id":"fdca31a2009a0aaf3f3ee9c5e30083ac59bf067f"},{"author":{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},"change_message_id":"d3e9b51e97172a036f62292043676d0f1fa09a36","unresolved":false,"context_lines":[{"line_number":328,"context_line":"          },"},{"line_number":329,"context_line":"          \u0027rabbitmq-pki-cafile\u0027 \u003d\u003e {"},{"line_number":330,"context_line":"            \u0027source-dir\u0027 \u003d\u003e $rabbitmq_cacert,"},{"line_number":331,"context_line":"            \u0027target-dir\u0027 \u003d\u003e \"/var/lib/kolla/config_files/src-tls${rabbitmq_cacert}\","},{"line_number":332,"context_line":"            \u0027options\u0027    \u003d\u003e \u0027ro\u0027,"},{"line_number":333,"context_line":"          },"},{"line_number":334,"context_line":"        }"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"8436857d_6fe8b17a","line":331,"in_reply_to":"b144b446_5d977f65","updated":"2021-10-11 07:48:41.000000000","message":"Done","commit_id":"fdca31a2009a0aaf3f3ee9c5e30083ac59bf067f"}]}