)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"995c3826fd3e157f76ff1199f4a5b6b893f5d23e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7b4474b2_8a30708e","updated":"2026-07-21 11:06:44.000000000","message":"The change looked good before, but now it\u0027s also passing CI. Thanks for adding the pbr change before this one.","commit_id":"8deaa4fbaf21ccd4d07bc261fbf12d9c3828caf1"}],"pyproject.toml":[{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"4ebe32e1f4c0c5181bc0fe682ebcec2648619133","unresolved":true,"context_lines":[{"line_number":1,"context_line":"[build-system]"},{"line_number":2,"context_line":"requires \u003d [\u0027setuptools\u003e\u003d68\u0027]"},{"line_number":3,"context_line":"build-backend \u003d \u0027setuptools.build_meta\u0027"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"[project]"},{"line_number":6,"context_line":"name \u003d \"python-observabilityclient\""}],"source_content_type":"text/x-toml","patch_set":1,"id":"36ed83cf_b64d2a72","line":3,"updated":"2026-07-17 19:02:07.000000000","message":"Looks like CI is complaining about this line.\n\n```\n2026-07-17 16:07:03.575992 | TASK [check-requirements : Run requirements check script]\n2026-07-17 16:07:04.150206 | ubuntu-noble | pyproject.toml has invalid \u0027build-system.build-backend\u0027. Expected \u0027pbr.build\u0027; got \u0027setuptools.build_meta\u0027\n```\n\nThe output comes from this script: https://opendev.org/openstack/requirements/src/branch/master/playbooks/files/project-requirements-change.py \n\nThe issue is hit running this line \n```pyproject_found \u003d project.verify_pyproject_toml(cwd)```\n\nwhich is from this file: https://opendev.org/openstack/requirements/src/commit/1244391f2eb1a5b626c84ea9623d631ce5820ff7/openstack_requirements/project.py#L137-L143\n\n```\n    if (build_backend :\u003d data[\u0027build-system\u0027].get(\u0027build-backend\u0027)) !\u003d \u0027pbr.build\u0027:\n         # print the failure message we saw       \n\n```\n\nThe change that added this was submitted on June 12th: https://review.opendev.org/c/openstack/requirements/+/989214\n\nHowever, it appears that there haven\u0027t been any changes to the files that trigger the requirements-check job since then.\n\nThis line was last updated in this commit: https://opendev.org/openstack/python-observabilityclient/commit/57cd2955456acf83da7b8a3991947fba1d4d55d4\nwhich doesn\u0027t have a change-id associated with it, so either it was an automated change, or was added before the repo was introduced into opendev.\n\n\nSo, this needs to be fixed, and this should fix it:\n\n```suggestion\nbuild-backend \u003d \u0027pbr.build\u0027\n```\n\nHowever, IMHO, the change should be in a separate review, WDYT?","commit_id":"9d8ff356815cdce87516cad171e7224b20176f0f"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"a3f1d779c33f40b2f8464294499b45b7371ad265","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[build-system]"},{"line_number":2,"context_line":"requires \u003d [\u0027setuptools\u003e\u003d68\u0027]"},{"line_number":3,"context_line":"build-backend \u003d \u0027setuptools.build_meta\u0027"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"[project]"},{"line_number":6,"context_line":"name \u003d \"python-observabilityclient\""}],"source_content_type":"text/x-toml","patch_set":1,"id":"bb071d58_029cf421","line":3,"in_reply_to":"36ed83cf_b64d2a72","updated":"2026-07-18 03:09:24.000000000","message":"It\u0027s wired this wasn\u0027t caught earlier but becasue setup.py requires pbr it makes sense to require it here too. I\u0027ve proposed https://review.opendev.org/c/openstack/python-observabilityclient/+/997795 to fix it","commit_id":"9d8ff356815cdce87516cad171e7224b20176f0f"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"995c3826fd3e157f76ff1199f4a5b6b893f5d23e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[build-system]"},{"line_number":2,"context_line":"requires \u003d [\u0027setuptools\u003e\u003d68\u0027]"},{"line_number":3,"context_line":"build-backend \u003d \u0027setuptools.build_meta\u0027"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"[project]"},{"line_number":6,"context_line":"name \u003d \"python-observabilityclient\""}],"source_content_type":"text/x-toml","patch_set":1,"id":"0c84f59a_7a02a48f","line":3,"in_reply_to":"bb071d58_029cf421","updated":"2026-07-21 11:06:44.000000000","message":"I think that was happened is that this is the first change to bbservabilityclient since the check for pbr was added to the openstack/requirements repo. Since the check was added in requirements, and wasn\u0027t a change here, this is the first time it\u0027s actually running.\n\nIt raises some concerns about low traffic repos around release time. It\u0027s possible that something like this could happen again, and we could release changes that fail these checks. In this case, it\u0027s not too serious, but there might be cases where it\u0027s a legitimate bug (e.g. new python version, or new version of a dependancy)","commit_id":"9d8ff356815cdce87516cad171e7224b20176f0f"}]}
