)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"ac0f0c77c442219401182b94154c36a5adc85a08","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ea471279_a6598fbe","updated":"2026-04-10 14:39:04.000000000","message":"We need to support all versions of OpenStack with OSC, not just the latest version. I get the impression this will break on older (pre-Gazpacho) clouds? If so, we likely can\u0027t merge it as-is. Is there an extension we could check for instead to determine what calls to make?","commit_id":"66b3694b31a551ad1ac65601209c632f91504a7e"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"d21c718bbca2828686dc15fb10fa810e209fd50f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9a69d861_a0b1f3b8","in_reply_to":"3488a6c8_af66eaf1","updated":"2026-04-10 17:00:22.000000000","message":"Yeah, maybe my commit message was misleading. We (me) made a lot of changes to make sure we used project_id over tenant_id. The API should always make sure both are present, but a plugin might only pass the latter. Eventually we\u0027ll rip out that shim.\n\nI would be interested in adding a hacking check, don\u0027t know how it would work as we still need to have some code to remove tenant_id from getting displayed.","commit_id":"66b3694b31a551ad1ac65601209c632f91504a7e"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f6a792d5d26a5362b321198435856029e0bb26e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3488a6c8_af66eaf1","in_reply_to":"3ec1a76e_88223055","updated":"2026-04-10 16:47:12.000000000","message":"Okay, so this wasn\u0027t a G-cycle thing? If so, can you update the commit message since the way it currently reads suggests (to me) that some APIs only added support for `project_id` filters/fields in G.\n\nAlso, this would be a trivial hacking check to add, if you were so inclined.","commit_id":"66b3694b31a551ad1ac65601209c632f91504a7e"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"de4e19708f16bab74a3bf85b82179e90b8cd0a84","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3ec1a76e_88223055","in_reply_to":"ea471279_a6598fbe","updated":"2026-04-10 16:18:20.000000000","message":"I see your point, but project_id should be supported on clouds back to Newton, at least that\u0027s when neutron started supporting keystone-v3. And this is mainly changing these last places to be like the other network code that uses project_id.\n\nBack in 2022 tenant_id was removed [0], some things just snuck back in when code was migrated from neutronclient or during other updates.\n\n[0] https://review.opendev.org/c/openstack/python-openstackclient/+/824746","commit_id":"66b3694b31a551ad1ac65601209c632f91504a7e"}],"openstackclient/network/v2/network_trunk.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"c8ae0dd180a8cee7f878037a06b5d0e579998a65","unresolved":true,"context_lines":[{"line_number":385,"context_line":"            network_client, parsed_args.add_subports"},{"line_number":386,"context_line":"        )"},{"line_number":387,"context_line":""},{"line_number":388,"context_line":"    # \"trunk set\" command doesn\u0027t support setting project."},{"line_number":389,"context_line":"    if \u0027project\u0027 in parsed_args and parsed_args.project is not None:"},{"line_number":390,"context_line":"        project_id \u003d identity_utils.find_project("},{"line_number":391,"context_line":"            identity_client,"}],"source_content_type":"text/x-python","patch_set":1,"id":"52979b8b_c68b3802","line":388,"updated":"2026-03-22 01:14:12.000000000","message":"I guess I can remove this too","commit_id":"66b3694b31a551ad1ac65601209c632f91504a7e"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"f0984e1d9703e86578d22b0e3b83e7efc9994781","unresolved":false,"context_lines":[{"line_number":385,"context_line":"            network_client, parsed_args.add_subports"},{"line_number":386,"context_line":"        )"},{"line_number":387,"context_line":""},{"line_number":388,"context_line":"    # \"trunk set\" command doesn\u0027t support setting project."},{"line_number":389,"context_line":"    if \u0027project\u0027 in parsed_args and parsed_args.project is not None:"},{"line_number":390,"context_line":"        project_id \u003d identity_utils.find_project("},{"line_number":391,"context_line":"            identity_client,"}],"source_content_type":"text/x-python","patch_set":1,"id":"b8d45b13_c923c2be","line":388,"in_reply_to":"52979b8b_c68b3802","updated":"2026-03-22 01:18:08.000000000","message":"Although maybe it doesn\u0027t mean what I think it means, will leave alone.","commit_id":"66b3694b31a551ad1ac65601209c632f91504a7e"}]}
