)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"50f2ada74e2d499c98a79aeec0db530804abe381","unresolved":true,"context_lines":[{"line_number":24,"context_line":"[2]. https://github.com/ansible/ansible/issues/67083"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Closes-Bug: #1977888"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Signed-off-by: Chandan Kumar (raukadah) \u003cchkumar@redhat.com\u003e"},{"line_number":29,"context_line":"Change-Id: Ib87eb7bbe6b523e3bf78c7f870cff96d5b5dd7e1"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"b778150e_9928234c","line":27,"updated":"2022-06-08 08:28:06.000000000","message":"testproject link: https://code.engineering.redhat.com/gerrit/c/testproject/+/398473\n\nperiodic-tripleo-centos-8-buildimage-overcloud-full-train https://review.rdoproject.org/zuul/build/4efea305fbcc41b9957f6e61569379fd : SUCCESS in 52m 30s","commit_id":"8b55ed57d5620f3a725c02633a3e605b9bc1fbba"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"9ead4cb464da06819e51ad038607962debf16471","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"4185c855_43f7b850","updated":"2022-06-08 08:16:54.000000000","message":"May be good to consider switching to packages instead of venv/source installs to avoid future issues like this? i think for containers build that is already supported. or there are some known issues for that?","commit_id":"8b55ed57d5620f3a725c02633a3e605b9bc1fbba"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"43d891fc9b0311265cf6ba2502904b47492ff9ff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"ff1e4193_a7f0434b","updated":"2022-06-09 05:48:54.000000000","message":"recheck ","commit_id":"8b55ed57d5620f3a725c02633a3e605b9bc1fbba"},{"author":{"_account_id":32458,"name":"Ananya","display_name":"Ananya Banerjee","email":"anbanerj@redhat.com","username":"frenzyfriday"},"change_message_id":"e2d3007cc4b01cda67c421d7412336cbd5445307","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"2e08cc6f_73e500a2","updated":"2022-06-08 08:11:28.000000000","message":"thanks! Waiting for tp: https://review.rdoproject.org/r/c/testproject/+/43512","commit_id":"8b55ed57d5620f3a725c02633a3e605b9bc1fbba"}],"roles/oooci-build-images/tasks/main.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"fe368d7338ca688c9ec415733de5d9df36bdd648","unresolved":true,"context_lines":[{"line_number":62,"context_line":"  package:"},{"line_number":63,"context_line":"    name: \"python*-tripleoclient\""},{"line_number":64,"context_line":"    state: present"},{"line_number":65,"context_line":"  become: true"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"- name: Generate build-images.sh script"},{"line_number":68,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"2c81269d_2f553f3d","side":"PARENT","line":65,"updated":"2022-06-08 08:28:56.000000000","message":"client is now installed there https://opendev.org/openstack/tripleo-ci/src/commit/a70a36db14e73008f5cfef17178170f2fbbc9b33/roles/tripleo-build-jobs-repos/tasks/main.yaml#L50\n\nhttps://logserver.rdoproject.org/12/43512/3/check/periodic-tripleo-centos-8-buildimage-overcloud-full-train/4efea30/job-output.txt\n\n2022-06-08 07:40:25.405225 | TASK [tripleo-build-jobs-repos : Install python3-tripleoclient python3-virtualenv]\n2022-06-08 07:44:16.692684 | primary | changed","commit_id":"947e8a93a865e16481d14a1dd9fe1f91216e1a8d"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"8d050c175bc05751ea362aa3887678397177f2cc","unresolved":false,"context_lines":[{"line_number":62,"context_line":"  package:"},{"line_number":63,"context_line":"    name: \"python*-tripleoclient\""},{"line_number":64,"context_line":"    state: present"},{"line_number":65,"context_line":"  become: true"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"- name: Generate build-images.sh script"},{"line_number":68,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"56fc4c40_7ff22541","side":"PARENT","line":65,"in_reply_to":"2c81269d_2f553f3d","updated":"2022-06-10 07:23:15.000000000","message":"Done","commit_id":"947e8a93a865e16481d14a1dd9fe1f91216e1a8d"}],"roles/oooci-build-images/tasks/pre.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"fe368d7338ca688c9ec415733de5d9df36bdd648","unresolved":true,"context_lines":[{"line_number":56,"context_line":"  become: true"},{"line_number":57,"context_line":"  when: python_v \u003d\u003d \"py2\""},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"- name: Use Python38 for EL8"},{"line_number":60,"context_line":"  when: ansible_distribution_major_version is version(8, \u0027\u003d\u003d\u0027)"},{"line_number":61,"context_line":"  become: true"},{"line_number":62,"context_line":"  block:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"7e7ed831_561ebda2","line":59,"updated":"2022-06-08 08:28:56.000000000","message":"looks ok per the commit message, just one thought. seems to be a lot of \u0027install python-foo\u0027 above this statement, so would it be better if this was at the top.\n\nfor example installing python3-setuptools may pull in a different version of python than the one we want?","commit_id":"8b55ed57d5620f3a725c02633a3e605b9bc1fbba"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"8d050c175bc05751ea362aa3887678397177f2cc","unresolved":false,"context_lines":[{"line_number":56,"context_line":"  become: true"},{"line_number":57,"context_line":"  when: python_v \u003d\u003d \"py2\""},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"- name: Use Python38 for EL8"},{"line_number":60,"context_line":"  when: ansible_distribution_major_version is version(8, \u0027\u003d\u003d\u0027)"},{"line_number":61,"context_line":"  become: true"},{"line_number":62,"context_line":"  block:"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"02493ad3_cb11ce77","line":59,"in_reply_to":"7e7ed831_561ebda2","updated":"2022-06-10 07:23:15.000000000","message":"at least this seems to work for now to unblock us so marking resolved","commit_id":"8b55ed57d5620f3a725c02633a3e605b9bc1fbba"}]}