)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"c5e3cff0a7b39b362b73bc40162cb10f9f7cccd4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"190e1f7b_0706f34c","updated":"2021-10-26 09:10:17.000000000","message":"testing in https://review.rdoproject.org/r/c/testproject/+/36339","commit_id":"a7d696d4c306e70d595a0ab43824614b045c2b0d"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"6029a0cd5df88f67dd90f4ce0013f26ff2a50f41","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"32de0263_35373ace","updated":"2021-10-27 14:56:57.000000000","message":"recheck","commit_id":"e6cdeb7da02d5fbf0dc2180f38df419c8fc3feca"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"cd56dc7c269e94c416f55b06f705ba68145d98bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e60eccc7_3b5562a7","updated":"2021-10-27 09:43:02.000000000","message":"recheck","commit_id":"e6cdeb7da02d5fbf0dc2180f38df419c8fc3feca"},{"author":{"_account_id":32926,"name":"Jiri Podivin","display_name":"jpodivin","email":"jpodivin@redhat.com","username":"jpodivin"},"change_message_id":"b03156def785f4ad6a94b05e1e31b4a625ccc9e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"d8ad397a_7aeac044","updated":"2021-11-19 14:00:47.000000000","message":"As I have changed one of the files I can\u0027t review anymore. So I will add myself to the CC list.","commit_id":"3877173e15c1cbee285e66c13580b62026e8439a"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"cc30340416c4683187198237758aaad9995690a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"bbfb9d94_833309a0","updated":"2021-11-22 10:08:38.000000000","message":"Tetsted both branched and branchless project https://review.rdoproject.org/r/c/rdo-jobs/+/36852","commit_id":"3877173e15c1cbee285e66c13580b62026e8439a"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"9112340b214055dbeecf3b81a8aa65e21d4ffd3b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"4938d57b_04e96e72","updated":"2021-11-22 19:00:49.000000000","message":"great, thanks for help","commit_id":"3877173e15c1cbee285e66c13580b62026e8439a"}],"roles/build-test-packages/defaults/main.yml":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"fd86f90418fd1e5e3ac1ee35708790a663600315","unresolved":true,"context_lines":[{"line_number":91,"context_line":"  - ansible-collections/ansible.netcommon"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"artg_branchless_projects:"},{"line_number":94,"context_line":"  - x/tripleo-ipa"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9b34c507_c90c0502","line":94,"range":{"start_line":94,"start_character":14,"end_line":94,"end_character":17},"updated":"2021-10-28 07:50:39.000000000","message":"may be better to add all known branchless projects in one go?","commit_id":"e6cdeb7da02d5fbf0dc2180f38df419c8fc3feca"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"d128e472f6f0cbf812f49aff76b107f16f6d2667","unresolved":true,"context_lines":[{"line_number":91,"context_line":"  - ansible-collections/ansible.netcommon"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"artg_branchless_projects:"},{"line_number":94,"context_line":"  - x/tripleo-ipa"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"a2ded092_cc26a3f9","line":94,"range":{"start_line":94,"start_character":14,"end_line":94,"end_character":17},"in_reply_to":"9b34c507_c90c0502","updated":"2021-10-28 12:01:25.000000000","message":"yeah, will add those I know about.","commit_id":"e6cdeb7da02d5fbf0dc2180f38df419c8fc3feca"}],"roles/build-test-packages/tasks/dlrn-build.yml":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"fd86f90418fd1e5e3ac1ee35708790a663600315","unresolved":true,"context_lines":[{"line_number":7,"context_line":"  debug:"},{"line_number":8,"context_line":"    var: artg_release"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"- name: Set DLRN distro (legacy distgit branch naming)"},{"line_number":11,"context_line":"  set_fact:"},{"line_number":12,"context_line":"    artg_distro: \"rpm-{{ artg_release }}\""},{"line_number":13,"context_line":"  when: artg_release in [\u0027master\u0027, \u0027mitaka\u0027]"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"- name: Set DLRN distro (new stable branch naming scheme)"},{"line_number":16,"context_line":"  set_fact:"},{"line_number":17,"context_line":"    artg_distro: \"{{ artg_release }}-rdo\""},{"line_number":18,"context_line":"  when: artg_release not in [\u0027master\u0027, \u0027mitaka\u0027]"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"- name: Make sure /usr/share/rhn/RHN-ORG-TRUSTED-SSL-CERT exists (RHEL)"},{"line_number":21,"context_line":"  shell:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"374fabda_c773fa2d","line":18,"range":{"start_line":10,"start_character":0,"end_line":18,"end_character":48},"updated":"2021-10-28 07:50:39.000000000","message":"Also need to adapt distgit branch for branchless projects, in RDO side distgit branches are maintained for branchless projects and using rpm-master branch for such cases may have issues.","commit_id":"e6cdeb7da02d5fbf0dc2180f38df419c8fc3feca"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"d128e472f6f0cbf812f49aff76b107f16f6d2667","unresolved":true,"context_lines":[{"line_number":7,"context_line":"  debug:"},{"line_number":8,"context_line":"    var: artg_release"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"- name: Set DLRN distro (legacy distgit branch naming)"},{"line_number":11,"context_line":"  set_fact:"},{"line_number":12,"context_line":"    artg_distro: \"rpm-{{ artg_release }}\""},{"line_number":13,"context_line":"  when: artg_release in [\u0027master\u0027, \u0027mitaka\u0027]"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"- name: Set DLRN distro (new stable branch naming scheme)"},{"line_number":16,"context_line":"  set_fact:"},{"line_number":17,"context_line":"    artg_distro: \"{{ artg_release }}-rdo\""},{"line_number":18,"context_line":"  when: artg_release not in [\u0027master\u0027, \u0027mitaka\u0027]"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"- name: Make sure /usr/share/rhn/RHN-ORG-TRUSTED-SSL-CERT exists (RHEL)"},{"line_number":21,"context_line":"  shell:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"502e3ad4_5491674e","line":18,"range":{"start_line":10,"start_character":0,"end_line":18,"end_character":48},"in_reply_to":"374fabda_c773fa2d","updated":"2021-10-28 12:01:25.000000000","message":"So what should be distgit branches in case of x/tripleo-ipa for example?","commit_id":"e6cdeb7da02d5fbf0dc2180f38df419c8fc3feca"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"ecd257f182f03881b940623e8c4d941c2b6308db","unresolved":true,"context_lines":[{"line_number":7,"context_line":"  debug:"},{"line_number":8,"context_line":"    var: artg_release"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"- name: Set DLRN distro (legacy distgit branch naming)"},{"line_number":11,"context_line":"  set_fact:"},{"line_number":12,"context_line":"    artg_distro: \"rpm-{{ artg_release }}\""},{"line_number":13,"context_line":"  when: artg_release in [\u0027master\u0027, \u0027mitaka\u0027]"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"- name: Set DLRN distro (new stable branch naming scheme)"},{"line_number":16,"context_line":"  set_fact:"},{"line_number":17,"context_line":"    artg_distro: \"{{ artg_release }}-rdo\""},{"line_number":18,"context_line":"  when: artg_release not in [\u0027master\u0027, \u0027mitaka\u0027]"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"- name: Make sure /usr/share/rhn/RHN-ORG-TRUSTED-SSL-CERT exists (RHEL)"},{"line_number":21,"context_line":"  shell:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"67023c43_0a3255b4","line":18,"range":{"start_line":10,"start_character":0,"end_line":18,"end_character":48},"in_reply_to":"502e3ad4_5491674e","updated":"2021-10-28 13:37:39.000000000","message":"should be release specfic when deploying train(release: train or override-checkout: stable/train etc), train-rdo, for victoria: victoria-rdo, for master: rpm-master.","commit_id":"e6cdeb7da02d5fbf0dc2180f38df419c8fc3feca"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"97fa4b10288c221505d14a6423adc1deb28aeafe","unresolved":true,"context_lines":[{"line_number":17,"context_line":"      set_fact:"},{"line_number":18,"context_line":"        artg_distro: \"{{ release }}-rdo\""},{"line_number":19,"context_line":"      when:"},{"line_number":20,"context_line":"        - release is defined and release not in [\u0027master\u0027, \u0027mitaka\u0027]"},{"line_number":21,"context_line":"        - artg_change.project in artg_branchless_projects"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"- name: Set DLRN distro (new stable branch naming scheme)"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"b4511b93_767772af","line":20,"range":{"start_line":20,"start_character":60,"end_line":20,"end_character":66},"updated":"2021-11-22 08:18:19.000000000","message":"i think we can remove that one ;)","commit_id":"3877173e15c1cbee285e66c13580b62026e8439a"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"cc30340416c4683187198237758aaad9995690a5","unresolved":true,"context_lines":[{"line_number":17,"context_line":"      set_fact:"},{"line_number":18,"context_line":"        artg_distro: \"{{ release }}-rdo\""},{"line_number":19,"context_line":"      when:"},{"line_number":20,"context_line":"        - release is defined and release not in [\u0027master\u0027, \u0027mitaka\u0027]"},{"line_number":21,"context_line":"        - artg_change.project in artg_branchless_projects"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"- name: Set DLRN distro (new stable branch naming scheme)"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"3dda98c5_5d187002","line":20,"range":{"start_line":20,"start_character":60,"end_line":20,"end_character":66},"in_reply_to":"b4511b93_767772af","updated":"2021-11-22 10:08:38.000000000","message":"can be cleaned up follow up","commit_id":"3877173e15c1cbee285e66c13580b62026e8439a"}],"roles/build-test-packages/tasks/main.yml":[{"author":{"_account_id":32926,"name":"Jiri Podivin","display_name":"jpodivin","email":"jpodivin@redhat.com","username":"jpodivin"},"change_message_id":"9932a6f3a0ef7927a3fcbe81756f724c864c36d2","unresolved":true,"context_lines":[{"line_number":194,"context_line":"    cacheable: true"},{"line_number":195,"context_line":"  with_items: \"{{ zuul[\u0027items\u0027] }}\""},{"line_number":196,"context_line":"  when:"},{"line_number":197,"context_line":"    - zuul is defined"},{"line_number":198,"context_line":"    - \"\u0027change_url\u0027 in item\""},{"line_number":199,"context_line":"    - item.project.name not in artg_change_list|default([]) | map(attribute\u003d\u0027project\u0027) |list"},{"line_number":200,"context_line":"    - \u003e-"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"7da09552_52c7e694","line":197,"updated":"2021-11-18 11:32:29.000000000","message":"We could rewrite the condition to use the YAML folding scalar.[1]\n\nIt would unify the look somewhat, and get rid of the dashes. \nBut it\u0027s only a stylistic suggestion so I\u0027m fine either way.\n\n[1]https://yaml.org/spec/1.2.2/#23-scalars","commit_id":"e6cdeb7da02d5fbf0dc2180f38df419c8fc3feca"}]}