)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f4c3de352ff3a262e345de51ab4c33eb8ff46ece","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Jorge San Emeterio \u003cjsanemet@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-02-21 11:10:00 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Changing \"python-devel\" to \"python3-devel\" on bindep test requirements for RPM based distros."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Although \"python-devel\" is a commonly accepted package among RPM distributions, it is not accepted by RHEL. Instead, I am changing it to the \"python3-devel\" alias which RHEL does have and is more common to find. This change will allow Zuul to install the package by itself. Versions after train also have this change made, which makes me think that it is something that has happened before."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ce7f965d_79077620","line":7,"updated":"2023-02-21 11:29:39.000000000","message":"unfortunetly train is required to support python 2.7 so we cannot remove python-devel in this case.\n\nyou can however add somethign specificaly for rhel to add python3-devel\nand filter out python-devel on rhel if you like.","commit_id":"85572d4e4b516728a6a7829fcac17d2b1114b24a"},{"author":{"_account_id":34443,"name":"Jorge San Emeterio","display_name":"jsanemet","email":"jsanemet@redhat.com","username":"jsanemet"},"change_message_id":"395b4523bb988e022c6d75cc24480260c14c3342","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Jorge San Emeterio \u003cjsanemet@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-02-21 11:10:00 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Changing \"python-devel\" to \"python3-devel\" on bindep test requirements for RPM based distros."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Although \"python-devel\" is a commonly accepted package among RPM distributions, it is not accepted by RHEL. Instead, I am changing it to the \"python3-devel\" alias which RHEL does have and is more common to find. This change will allow Zuul to install the package by itself. Versions after train also have this change made, which makes me think that it is something that has happened before."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"6698f183_a6a52cbb","line":7,"in_reply_to":"ce7f965d_79077620","updated":"2023-02-21 14:16:20.000000000","message":"I have updated the change so that it differentiates between python 2 and 3 based distributions. I have seen it work for RHEL-8, so I think this new filter should do the trick.","commit_id":"85572d4e4b516728a6a7829fcac17d2b1114b24a"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f4c3de352ff3a262e345de51ab4c33eb8ff46ece","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"62c78124_b1f23536","updated":"2023-02-21 11:29:39.000000000","message":"this also needs [stable-only] in the commit","commit_id":"85572d4e4b516728a6a7829fcac17d2b1114b24a"},{"author":{"_account_id":34443,"name":"Jorge San Emeterio","display_name":"jsanemet","email":"jsanemet@redhat.com","username":"jsanemet"},"change_message_id":"395b4523bb988e022c6d75cc24480260c14c3342","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"64922f41_f8f737ec","in_reply_to":"62c78124_b1f23536","updated":"2023-02-21 14:16:20.000000000","message":"Done.","commit_id":"85572d4e4b516728a6a7829fcac17d2b1114b24a"},{"author":{"_account_id":8864,"name":"Artom Lifshitz","email":"notartom@gmail.com","username":"artom"},"change_message_id":"b636e3a8b4199977a6571cc43eaf04b4b52baba3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b5de8160_cfb5db9c","updated":"2023-02-21 14:51:55.000000000","message":"My only concern is that the base-py2 and base-py3 profiles are new in 2.9.0 [1], but I don\u0027t see a bindep version cap anywhere in upper-constraints, so I think we\u0027re good?\n\n[1] https://docs.opendev.org/opendev/bindep/latest/releasenotes.html#relnotes-2-9-0","commit_id":"47df96c09f0e7ca0ebaf30902607fc983590ecfa"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"9ad069c6fc5815192811a781dc7f6997e9fff118","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e925ec8d_1e46a0a2","updated":"2023-02-21 23:17:50.000000000","message":"Well, we don\u0027t have tests for this (RHEL 8 or RHEL 7), but if that is tested downstream, then why not...\nbindep is not the kind of thing that we maintain on stable branches, except if it fixes some gate in Zuul. Anyway, this is really just a bindep (and RH folks seem to be eager to merge this), so meh...","commit_id":"47df96c09f0e7ca0ebaf30902607fc983590ecfa"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"76d2414d89399921c8b255e68b6e0f635ffe958e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"6cdd8be5_80835a18","updated":"2023-02-21 18:09:43.000000000","message":"by the way rhel is not officially a plathform we support/test\n\nwe fixed this in wallaby when python-devel was removed from centos 8 stream and fedora\n\nhttps://github.com/openstack/nova/commit/ead034d9693727dc3f74044ab1c8e4677036f3e1\n\nthat happened a long time after the train release.\n\nim ok with a stable only patch  since on train we need to support python 2 where as wallaby only supported python3 so the wallaby patch is not directly applicable.","commit_id":"47df96c09f0e7ca0ebaf30902607fc983590ecfa"}]}