)]}'
{"id":"openstack%2Fneutron-lib~953733","triplet_id":"openstack%2Fneutron-lib~master~Ia7ac03edeed48684c549e4dcfb8d5332dd4ef71b","project":"openstack/neutron-lib","branch":"master","attention_set":{},"removed_from_attention_set":{"11975":{"account":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"last_update":"2025-07-03 14:53:30.000000000","reason":"Change was abandoned"}},"hashtags":[],"change_id":"Ia7ac03edeed48684c549e4dcfb8d5332dd4ef71b","subject":"Make model_query to be project scoped only with old API policies","status":"ABANDONED","created":"2025-06-30 09:07:19.000000000","updated":"2025-07-03 14:53:30.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"7e314de4c7f34f466c7f6911f59432f7d517ea13","_number":953733,"virtual_id_number":953733,"owner":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":-1,"date":"2025-06-30 11:15:40.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"CC":[{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2025-06-30 11:15:40.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2025-06-30 20:10:02.000000000","updated_by":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"reviewer":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"state":"CC"}],"messages":[{"id":"40a3e0d93b64d05facabbec3d96e8c0ce6cfb127","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2025-06-30 09:07:19.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"8a0621bbe416798374c6345286faea7aa1291b9e","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-06-30 11:15:40.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/af8b5d6e537b41e68c4664b62c0b625f\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/d19d9fd3bb2a4c0a94499cb9735e2629 : SUCCESS in 8m 53s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/4bcdce751e834604af54c7b4d80b84ef : SUCCESS in 5m 47s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/73ba3e29b63d4136881ee3d291560fe5 : SUCCESS in 7m 29s\n- openstack-tox-py310 https://zuul.opendev.org/t/openstack/build/059531fa6d6c424993fc6864b049e566 : SUCCESS in 4m 43s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/f931b58cbbd24630ad07130facf09255 : SUCCESS in 5m 21s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/47ce92c786bf4f8d8ab4d5491eaaf2ae : SUCCESS in 7m 47s (non-voting)\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/3d0fe8ca6d4c4db7b3dd3e0dcefaaa77 : SUCCESS in 1h 53m 53s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/a646ff20b33b4b7886d1bf7db942d717 : SUCCESS in 4m 20s\n- neutron-tempest-plugin-openvswitch https://zuul.opendev.org/t/openstack/build/25263e99ca074f9e9a413fac2e28d2b3 : FAILURE in 1h 47m 56s\n- neutron-tempest-plugin-ovn https://zuul.opendev.org/t/openstack/build/42ab7f8fc5d449cdb967d49b967a2ce8 : FAILURE in 2h 07m 11s\n- neutron-tempest-plugin-designate-scenario https://zuul.opendev.org/t/openstack/build/617b945d3feb409aa1c751aa13f7bf94 : SUCCESS in 36m 21s\n- openstack-tox-py312-with-neutron https://zuul.opendev.org/t/openstack/build/e7ca7af550fe4e16a5f11f668ff67434 : FAILURE in 15m 20s\n- openstack-tox-py312-with-sqlalchemy-master https://zuul.opendev.org/t/openstack/build/ee30d9604be0490e8902a978ac7ad75a : SUCCESS in 4m 35s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/efe67f65945d40c1ad0c2287ad83d1f1 : FAILURE in 26m 06s","accounts_in_message":[],"_revision_number":1},{"id":"7e314de4c7f34f466c7f6911f59432f7d517ea13","tag":"autogenerated:gerrit:abandon","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2025-07-03 14:53:30.000000000","message":"Abandoned\n\nsupreseeded by https://review.opendev.org/c/openstack/neutron-lib/+/954054","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"ba6cbeeff9cc1c21b84e0093d337f3e2087fd64c","revisions":{"ba6cbeeff9cc1c21b84e0093d337f3e2087fd64c":{"kind":"REWORK","_number":1,"created":"2025-06-30 09:07:19.000000000","uploader":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"ref":"refs/changes/33/953733/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron-lib","ref":"refs/changes/33/953733/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron-lib refs/changes/33/953733/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron-lib refs/changes/33/953733/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron-lib refs/changes/33/953733/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron-lib refs/changes/33/953733/1"}}},"commit":{"parents":[{"commit":"d1a62372592d79658b37b1fee1975aa0be5c4d2b","subject":"Merge \"Prepare migration of NetworkMTUSubnetConflict\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron-lib/commit/d1a62372592d79658b37b1fee1975aa0be5c4d2b"}]}],"author":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2025-06-30 08:29:49.000000000","tz":120},"committer":{"name":"Slawek Kaplonski","email":"skaplons@redhat.com","date":"2025-06-30 09:07:03.000000000","tz":120},"subject":"Make model_query to be project scoped only with old API policies","message":"Make model_query to be project scoped only with old API policies\n\nFunction model_query_scope_is_project from the neutron_lib.db.utils\nreturns True if the the non-admin and non-service token is used and if\nthe model has \"project_id\" attribute. This is done like that to limit\ndata fetched by regular users from the DB to only resources which are\nowned by the same project as in the query context.\n\nWith new S-RBAC policies where user should be able to define custom\nroles, like e.g. \"admin-reader\" this is not really working well as we\nshould always get all from the db and rely on the policy module with\nfiltering the final result returned to the user.\n\nSo this patch adds check of the config option\n`oslo_policy.enforce_new_defaults` in the model_query_scope_is_project\nfunction and if this config option is enabled, it always retuns `False`\nso that there is no filtering based on the project on the DB query\nlevel.\n\nThis function `model_query_scope_is_project` may even be removed\ncompletely once old default API policies will be removed.\n\nCloses-Bug: #2115184\nChange-Id: Ia7ac03edeed48684c549e4dcfb8d5332dd4ef71b\nSigned-off-by: Slawek Kaplonski \u003cskaplons@redhat.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron-lib/commit/ba6cbeeff9cc1c21b84e0093d337f3e2087fd64c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron-lib/commit/ba6cbeeff9cc1c21b84e0093d337f3e2087fd64c"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{}}},{"name":"Review-Priority","description":"Review priority","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"-label:Review-Priority\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":[],"failing_atoms":["label:Review-Priority\u003dMIN"],"atom_explanations":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{}}}]}
