)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"72ae2d322d8a8023393013939cc03108dce31cf3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"19f40f77_962c41bd","updated":"2022-09-07 07:28:31.000000000","message":"Any idea why the original patch passed in Tenks CI yet failed in K-A CI?","commit_id":"8c8bc137b6094c8f0556fb4159845039c87b06bf"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"44c3e0925ba3f6269a726167df5d800ecbd1a203","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2b904266_5bc7875a","in_reply_to":"19f40f77_962c41bd","updated":"2022-09-07 07:38:30.000000000","message":"Seems tenks CI is not using ansible/hypervisor_setup.yml","commit_id":"8c8bc137b6094c8f0556fb4159845039c87b06bf"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"6063b3966d8ea9ed35f7a27a1c48ac960088ced5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f6259dab_77dc4de5","in_reply_to":"2b904266_5bc7875a","updated":"2022-09-07 08:33:15.000000000","message":"Odd, can @mgoddard explain?","commit_id":"8c8bc137b6094c8f0556fb4159845039c87b06bf"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"aeb1f1fcb2dd32bd76f1c6a001d6746650ee2ca1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bb3f5ce0_27e15a25","in_reply_to":"f6259dab_77dc4de5","updated":"2022-09-07 12:17:35.000000000","message":"Well, it is running, but it seems some of those things (like crb/epel repo enablement and so on) are in pre.","commit_id":"8c8bc137b6094c8f0556fb4159845039c87b06bf"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"aeb1f1fcb2dd32bd76f1c6a001d6746650ee2ca1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9c075f1e_d28753c4","updated":"2022-09-07 12:17:35.000000000","message":"I ","commit_id":"e0f4e8525090944417d6ec53e69c8f7dcea44549"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"16b958aebe2144c5a1a37ab8ee0caa395eba5e56","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d3662385_fbba138a","updated":"2022-09-07 12:31:18.000000000","message":"Or may we switch to venv and avoid the virtualenv drama that we don\u0027t need?","commit_id":"e0f4e8525090944417d6ec53e69c8f7dcea44549"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"4cbf5a3ed464372a0e895a045e9470bf756159bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a07beb92_d9c253c2","in_reply_to":"8d3f09b4_9bd34514","updated":"2022-09-07 16:01:30.000000000","message":"the patch is https://review.opendev.org/c/openstack/tenks/+/856296","commit_id":"e0f4e8525090944417d6ec53e69c8f7dcea44549"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"a63800942307c67b7e9e300ec7071c2138e6bfb9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8d3f09b4_9bd34514","in_reply_to":"d3662385_fbba138a","updated":"2022-09-07 15:19:03.000000000","message":"Here, and in the previous patch as well, I propose to simply move to venv and avoid all the virtualenv drama. I will post a relevant patch today...","commit_id":"e0f4e8525090944417d6ec53e69c8f7dcea44549"}],"ansible/group_vars/hypervisors":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"509eb6a80154113b2662b719aff741aed05a8c56","unresolved":true,"context_lines":[{"line_number":4,"context_line":"physnet_mappings: {}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"system_requirements:"},{"line_number":7,"context_line":"  - \"{% if ansible_facts.os_family !\u003d \u0027RedHat\u0027 and ansible_facts.distribution_major_version !\u003d \u00279\u0027 %}python3-virtualenv{% endif %}\""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"# Tenks bridge type. Options are \"openvswitch\", \"linuxbridge\".  Default is"},{"line_number":10,"context_line":"# \"openvswitch\". Note that this relates to bridges created by Tenks, not the"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"148b331d_a3e8202d","line":7,"updated":"2022-09-07 12:25:28.000000000","message":"Why is it not required on EL9?\n\nLogic not quite right - should be not (A and B)","commit_id":"e0f4e8525090944417d6ec53e69c8f7dcea44549"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"47db417d000ae3dcded1e199476d0acc757da8f1","unresolved":true,"context_lines":[{"line_number":4,"context_line":"physnet_mappings: {}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"system_requirements:"},{"line_number":7,"context_line":"  - \"{% if ansible_facts.os_family !\u003d \u0027RedHat\u0027 and ansible_facts.distribution_major_version !\u003d \u00279\u0027 %}python3-virtualenv{% endif %}\""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"# Tenks bridge type. Options are \"openvswitch\", \"linuxbridge\".  Default is"},{"line_number":10,"context_line":"# \"openvswitch\". Note that this relates to bridges created by Tenks, not the"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"778bd57a_88381b58","line":7,"in_reply_to":"148b331d_a3e8202d","updated":"2022-09-07 12:31:38.000000000","message":"Basically there\u0027s no python3-virtualenv on EL9 (well, there is one in EPEL, but who likes EPEL).","commit_id":"e0f4e8525090944417d6ec53e69c8f7dcea44549"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"d9c12dcc2ccb138008fdf9b3c1c93b81827aa013","unresolved":true,"context_lines":[{"line_number":4,"context_line":"physnet_mappings: {}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"system_requirements:"},{"line_number":7,"context_line":"  - \"{% if ansible_facts.os_family !\u003d \u0027RedHat\u0027 and ansible_facts.distribution_major_version !\u003d \u00279\u0027 %}python3-virtualenv{% endif %}\""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"# Tenks bridge type. Options are \"openvswitch\", \"linuxbridge\".  Default is"},{"line_number":10,"context_line":"# \"openvswitch\". Note that this relates to bridges created by Tenks, not the"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"b7f1aa23_65b8bb9d","line":7,"in_reply_to":"778bd57a_88381b58","updated":"2022-09-07 14:58:33.000000000","message":"Is the virtualenv command still available, or should we switch to python3 -m venv?","commit_id":"e0f4e8525090944417d6ec53e69c8f7dcea44549"}],"ansible/roles/virtualbmc-daemon/tasks/main.yml":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"509eb6a80154113b2662b719aff741aed05a8c56","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- name: Ensure package dependencies are installed (RedHat)"},{"line_number":13,"context_line":"  dnf:"},{"line_number":14,"context_line":"    enablerepo: \"{{ \u0027crb\u0027 if ansible_facts.distribution_major_version \u003d\u003d \u00279\u0027 else omit }}\""},{"line_number":15,"context_line":"    name: \"{{ vbmcd_packages }}\""},{"line_number":16,"context_line":"    state: present"},{"line_number":17,"context_line":"  register: result"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"bbef4671_afbc0be3","line":14,"updated":"2022-09-07 12:25:28.000000000","message":"You can use the package module, and omit this on non EL9 distros","commit_id":"e0f4e8525090944417d6ec53e69c8f7dcea44549"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"47db417d000ae3dcded1e199476d0acc757da8f1","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- name: Ensure package dependencies are installed (RedHat)"},{"line_number":13,"context_line":"  dnf:"},{"line_number":14,"context_line":"    enablerepo: \"{{ \u0027crb\u0027 if ansible_facts.distribution_major_version \u003d\u003d \u00279\u0027 else omit }}\""},{"line_number":15,"context_line":"    name: \"{{ vbmcd_packages }}\""},{"line_number":16,"context_line":"    state: present"},{"line_number":17,"context_line":"  register: result"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"0721678a_525997a0","line":14,"in_reply_to":"bbef4671_afbc0be3","updated":"2022-09-07 12:31:38.000000000","message":"Ack","commit_id":"e0f4e8525090944417d6ec53e69c8f7dcea44549"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"509eb6a80154113b2662b719aff741aed05a8c56","unresolved":true,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"- name: Ensure python3-devel is installed on EL9"},{"line_number":34,"context_line":"  package:"},{"line_number":35,"context_line":"    name: \"python3-devel\""},{"line_number":36,"context_line":"    state: present"},{"line_number":37,"context_line":"  register: result"},{"line_number":38,"context_line":"  until: result is success"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"27e86a68_2533c4d9","line":35,"updated":"2022-09-07 12:25:28.000000000","message":"Could we just add it to ansible/roles/virtualbmc-daemon/vars/RedHat.yml? Shouldn\u0027t be a problem to install on EL8","commit_id":"e0f4e8525090944417d6ec53e69c8f7dcea44549"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"47db417d000ae3dcded1e199476d0acc757da8f1","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"- name: Ensure python3-devel is installed on EL9"},{"line_number":34,"context_line":"  package:"},{"line_number":35,"context_line":"    name: \"python3-devel\""},{"line_number":36,"context_line":"    state: present"},{"line_number":37,"context_line":"  register: result"},{"line_number":38,"context_line":"  until: result is success"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"ae9e4caa_e8bf3326","line":35,"in_reply_to":"27e86a68_2533c4d9","updated":"2022-09-07 12:31:38.000000000","message":"Ok then, let\u0027s install that on EL8 as well.","commit_id":"e0f4e8525090944417d6ec53e69c8f7dcea44549"}]}