)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"1fd5759626b570e447c330848c2afb480e073c1f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f0d2f2ee_439cdb20","updated":"2024-07-04 13:41:16.000000000","message":"It looks weird to update by a major version for just a simple python version removal but I think Elod is correct.","commit_id":"b98bb3c7c2e17b77e6f355267ada87b6b5a15d30"}],"deliverables/dalmatian/osc-placement.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"bd990dcd5efc200b54fa2d11ae14112feab48662","unresolved":true,"context_lines":[{"line_number":7,"context_line":"repository-settings:"},{"line_number":8,"context_line":"  openstack/osc-placement: {}"},{"line_number":9,"context_line":"releases:"},{"line_number":10,"context_line":"  - version: 4.4.0"},{"line_number":11,"context_line":"    projects:"},{"line_number":12,"context_line":"      - repo: openstack/osc-placement"},{"line_number":13,"context_line":"        hash: f40890c972ed51dde0dac4d40c844d2d8a142c4e"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"e7b9f24d_1863f8dc","line":10,"updated":"2024-07-04 13:35:10.000000000","message":"I\u0027m a bit hesitant with this as we might need a MAJOR version bump as we are dropping py38 classifier from setup.cfg [1], though we are not adding a minimum python version... so I\u0027m not sure whether pip would allow the installation of this package to python3.8 version environments... and if it doesn\u0027t allow, then we should bump MAJOR version instead.\n\n[1] https://review.opendev.org/c/openstack/osc-placement/+/904612","commit_id":"b98bb3c7c2e17b77e6f355267ada87b6b5a15d30"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"fcd69ce4558606c52eccb5bc8ffefeb0368d3e3f","unresolved":true,"context_lines":[{"line_number":7,"context_line":"repository-settings:"},{"line_number":8,"context_line":"  openstack/osc-placement: {}"},{"line_number":9,"context_line":"releases:"},{"line_number":10,"context_line":"  - version: 4.4.0"},{"line_number":11,"context_line":"    projects:"},{"line_number":12,"context_line":"      - repo: openstack/osc-placement"},{"line_number":13,"context_line":"        hash: f40890c972ed51dde0dac4d40c844d2d8a142c4e"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"6c27f277_34e9e0e6","line":10,"in_reply_to":"46b81039_5e53e882","updated":"2024-07-05 10:00:06.000000000","message":"Anyway, I\u0027ve bumped the MAJOR version, just to be on the safe(r) side.","commit_id":"b98bb3c7c2e17b77e6f355267ada87b6b5a15d30"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"6c79d759b7d259c6a53e22aff7f18c4ce1f41804","unresolved":true,"context_lines":[{"line_number":7,"context_line":"repository-settings:"},{"line_number":8,"context_line":"  openstack/osc-placement: {}"},{"line_number":9,"context_line":"releases:"},{"line_number":10,"context_line":"  - version: 4.4.0"},{"line_number":11,"context_line":"    projects:"},{"line_number":12,"context_line":"      - repo: openstack/osc-placement"},{"line_number":13,"context_line":"        hash: f40890c972ed51dde0dac4d40c844d2d8a142c4e"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"46b81039_5e53e882","line":10,"in_reply_to":"58284aeb_de8cbe82","updated":"2024-07-04 14:23:23.000000000","message":"yeah, but we keep here \u0027python_requires \u003d \u003e\u003d3.8\u0027 [1], so i don\u0027t know how pypi handle this... :-/\n\n[1] see my comments here: https://review.opendev.org/c/openstack/osc-placement/+/904612","commit_id":"b98bb3c7c2e17b77e6f355267ada87b6b5a15d30"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"e89652072917e2ff53486119fc9a2f1514d0f3d0","unresolved":false,"context_lines":[{"line_number":7,"context_line":"repository-settings:"},{"line_number":8,"context_line":"  openstack/osc-placement: {}"},{"line_number":9,"context_line":"releases:"},{"line_number":10,"context_line":"  - version: 4.4.0"},{"line_number":11,"context_line":"    projects:"},{"line_number":12,"context_line":"      - repo: openstack/osc-placement"},{"line_number":13,"context_line":"        hash: f40890c972ed51dde0dac4d40c844d2d8a142c4e"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"866782a7_83d7a62c","line":10,"in_reply_to":"6c27f277_34e9e0e6","updated":"2024-07-05 13:46:34.000000000","message":"fungi explained us [1] that:\n\n    15:34 \u003c fungi\u003e hberaud: elodilles: trove classifiers are purely advisory, no installers i\u0027m aware of take them into consideration (especially not pip). the python_requires is really the only thing \n               installers look at to determine whether something can be installed for the current interpreter version\n    15:35 \u003c fungi\u003e so if you want to soft-deprecate a python interpreter version, removing the trove classifier saying you support it while not increasing requires_python right away has the desired effect of \n               not immediately breaking consumers of a new release of the package on the older python interpreter\n\nso in that sense we are still good with MINOR version bump, hence i\u0027ve set back the original MINOR version bump. and we should be OK with this now.\n\n[1] https://meetings.opendev.org/irclogs/%23openstack-release/%23openstack-release.2024-07-05.log.html#t2024-07-05T13:34:20","commit_id":"b98bb3c7c2e17b77e6f355267ada87b6b5a15d30"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"1fd5759626b570e447c330848c2afb480e073c1f","unresolved":true,"context_lines":[{"line_number":7,"context_line":"repository-settings:"},{"line_number":8,"context_line":"  openstack/osc-placement: {}"},{"line_number":9,"context_line":"releases:"},{"line_number":10,"context_line":"  - version: 4.4.0"},{"line_number":11,"context_line":"    projects:"},{"line_number":12,"context_line":"      - repo: openstack/osc-placement"},{"line_number":13,"context_line":"        hash: f40890c972ed51dde0dac4d40c844d2d8a142c4e"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"58284aeb_de8cbe82","line":10,"in_reply_to":"e7b9f24d_1863f8dc","updated":"2024-07-04 13:41:16.000000000","message":"We had a precedent with osc-placement-2.0.0 which was a py2.7 removal support.","commit_id":"b98bb3c7c2e17b77e6f355267ada87b6b5a15d30"}]}