)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"3783e93d3e24606a00c46102ecd1dd5fe9b87beb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"7e172e8a_a813fd0e","updated":"2022-01-31 12:43:15.000000000","message":"LGTM,thanks","commit_id":"915b32f1d340724f36cf4888791880e21e13517d"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"9174dfb9b4cb1b5f747a564dfe159d5b2e5cf384","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"c6188bec_ec8c42b3","updated":"2022-01-31 14:46:19.000000000","message":"Thanks for updating it. LGTM","commit_id":"41c31fec42a436e0d48619c933158db89a450c02"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"d68b4823aba9a87ef80021b4ce8d397347117f7c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"efaa7962_e45a6f1b","updated":"2022-02-01 00:11:27.000000000","message":"recheck","commit_id":"b10bfd80307f0c5c5d27e4dced7b90aac0b36e56"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"969a9d7619cd40cbe37f50c71efd118fa1d4c760","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"2607c677_f20ab58d","updated":"2022-02-01 07:49:34.000000000","message":"BM job still failing with this patch, need further work","commit_id":"cac255c9892494c246d0b2343b9a96942a9eec2d"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"75d2e351579ae4aa118bd20a9641761be8a50cfb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"1fb65a1b_4b33616f","updated":"2022-02-01 18:10:35.000000000","message":"Thanks Sandeep and Ronelle, LGTM","commit_id":"846e0f0763e8f1b5e5cc6642d522da5d5b1175f5"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"0a841c569c6590d211a4533d7f9d28f913775991","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"2aa89846_69414aa7","updated":"2022-02-01 22:57:35.000000000","message":"recheck","commit_id":"846e0f0763e8f1b5e5cc6642d522da5d5b1175f5"}],"roles/oooci-build-images/add_cert.yaml":[{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"296a41fde0f2c4b9a7d49cc8e39c11e8eca00ae0","unresolved":true,"context_lines":[{"line_number":27,"context_line":"- name: Add cert if it exists"},{"line_number":28,"context_line":"  shell: \u003e"},{"line_number":29,"context_line":"    virt-customize -a {{ tripleo_image_source | urlsplit(\u0027path\u0027) | basename }}"},{"line_number":30,"context_line":"    --upload /etc/pki/ca-trust/source/anchors/rh.crt:/etc/pki/ca-trust/source/anchors/rh.crt"},{"line_number":31,"context_line":"    --run-command \u0027update-ca-trust\u0027"},{"line_number":32,"context_line":"  args:"},{"line_number":33,"context_line":"    chdir: \"{{ workspace }}\""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"fa2c6e3a_f2f82599","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":92},"updated":"2022-01-31 09:18:53.000000000","message":"We can use rpm install for this cert, But this approach looks better as rpm have other dependencies.","commit_id":"b896bf296860f6c24e391c222c68bce12aa21206"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"e084ccfb98fe497d79d51069c0eab627defbf5f7","unresolved":false,"context_lines":[{"line_number":27,"context_line":"- name: Add cert if it exists"},{"line_number":28,"context_line":"  shell: \u003e"},{"line_number":29,"context_line":"    virt-customize -a {{ tripleo_image_source | urlsplit(\u0027path\u0027) | basename }}"},{"line_number":30,"context_line":"    --upload /etc/pki/ca-trust/source/anchors/rh.crt:/etc/pki/ca-trust/source/anchors/rh.crt"},{"line_number":31,"context_line":"    --run-command \u0027update-ca-trust\u0027"},{"line_number":32,"context_line":"  args:"},{"line_number":33,"context_line":"    chdir: \"{{ workspace }}\""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"07ce2f9f_569b3a94","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":92},"in_reply_to":"fa2c6e3a_f2f82599","updated":"2022-01-31 14:53:46.000000000","message":"Done","commit_id":"b896bf296860f6c24e391c222c68bce12aa21206"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"f8503bc0da1431c75cc212cb6def88664d245d6d","unresolved":true,"context_lines":[{"line_number":34,"context_line":"  environment:"},{"line_number":35,"context_line":"    LIBGUESTFS_BACKEND_SETTINGS: force_tcg"},{"line_number":36,"context_line":"    LIBGUESTFS_BACKEND: direct"},{"line_number":37,"context_line":"  when: cert_path.stat.exists|bool"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"7a64013f_1b8e7f74","line":37,"range":{"start_line":37,"start_character":0,"end_line":37,"end_character":34},"updated":"2022-01-31 12:39:01.000000000","message":"this should be first condition for running the whole file","commit_id":"915b32f1d340724f36cf4888791880e21e13517d"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"7f497eedf0ecc253042fdf88a8fd22b776918f30","unresolved":false,"context_lines":[{"line_number":34,"context_line":"  environment:"},{"line_number":35,"context_line":"    LIBGUESTFS_BACKEND_SETTINGS: force_tcg"},{"line_number":36,"context_line":"    LIBGUESTFS_BACKEND: direct"},{"line_number":37,"context_line":"  when: cert_path.stat.exists|bool"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3e63da93_8188ac4c","line":37,"range":{"start_line":37,"start_character":0,"end_line":37,"end_character":34},"in_reply_to":"6cb7d086_7c088d93","updated":"2022-01-31 13:33:07.000000000","message":"Done","commit_id":"915b32f1d340724f36cf4888791880e21e13517d"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"004d64680378ea00681c10edd50aa22d2b239a36","unresolved":true,"context_lines":[{"line_number":34,"context_line":"  environment:"},{"line_number":35,"context_line":"    LIBGUESTFS_BACKEND_SETTINGS: force_tcg"},{"line_number":36,"context_line":"    LIBGUESTFS_BACKEND: direct"},{"line_number":37,"context_line":"  when: cert_path.stat.exists|bool"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"6cb7d086_7c088d93","line":37,"range":{"start_line":37,"start_character":0,"end_line":37,"end_character":34},"in_reply_to":"7a64013f_1b8e7f74","updated":"2022-01-31 12:59:33.000000000","message":"+1 Good point.\n\nWe can run task \"Check if cert exits\" first and move everything under a block with when condition - cert_path.stat.exists|bool\n\nUpdating patch.","commit_id":"915b32f1d340724f36cf4888791880e21e13517d"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"f20d145a776a4e95772572b018d4c37a11cd5fa0","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- name: Check if cert exits"},{"line_number":3,"context_line":"  stat:"},{"line_number":4,"context_line":"    path: /etc/pki/ca-trust/source/anchors/rh.crt"},{"line_number":5,"context_line":"  register: cert_path"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"- name: Install neccessary rpm and customize image to push correct certs in Image."}],"source_content_type":"text/x-yaml","patch_set":7,"id":"e2231dac_a6539dce","line":4,"range":{"start_line":4,"start_character":11,"end_line":4,"end_character":49},"updated":"2022-01-31 14:45:58.000000000","message":"baremetal job - undercloud node will not have this rpm, for baremetal job correct path is /etc/pki/ca-trust/extracted/openssl/ca-bundle.trust.crt.\n\nWe need to check for this alternative path as well.","commit_id":"41c31fec42a436e0d48619c933158db89a450c02"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"e084ccfb98fe497d79d51069c0eab627defbf5f7","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- name: Check if cert exits"},{"line_number":3,"context_line":"  stat:"},{"line_number":4,"context_line":"    path: /etc/pki/ca-trust/source/anchors/rh.crt"},{"line_number":5,"context_line":"  register: cert_path"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"- name: Install neccessary rpm and customize image to push correct certs in Image."}],"source_content_type":"text/x-yaml","patch_set":7,"id":"54bab2e9_de5cbc91","line":4,"range":{"start_line":4,"start_character":11,"end_line":4,"end_character":49},"in_reply_to":"e2231dac_a6539dce","updated":"2022-01-31 14:53:46.000000000","message":"Done","commit_id":"41c31fec42a436e0d48619c933158db89a450c02"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"b8659360e1c4f7456d41ac3c8a7dd80726f98bc2","unresolved":true,"context_lines":[{"line_number":46,"context_line":"        LIBGUESTFS_BACKEND: direct"},{"line_number":47,"context_line":"      with_items: \"{{ found_repos.files }}\""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    - name: Install cert install rpm"},{"line_number":50,"context_line":"      shell: \u003e"},{"line_number":51,"context_line":"        virt-customize -a {{ tripleo_image_source | urlsplit(\u0027path\u0027) | basename }}"},{"line_number":52,"context_line":"        --run-command \u0027{{ downstream_cert_install_command }}\u0027"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"8a1be1ee_2cae441f","line":49,"range":{"start_line":49,"start_character":25,"end_line":49,"end_character":32},"updated":"2022-02-01 14:54:54.000000000","message":"typo.","commit_id":"be768a03a25d2a2a939e4476eadbb4ae27e67f33"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"156ad4e15eb5160ffeb8610946bbf60542316be7","unresolved":false,"context_lines":[{"line_number":46,"context_line":"        LIBGUESTFS_BACKEND: direct"},{"line_number":47,"context_line":"      with_items: \"{{ found_repos.files }}\""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    - name: Install cert install rpm"},{"line_number":50,"context_line":"      shell: \u003e"},{"line_number":51,"context_line":"        virt-customize -a {{ tripleo_image_source | urlsplit(\u0027path\u0027) | basename }}"},{"line_number":52,"context_line":"        --run-command \u0027{{ downstream_cert_install_command }}\u0027"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"2350e67f_83338bb8","line":49,"range":{"start_line":49,"start_character":25,"end_line":49,"end_character":32},"in_reply_to":"8a1be1ee_2cae441f","updated":"2022-02-01 15:19:15.000000000","message":"Done","commit_id":"be768a03a25d2a2a939e4476eadbb4ae27e67f33"}]}
