)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"189c427bd25d25731ef33c6338f888d9f3e24644","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8a121fbe_f351c23d","updated":"2023-03-01 13:57:09.000000000","message":"setting this to WIP as this is a fix for a non voting job (bifrost-integration-dibipa-debian-centos-9) and it could be missed if the CI is green","commit_id":"1e2a2d0ea03cbde6b9227ce037205c213b3943f4"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"fa149d8826de19e9e294998e2cf88ed7aad86b55","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"47ece2af_8a8a8e87","updated":"2023-03-02 15:30:20.000000000","message":"debootstrap package still not found  despite the epel repo being installed\nneed to troubleshoot more","commit_id":"27dc7edba54ca4f0488a75d3eae301ea2a774cf1"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"6a152066ddebff4a4afe58be98d1721361366641","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e43f354f_18e3d6b4","updated":"2023-03-03 15:56:43.000000000","message":"recheck EPEL is fixed, this was a timeout","commit_id":"47a35c762a55aef06e7e3b21ab3e754213730a38"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"bebf3c42c15aa778c7c43f803b4c40d11b93a479","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f2cf44e6_144f2f95","updated":"2023-03-06 11:05:01.000000000","message":"recheck and another timeout, cause why not","commit_id":"47a35c762a55aef06e7e3b21ab3e754213730a38"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"640f769ce7a3654bf962331d9f53f51b352d6ab7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"fc1c7622_d0d62cc1","updated":"2023-03-09 13:53:58.000000000","message":"All green, great job!","commit_id":"fb6d6d6fe9fce956ea1da3bbf924222f56692917"}],"playbooks/roles/bifrost-ironic-install/tasks/install.yml":[{"author":{"_account_id":25600,"name":"Ebbex","display_name":"ebbex","email":"openstack@slipsprogrammor.no","username":"ebbex"},"change_message_id":"c0dade9532347ca53a9923e5d1b9f49f459977ed","unresolved":true,"context_lines":[{"line_number":33,"context_line":"    when:"},{"line_number":34,"context_line":"      - enable_epel | bool"},{"line_number":35,"context_line":"      - ansible_distribution in [\"AlmaLinux\", \"CentOS\", \"Rocky\"]"},{"line_number":36,"context_line":"      - ansible_distribution_version|int \u003d\u003d 8"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"  - name: \"Enable EPEL repositories for EL9\""},{"line_number":39,"context_line":"    ansible.builtin.dnf:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"e31de338_e0bb9936","line":36,"updated":"2023-03-02 19:19:29.000000000","message":"Limiting this to will break for alma and rocky 9, when the next task is just centos.","commit_id":"27dc7edba54ca4f0488a75d3eae301ea2a774cf1"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ae91167396efc6b254af77bbd5777af35676f30f","unresolved":true,"context_lines":[{"line_number":43,"context_line":"      state: latest"},{"line_number":44,"context_line":"    when:"},{"line_number":45,"context_line":"      - enable_epel | bool"},{"line_number":46,"context_line":"      - ansible_distribution \u003d\u003d \u0027CentOS\u0027"},{"line_number":47,"context_line":"      - ansible_distribution_version|int \u003d\u003d 9"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"- name: \"Enable EPEL repository for RedHat 9\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"6184363a_5188dabf","line":46,"updated":"2023-03-02 13:00:52.000000000","message":"Isn\u0027t it the same for Alma/Rocky?","commit_id":"27dc7edba54ca4f0488a75d3eae301ea2a774cf1"},{"author":{"_account_id":25600,"name":"Ebbex","display_name":"ebbex","email":"openstack@slipsprogrammor.no","username":"ebbex"},"change_message_id":"c0dade9532347ca53a9923e5d1b9f49f459977ed","unresolved":true,"context_lines":[{"line_number":43,"context_line":"      state: latest"},{"line_number":44,"context_line":"    when:"},{"line_number":45,"context_line":"      - enable_epel | bool"},{"line_number":46,"context_line":"      - ansible_distribution \u003d\u003d \u0027CentOS\u0027"},{"line_number":47,"context_line":"      - ansible_distribution_version|int \u003d\u003d 9"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"- name: \"Enable EPEL repository for RedHat 9\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9a336b9e_fc986981","line":46,"in_reply_to":"239b7ab3_68348d8c","updated":"2023-03-02 19:19:29.000000000","message":"Seems what you want is to just install epel-next-release on centos-9, and leave the normal epel-next as is.","commit_id":"27dc7edba54ca4f0488a75d3eae301ea2a774cf1"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"a31fe1d9b3238f356967e693c542bb69135c48fb","unresolved":true,"context_lines":[{"line_number":43,"context_line":"      state: latest"},{"line_number":44,"context_line":"    when:"},{"line_number":45,"context_line":"      - enable_epel | bool"},{"line_number":46,"context_line":"      - ansible_distribution \u003d\u003d \u0027CentOS\u0027"},{"line_number":47,"context_line":"      - ansible_distribution_version|int \u003d\u003d 9"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"- name: \"Enable EPEL repository for RedHat 9\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"239b7ab3_68348d8c","line":46,"in_reply_to":"6184363a_5188dabf","updated":"2023-03-02 13:49:40.000000000","message":"AFAIK Alma/Rocky do not have epel-next-release\nif we want to support them for EL9 then we\u0027ll have to create a separate entry","commit_id":"27dc7edba54ca4f0488a75d3eae301ea2a774cf1"},{"author":{"_account_id":25600,"name":"Ebbex","display_name":"ebbex","email":"openstack@slipsprogrammor.no","username":"ebbex"},"change_message_id":"c0dade9532347ca53a9923e5d1b9f49f459977ed","unresolved":true,"context_lines":[{"line_number":44,"context_line":"    when:"},{"line_number":45,"context_line":"      - enable_epel | bool"},{"line_number":46,"context_line":"      - ansible_distribution \u003d\u003d \u0027CentOS\u0027"},{"line_number":47,"context_line":"      - ansible_distribution_version|int \u003d\u003d 9"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"- name: \"Enable EPEL repository for RedHat 9\""},{"line_number":50,"context_line":"  command: dnf install -y https://dl.fedoraproject.org/pub/epel/epel-release-latest-9.noarch.rpm"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"d2dbba28_b2983466","line":47,"range":{"start_line":47,"start_character":8,"end_line":47,"end_character":40},"updated":"2023-03-02 19:19:29.000000000","message":"jinja-spacing","commit_id":"27dc7edba54ca4f0488a75d3eae301ea2a774cf1"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"aa9a93f7fb0f583368d55af10b1aae549796737c","unresolved":true,"context_lines":[{"line_number":26,"context_line":"    command: dnf config-manager --set-enabled crb"},{"line_number":27,"context_line":"    when: ansible_distribution_version|int \u003d\u003d 9"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"  - name: \"Remove old EPEL repositories\""},{"line_number":30,"context_line":"    ansible.builtin.dnf:"},{"line_number":31,"context_line":"      name:"},{"line_number":32,"context_line":"        - epel-release"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3896a884_8339d8b4","line":29,"updated":"2023-03-06 15:01:11.000000000","message":"If that\u0027s for the CI, please only do it in the CI (so, test-bifrost.sh or something).","commit_id":"0b972a5923524f28e30ae1a256045298a83b6e7b"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"9894c2717aa84387a26066c9a747c4fecfe3e9e9","unresolved":false,"context_lines":[{"line_number":26,"context_line":"    command: dnf config-manager --set-enabled crb"},{"line_number":27,"context_line":"    when: ansible_distribution_version|int \u003d\u003d 9"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"  - name: \"Remove old EPEL repositories\""},{"line_number":30,"context_line":"    ansible.builtin.dnf:"},{"line_number":31,"context_line":"      name:"},{"line_number":32,"context_line":"        - epel-release"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"67a5c90e_f9c6ace5","line":29,"in_reply_to":"3896a884_8339d8b4","updated":"2023-03-07 13:43:32.000000000","message":"Ack","commit_id":"0b972a5923524f28e30ae1a256045298a83b6e7b"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"d22d30705b1377ca08ad83794f9ef857f458c20c","unresolved":true,"context_lines":[{"line_number":35,"context_line":"    when:"},{"line_number":36,"context_line":"      - enable_epel | bool"},{"line_number":37,"context_line":"      - ansible_distribution in [\"AlmaLinux\", \"CentOS\", \"Rocky\"]"},{"line_number":38,"context_line":"      - remove_old_epel | bool"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"  - name: \"Enable EPEL repository for EL7/8\""},{"line_number":41,"context_line":"    ansible.builtin.dnf:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"11f5413d_070268ae","line":38,"updated":"2023-03-08 13:27:29.000000000","message":"Sorry, I did not mean adding a new option here (we should really reduce the number of options we have in bifrost, but that\u0027s another topic), but rather moving the complete thing out of the playbooks and somewhere in the CI scripts. Akin to hacks here https://opendev.org/openstack/bifrost/src/branch/master/scripts/test-bifrost.sh#L67-L81 or here https://opendev.org/openstack/bifrost/src/branch/master/scripts/install-deps.sh#L50","commit_id":"4459caf1e14b4940697548575a6994bb1ac5f15c"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"a3d157a3f5ca79a6038372205aadf1c02c95ca5a","unresolved":true,"context_lines":[{"line_number":35,"context_line":"    when:"},{"line_number":36,"context_line":"      - enable_epel | bool"},{"line_number":37,"context_line":"      - ansible_distribution in [\"AlmaLinux\", \"CentOS\", \"Rocky\"]"},{"line_number":38,"context_line":"      - remove_old_epel | bool"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"  - name: \"Enable EPEL repository for EL7/8\""},{"line_number":41,"context_line":"    ansible.builtin.dnf:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"c95d9206_f0922ed0","line":38,"in_reply_to":"11f5413d_070268ae","updated":"2023-03-08 13:29:56.000000000","message":"right, ok","commit_id":"4459caf1e14b4940697548575a6994bb1ac5f15c"}]}