)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"6414510c216d1199ebe837daa028a3752ea01bef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e42142f0_34cc4cf6","updated":"2022-11-25 08:23:25.000000000","message":"Sounds ok, till we have get the build-test-packages support in container and image build.","commit_id":"6dbbaf0d33776ad821cd7966b131b46a3e6b9a26"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"c93e51921145f756175e9f0c3204f7af90205026","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"76c5d0a0_1e130f7b","updated":"2022-11-24 11:50:33.000000000","message":"Thanks Rabi for the fix, this patch looks good to me\n\nAdding context for other reviewers.\n\nWe recently hit bug[0] in the container build job, Patch[1] brought the issue to notice as we changed that playbook to use the ansible controller interpreter rather than discovering it.\n\nTo fix the issue we merged patches [2] and [3] that add `buildcontainers_venv: false` and use system packages(rpm content) instead of using venv. Moving away from venv bring another issue into notice that depends-on with container-build job doesn\u0027t work(as currently we don\u0027t include `build-test-package` role in container-build job, more details in [4]) \n\nWith this patch of venv using --site-packages, we can go back to `buildcontainers_venv: true` if we want to, Rabi already tested that(details in [5]).\n\n\n[0] https://bugs.launchpad.net/tripleo/+bug/1996874\n[1] https://review.opendev.org/c/openstack/python-tripleoclient/+/864647/3/tripleoclient/v2/tripleo_container_image.py\n[2] https://review.rdoproject.org/r/c/config/+/46139\n[3] https://review.rdoproject.org/r/c/rdo-jobs/+/46141\n[4] https://bugs.launchpad.net/tripleo/+bug/1997539/\n[5] https://bugs.launchpad.net/tripleo/+bug/1997539/comments/2","commit_id":"6dbbaf0d33776ad821cd7966b131b46a3e6b9a26"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"08f25669f94ff9910961d5ca340be13ea2447767","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fd4ee03d_f7233c11","in_reply_to":"e42142f0_34cc4cf6","updated":"2022-11-25 08:43:30.000000000","message":"Just want to clarify that irrespective of if we go back to `buildcontainers_venv: true` or not for container-build, this patch is the right thing to do to fix the workflow. We will not need the revert of this patch even if we add build-test-packages support in the container and image build job.","commit_id":"6dbbaf0d33776ad821cd7966b131b46a3e6b9a26"}]}