)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"6809dd7bf27ca51983a4d7a25f3d3dec18db785b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d22b3688_340d00aa","updated":"2021-12-02 15:59:20.000000000","message":"Thanks Marios,\n-1: just because there is more things to remove.","commit_id":"b78c7d314cf5a45201e50a7133914361f0cc0fd0"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"579d9c90463ef408dd262971db372f8077bc0464","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"59992bd5_676b8c81","updated":"2021-12-03 06:23:40.000000000","message":"One nit, rest looks good.","commit_id":"b6fdd2ee16565e6b2c001ec17b6a7a814e1fff1e"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"be4d758380721ef7919b655ac7a4d2be457eb0ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"275d2890_ccbc88cb","updated":"2021-12-10 11:51:39.000000000","message":"LGTM, thanks","commit_id":"2e6b614423f521e6d96a0143b5e45027e13f5f9f"}],"roles/tripleo-repos/defaults/main.yml":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"6809dd7bf27ca51983a4d7a25f3d3dec18db785b","unresolved":true,"context_lines":[{"line_number":6,"context_line":"fedora_mirror_host: \"https://mirrors.fedoraproject.org\""},{"line_number":7,"context_line":"rdo_mirror_host: \"https://trunk.rdoproject.org\""},{"line_number":8,"context_line":"override_repos: \"\""},{"line_number":9,"context_line":"# Url for the CentOS Compose for tripleo yum config module"},{"line_number":10,"context_line":"compose_url: \"https://composes.stream.centos.org/production/{{ compose_id }}/compose\""},{"line_number":11,"context_line":"# Particular compose id to use"},{"line_number":12,"context_line":"compose_id: \"latest-CentOS-Stream\""},{"line_number":13,"context_line":"rpm_packages:"},{"line_number":14,"context_line":"  - git"},{"line_number":15,"context_line":"  - python3-libselinux"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"f82e8431_49ce7902","line":12,"range":{"start_line":9,"start_character":0,"end_line":12,"end_character":34},"updated":"2021-12-02 15:59:20.000000000","message":"This is not needed anymore since you are removing compose-repos generation.","commit_id":"b78c7d314cf5a45201e50a7133914361f0cc0fd0"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1c6f39ad6cfb4ba5566efb7de3da1244dc719526","unresolved":false,"context_lines":[{"line_number":6,"context_line":"fedora_mirror_host: \"https://mirrors.fedoraproject.org\""},{"line_number":7,"context_line":"rdo_mirror_host: \"https://trunk.rdoproject.org\""},{"line_number":8,"context_line":"override_repos: \"\""},{"line_number":9,"context_line":"# Url for the CentOS Compose for tripleo yum config module"},{"line_number":10,"context_line":"compose_url: \"https://composes.stream.centos.org/production/{{ compose_id }}/compose\""},{"line_number":11,"context_line":"# Particular compose id to use"},{"line_number":12,"context_line":"compose_id: \"latest-CentOS-Stream\""},{"line_number":13,"context_line":"rpm_packages:"},{"line_number":14,"context_line":"  - git"},{"line_number":15,"context_line":"  - python3-libselinux"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7a4edd09_c698d519","line":12,"range":{"start_line":9,"start_character":0,"end_line":12,"end_character":34},"in_reply_to":"f82e8431_49ce7902","updated":"2021-12-02 16:02:57.000000000","message":"Ack","commit_id":"b78c7d314cf5a45201e50a7133914361f0cc0fd0"}],"roles/tripleo-repos/tasks/main.yml":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"6809dd7bf27ca51983a4d7a25f3d3dec18db785b","unresolved":true,"context_lines":[{"line_number":139,"context_line":"  register: result"},{"line_number":140,"context_line":"  when: ansible_distribution_major_version is version(8, \u0027\u003e\u0027)"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"- name: Print out the stdout of result"},{"line_number":143,"context_line":"  debug:"},{"line_number":144,"context_line":"    msg: \"{{ result }}\""},{"line_number":145,"context_line":""},{"line_number":146,"context_line":"- name: Update all installed packages after new repos are setup  # noqa package-latest"},{"line_number":147,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"d38c18f1_0e0077a9","side":"PARENT","line":144,"range":{"start_line":142,"start_character":0,"end_line":144,"end_character":23},"updated":"2021-12-02 15:59:20.000000000","message":"Still useful?","commit_id":"4bc91c7fa94594046e5781d75894a58e3f654c71"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1c6f39ad6cfb4ba5566efb7de3da1244dc719526","unresolved":false,"context_lines":[{"line_number":139,"context_line":"  register: result"},{"line_number":140,"context_line":"  when: ansible_distribution_major_version is version(8, \u0027\u003e\u0027)"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"- name: Print out the stdout of result"},{"line_number":143,"context_line":"  debug:"},{"line_number":144,"context_line":"    msg: \"{{ result }}\""},{"line_number":145,"context_line":""},{"line_number":146,"context_line":"- name: Update all installed packages after new repos are setup  # noqa package-latest"},{"line_number":147,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"c3a61bd0_526a9fcc","side":"PARENT","line":144,"range":{"start_line":142,"start_character":0,"end_line":144,"end_character":23},"in_reply_to":"d38c18f1_0e0077a9","updated":"2021-12-02 16:02:57.000000000","message":"yeah we have \u0027result\u0027 registered for the centos8 task above line 124","commit_id":"4bc91c7fa94594046e5781d75894a58e3f654c71"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"579d9c90463ef408dd262971db372f8077bc0464","unresolved":true,"context_lines":[{"line_number":122,"context_line":"          --no-stream"},{"line_number":123,"context_line":"    fi"},{"line_number":124,"context_line":"  register: result"},{"line_number":125,"context_line":"  when: ansible_distribution_major_version is version(8, \u0027\u003c\u003d\u0027)"},{"line_number":126,"context_line":"  changed_when: \"\u0027Installed:\u0027 in result.stdout_lines\""},{"line_number":127,"context_line":"  args:"},{"line_number":128,"context_line":"    warn: false"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"1ecdd565_aa221e2c","side":"PARENT","line":125,"range":{"start_line":125,"start_character":2,"end_line":125,"end_character":62},"updated":"2021-12-03 06:23:40.000000000","message":"We can also get rid of this conditional as it was added with https://review.opendev.org/c/openstack/tripleo-ci/+/800577/58/roles/tripleo-repos/tasks/main.yml","commit_id":"4bc91c7fa94594046e5781d75894a58e3f654c71"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"eb3844444676ad9bade0ce5cb3188d002d7791f1","unresolved":false,"context_lines":[{"line_number":122,"context_line":"          --no-stream"},{"line_number":123,"context_line":"    fi"},{"line_number":124,"context_line":"  register: result"},{"line_number":125,"context_line":"  when: ansible_distribution_major_version is version(8, \u0027\u003c\u003d\u0027)"},{"line_number":126,"context_line":"  changed_when: \"\u0027Installed:\u0027 in result.stdout_lines\""},{"line_number":127,"context_line":"  args:"},{"line_number":128,"context_line":"    warn: false"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"e43f29a5_71867402","side":"PARENT","line":125,"range":{"start_line":125,"start_character":2,"end_line":125,"end_character":62},"in_reply_to":"1ecdd565_aa221e2c","updated":"2021-12-10 11:46:26.000000000","message":"No we need to keep this... we don\u0027t want to run tripleo-repos for centos9 - we are using repo-setup per the referenced patch in the commit message","commit_id":"4bc91c7fa94594046e5781d75894a58e3f654c71"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"8908743ada4dd78d211468f5aef3f6dae1d3e8c7","unresolved":false,"context_lines":[{"line_number":122,"context_line":"          --no-stream"},{"line_number":123,"context_line":"    fi"},{"line_number":124,"context_line":"  register: result"},{"line_number":125,"context_line":"  when: ansible_distribution_major_version is version(8, \u0027\u003c\u003d\u0027)"},{"line_number":126,"context_line":"  changed_when: \"\u0027Installed:\u0027 in result.stdout_lines\""},{"line_number":127,"context_line":"  args:"},{"line_number":128,"context_line":"    warn: false"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"71e2e6a0_c95c74b7","side":"PARENT","line":125,"range":{"start_line":125,"start_character":2,"end_line":125,"end_character":62},"in_reply_to":"e43f29a5_71867402","updated":"2021-12-10 11:53:21.000000000","message":"Ok then, looks good.","commit_id":"4bc91c7fa94594046e5781d75894a58e3f654c71"}]}