)]}'
{"id":"openstack%2Fneutron~720686","triplet_id":"openstack%2Fneutron~stable%2Fqueens~I25d3da929f8d0b6ee15d7b90ec59b9d58a4ae6a5","project":"openstack/neutron","branch":"stable/queens","topic":"bug/1863201","hashtags":[],"change_id":"I25d3da929f8d0b6ee15d7b90ec59b9d58a4ae6a5","subject":"Filter by owner SGs when retrieving the SG rules","status":"MERGED","created":"2020-04-17 11:06:27.000000000","updated":"2020-04-20 17:18:12.000000000","submitted":"2020-04-20 17:18:12.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"720686-1587403092919-31e7e898","meta_rev_id":"da211bc93c3a70a8d1f7d7c9b795d916eed7f016","_number":720686,"virtual_id_number":720686,"owner":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"value":0,"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-04-20 17:18:12.000000000","post_submit":true,"permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"date":"2020-04-17 11:14:31.000000000","_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"all":[{"value":2,"date":"2020-04-17 13:02:58.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"value":2,"date":"2020-04-20 09:02:12.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}],"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":{"approved":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"all":[{"value":0,"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"value":1,"date":"2020-04-20 09:02:12.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"value":0,"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}],"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":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-04-17 11:14:31.000000000","updated_by":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"reviewer":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"state":"REVIEWER"},{"updated":"2020-04-17 13:02:58.000000000","updated_by":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"reviewer":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"state":"REVIEWER"},{"updated":"2020-04-20 09:02:12.000000000","updated_by":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"reviewer":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"state":"REVIEWER"},{"updated":"2020-04-20 17:18:12.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"}],"messages":[{"id":"5dd7a8105ab47ce17d3b6400a45f6784892f258b","author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"date":"2020-04-17 11:06:27.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"30bdf378daeadcdf082f6277c9dbc0859ecbdf57","author":{"_account_id":26622,"name":"Cloudbase Neutron Hyper-V CI","email":"neutron_hyperv_ci@cloudbasesolutions.com","username":"neutron_hyperv_ci"},"date":"2020-04-17 11:14:31.000000000","message":"Patch Set 1:\n\nFor rechecking only on the Cloudbase Neutron Hyper-V CI, add a review comment with run-Cloudbase Neutron Hyper-V CI\n\n- neutron http://cloudbase-ci.com/neutron/720686/1 : FAILURE in 7m 21s","accounts_in_message":[],"_revision_number":1},{"id":"0408747975b661373f42c2fc641d33b9be10f467","author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"date":"2020-04-17 13:02:58.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"5510b22646d1b9ecb0c5cc17a9cb044b3181c071","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-17 13:17:39.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- neutron-tempest-plugin-api-queens https://zuul.opendev.org/t/openstack/build/a1e8d65efe0e46379780a1175b9795ab : SUCCESS in 48m 11s\n- neutron-tempest-plugin-designate-scenario-queens https://zuul.opendev.org/t/openstack/build/f96d7e9916f74e63853c67a48ac5de68 : SUCCESS in 38m 42s\n- neutron-tempest-plugin-dvr-multinode-scenario-queens https://zuul.opendev.org/t/openstack/build/2d18b8bfc831463ba44f71c4f69c0c1a : FAILURE in 1h 32m 52s (non-voting)\n- neutron-tempest-plugin-scenario-linuxbridge-queens https://zuul.opendev.org/t/openstack/build/566711612f2043b0a74bf98f43a2aa2f : SUCCESS in 54m 28s\n- neutron-tempest-plugin-scenario-openvswitch-queens https://zuul.opendev.org/t/openstack/build/7fa4cd5a7fbd44a69c9d02d1d1f8a98f : SUCCESS in 51m 23s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b82b5fe618fa4510a40535ebc26f514d : SUCCESS in 11m 57s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/91879b5423c44bf4bc4bbc4bb6fc4d7b : SUCCESS in 27m 46s\n- openstack-tox-py35 https://zuul.opendev.org/t/openstack/build/c10f5472ab3641cf9df84a2c29b4b792 : SUCCESS in 28m 09s\n- build-openstack-sphinx-docs https://zuul.opendev.org/t/openstack/build/e418286b4b244ff690943c06b3a260e4 : SUCCESS in 7m 35s\n- tempest-full https://zuul.opendev.org/t/openstack/build/ce44323f8c8f45c5bba9103d44497e79 : SUCCESS in 1h 22m 17s\n- neutron-grenade https://zuul.opendev.org/t/openstack/build/7ca03b1b413b4151bf4b56fb4cc14752 : SUCCESS in 57m 10s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/8276d0b66a99475284b2cfb5d8b08c02 : SUCCESS in 1h 10m 19s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/ad4838c157e44a9bafd753de302e8e70 : SUCCESS in 39m 09s\n- neutron-fullstack https://zuul.opendev.org/t/openstack/build/952e6b6aa3f943ca8367e9104b987f85 : FAILURE in 1h 22m 51s (non-voting)\n- neutron-rally-neutron https://zuul.opendev.org/t/openstack/build/12f05d373dd540a893a3ce5e3d0b11a0 : SUCCESS in 1h 34m 22s\n- neutron-tempest-dvr https://zuul.opendev.org/t/openstack/build/2e99643200444bfca5788961049afa52 : SUCCESS in 1h 14m 49s\n- neutron-tempest-linuxbridge https://zuul.opendev.org/t/openstack/build/bc032fc9865c44239b4ed87414bc6285 : SUCCESS in 1h 41m 27s\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/7e0522597a824b1c8d39ab99f3d5242e : SUCCESS in 1h 04m 18s\n- neutron-grenade-dvr-multinode https://zuul.opendev.org/t/openstack/build/bc24eb5e01b64ecc95cb828a4160f3ed : SUCCESS in 1h 17m 41s\n- neutron-tempest-multinode-full https://zuul.opendev.org/t/openstack/build/5aeb7a178f6a407eab5843a16fde9015 : SUCCESS in 1h 36m 20s (non-voting)\n- neutron-tempest-dvr-ha-multinode-full https://zuul.opendev.org/t/openstack/build/b43ccd3de24f4e6788bb7a3560e8deae : SUCCESS in 1h 52m 14s (non-voting)\n- neutron-tempest-ovsfw https://zuul.opendev.org/t/openstack/build/77225cc0f70845bdafffcebcb509837d : SUCCESS in 1h 47m 09s (non-voting)\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/0a45702dfe494e76894586ba1b4f19f3 : SUCCESS in 28m 26s","accounts_in_message":[],"_revision_number":1},{"id":"19566aed60469e07b5cfa7b7e4acecd7ef4bba70","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2020-04-20 09:02:12.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"f44b38b508a92e0a88bf2062b043f9a4b6e056ec","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-20 09:03:45.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"e3f6500be329caca52541e5128d20b93b358a180","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-20 14:26:26.000000000","message":"Patch Set 1:\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"4c5ef25a49e54e5b73f006e8a474b8b341bc5705","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-20 17:18:12.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- neutron-tempest-plugin-api-queens https://zuul.opendev.org/t/openstack/build/c2852f6f2b0a4fa29383e178919a1089 : SUCCESS in 48m 16s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/89e66fc4b7ae44e6bb5378e19b893a0c : SUCCESS in 15m 57s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/1cee4c2e8cbc4725b6357942d9e83fc8 : SUCCESS in 26m 56s\n- openstack-tox-py35 https://zuul.opendev.org/t/openstack/build/985719ddfa154dc098e9e29615c334df : SUCCESS in 28m 05s\n- build-openstack-sphinx-docs https://zuul.opendev.org/t/openstack/build/36b18f0101b143ce8820a30a3dbecbf6 : SUCCESS in 6m 51s\n- tempest-full https://zuul.opendev.org/t/openstack/build/3addf74525d64023a2990bcc72df7a68 : SUCCESS in 1h 23m 09s\n- neutron-grenade https://zuul.opendev.org/t/openstack/build/24f96e70533a4670aff0396407d371e0 : SUCCESS in 50m 18s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/56f77f8fe3be456b9e4b66db0d8e73ce : SUCCESS in 1h 18m 20s\n- neutron-functional https://zuul.opendev.org/t/openstack/build/ebd1ea4336164186861f9ce667ea037d : SUCCESS in 44m 35s\n- neutron-tempest-dvr https://zuul.opendev.org/t/openstack/build/178e6540bfa940a6953ab109c4a12c1c : SUCCESS in 1h 18m 58s\n- neutron-tempest-linuxbridge https://zuul.opendev.org/t/openstack/build/51775d56cc56447396184441d9558d8d : SUCCESS in 1h 23m 53s\n- neutron-grenade-multinode https://zuul.opendev.org/t/openstack/build/0a7dc119f9b54f4fbe4203dd21443470 : SUCCESS in 1h 21m 10s\n- neutron-grenade-dvr-multinode https://zuul.opendev.org/t/openstack/build/491bc912917d4e89a213ac87cd0e928f : SUCCESS in 1h 06m 28s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/2c0404168f1a43bc874a2573c595ced3 : SUCCESS in 37m 53s","accounts_in_message":[],"_revision_number":1},{"id":"3386e36297ab2f517c080ece5d890b7412e9f604","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-20 17:18:12.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"093b861bb4edcd2af0cdd31351158ba4e2fa2435","revisions":{"093b861bb4edcd2af0cdd31351158ba4e2fa2435":{"kind":"REWORK","_number":1,"created":"2020-04-17 11:06:27.000000000","uploader":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"ref":"refs/changes/86/720686/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/86/720686/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/86/720686/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/86/720686/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/86/720686/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/86/720686/1"}}},"commit":{"parents":[{"commit":"a96f950e4bbe03495dd80cc541a644dce94528ac","subject":"Merge \"Check dnsmasq process is active when spawned\" into stable/queens","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/a96f950e4bbe03495dd80cc541a644dce94528ac"}]}],"author":{"name":"Rodolfo Alonso Hernandez","email":"ralonsoh@redhat.com","date":"2020-02-18 17:08:22.000000000","tz":0},"committer":{"name":"Rodolfo Alonso Hernandez","email":"ralonsoh@redhat.com","date":"2020-04-17 11:05:53.000000000","tz":0},"subject":"Filter by owner SGs when retrieving the SG rules","message":"Filter by owner SGs when retrieving the SG rules\n\nRetrieving the SG rules now is used the admin context. This allows to\nget all possible rules, independently of the user calling. The filters\npassed and the RBAC policies filter those results, returning only:\n- The SG rules belonging to the user.\n- The SG rules belonging to a SG owned by the user.\n\nHowever, if the SG list is too long, the query can take a lot of time.\nInstead of this, the filtering is done in the DB query. If no filters\nare passed to \"get_security_group_rules\" and the context is not the\nadmin context, only the rules specified in the first paragraph will\nbe retrieved.\n\nBecause overwriting the method \"get_objects\" is too complex, an\nintermediate query is done to retrieve the SG rule IDs. Those IDs\nwill be used as a filter in the \"get_objects\" call.\n\nConflicts:\n      neutron/objects/securitygroup.py\n      neutron/tests/unit/db/test_securitygroups_db.py\n      neutron/tests/unit/objects/test_securitygroup.py\n\nCloses-Bug: #1863201\n\nChange-Id: I25d3da929f8d0b6ee15d7b90ec59b9d58a4ae6a5\n(cherry picked from commit d874c46bff7045ba25f5dd6e790f7ddb209cb224)\n(cherry picked from commit d3905264b7659b1d10a68e3629861d5f0ba13568)\n(cherry picked from commit 61dc621c1ba40efcedabdfb9f3a1854cea227d2c)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/093b861bb4edcd2af0cdd31351158ba4e2fa2435"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/093b861bb4edcd2af0cdd31351158ba4e2fa2435"}]},"branch":"refs/heads/stable/queens"}},"requirements":[],"submit_records":[],"submit_requirements":[]}