)]}'
{"id":"openstack%2Fkeystone~132152","triplet_id":"openstack%2Fkeystone~master~I86d4dd65f0eae8563723f6cbe02ecbc6962654e9","project":"openstack/keystone","branch":"master","topic":"udpate_policy","hashtags":[],"change_id":"I86d4dd65f0eae8563723f6cbe02ecbc6962654e9","subject":"move matching id check in policy update into controller","status":"ABANDONED","created":"2014-10-31 03:14:35.000000000","updated":"2014-12-08 02:46:35.000000000","total_comment_count":8,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"a22346d25cd3a98b47080689b4789df56151414c","_number":132152,"virtual_id_number":132152,"owner":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":1,"date":"2014-11-27 04:30:20.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},{"_account_id":13055,"name":"Alexander Makarov","email":"ajiekcahdp.makapob@gmail.com","username":"amakarov"}],"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":{"disliked":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},{"value":-1,"date":"2014-12-04 15:25:09.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},{"value":-1,"date":"2014-12-03 17:28:48.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":13055,"name":"Alexander Makarov","email":"ajiekcahdp.makapob@gmail.com","username":"amakarov"}],"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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},{"_account_id":13055,"name":"Alexander Makarov","email":"ajiekcahdp.makapob@gmail.com","username":"amakarov"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},{"_account_id":13055,"name":"Alexander Makarov","email":"ajiekcahdp.makapob@gmail.com","username":"amakarov"},{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2014-11-13 09:45:40.000000000","updated_by":{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},"reviewer":{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},"state":"REVIEWER"},{"updated":"2014-11-27 04:30:20.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2014-12-03 17:28:48.000000000","updated_by":{"_account_id":13055,"name":"Alexander Makarov","email":"ajiekcahdp.makapob@gmail.com","username":"amakarov"},"reviewer":{"_account_id":13055,"name":"Alexander Makarov","email":"ajiekcahdp.makapob@gmail.com","username":"amakarov"},"state":"REVIEWER"},{"updated":"2014-12-04 15:25:09.000000000","updated_by":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"reviewer":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"state":"REVIEWER"}],"messages":[{"id":"ad532781178513d9c01abf56abd1bfc2874ada34","author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"date":"2014-10-31 03:14:35.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"e2e3bde0b9590d4cd796ef13a6ada2271732bb96","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-10-31 04:18:55.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- gate-keystone-pep8 http://logs.openstack.org/52/132152/1/check/gate-keystone-pep8/41f5c6a : SUCCESS in 4m 10s\n- gate-keystone-docs http://docs-draft.openstack.org/52/132152/1/check/gate-keystone-docs/c6ad21b/doc/build/html/ : SUCCESS in 3m 44s\n- gate-keystone-python27 http://logs.openstack.org/52/132152/1/check/gate-keystone-python27/e6f2796 : FAILURE in 9m 04s\n- check-tempest-dsvm-full http://logs.openstack.org/52/132152/1/check/check-tempest-dsvm-full/1950cdd : SUCCESS in 51m 09s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/52/132152/1/check/check-tempest-dsvm-postgres-full/6db500a : SUCCESS in 1h 04m 16s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/52/132152/1/check/check-tempest-dsvm-neutron-full/4d53626 : SUCCESS in 52m 31s\n- check-tempest-dsvm-neutron-heat-slow http://logs.openstack.org/52/132152/1/check/check-tempest-dsvm-neutron-heat-slow/d05a0bc : SUCCESS in 18m 27s\n- check-grenade-dsvm http://logs.openstack.org/52/132152/1/check/check-grenade-dsvm/3cca6a0 : SUCCESS in 39m 53s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/52/132152/1/check/gate-tempest-dsvm-large-ops/98739da : SUCCESS in 18m 35s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/52/132152/1/check/gate-tempest-dsvm-neutron-large-ops/d4cbb4d : SUCCESS in 23m 06s\n- check-devstack-dsvm-cells http://logs.openstack.org/52/132152/1/check/check-devstack-dsvm-cells/fcf9734 : SUCCESS in 15m 16s\n- check-swift-dsvm-functional http://logs.openstack.org/52/132152/1/check/check-swift-dsvm-functional/1fa281c : SUCCESS in 16m 38s\n- gate-rally-dsvm-keystone http://logs.openstack.org/52/132152/1/check/gate-rally-dsvm-keystone/bebf5e9 : SUCCESS in 17m 55s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"c43a41baaf89cafcb385e2648ac4de9b45971190","author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"date":"2014-10-31 06:30:33.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"aa016da96ff2bf20a08997bf5c0270f482b7873a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-10-31 07:22:29.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-keystone-pep8 http://logs.openstack.org/52/132152/2/check/gate-keystone-pep8/d144559 : SUCCESS in 2m 54s\n- gate-keystone-docs http://docs-draft.openstack.org/52/132152/2/check/gate-keystone-docs/fa7ebba/doc/build/html/ : SUCCESS in 6m 04s\n- gate-keystone-python27 http://logs.openstack.org/52/132152/2/check/gate-keystone-python27/660c42f : SUCCESS in 7m 18s\n- check-tempest-dsvm-full http://logs.openstack.org/52/132152/2/check/check-tempest-dsvm-full/2f620f4 : SUCCESS in 49m 59s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/52/132152/2/check/check-tempest-dsvm-postgres-full/527702b : SUCCESS in 51m 52s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/52/132152/2/check/check-tempest-dsvm-neutron-full/7866382 : SUCCESS in 49m 59s\n- check-tempest-dsvm-neutron-heat-slow http://logs.openstack.org/52/132152/2/check/check-tempest-dsvm-neutron-heat-slow/cc4e24b : SUCCESS in 18m 50s\n- check-grenade-dsvm http://logs.openstack.org/52/132152/2/check/check-grenade-dsvm/8fb468e : SUCCESS in 44m 23s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/52/132152/2/check/gate-tempest-dsvm-large-ops/152028f : SUCCESS in 25m 03s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/52/132152/2/check/gate-tempest-dsvm-neutron-large-ops/b7e5699 : SUCCESS in 22m 53s\n- check-devstack-dsvm-cells http://logs.openstack.org/52/132152/2/check/check-devstack-dsvm-cells/8df975f : SUCCESS in 13m 37s\n- check-swift-dsvm-functional http://logs.openstack.org/52/132152/2/check/check-swift-dsvm-functional/b33e585 : SUCCESS in 17m 40s\n- gate-rally-dsvm-keystone http://logs.openstack.org/52/132152/2/check/gate-rally-dsvm-keystone/3c9931f : SUCCESS in 20m 43s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"ad96ef4b1ec70f1872bcda25410734d88534330d","author":{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},"date":"2014-11-14 11:31:39.000000000","message":"Patch Set 2: Code-Review-1\n\n(1 comment)\n\nOverall, the change is make sense, but would you pls note the exception in your testcase?","accounts_in_message":[],"_revision_number":2},{"id":"aa5ffc83f70c82d29b21035bce5473e38754bda1","author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"date":"2014-11-15 01:02:15.000000000","message":"Patch Set 2:\n\n(1 comment)\n\n@Dave Chen, see my reply in the comments please.","accounts_in_message":[],"_revision_number":2},{"id":"87cb9591bc92d0e7d231931c582e2ec101ef258d","author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"date":"2014-11-18 01:03:30.000000000","message":"Patch Set 2:\n\nHello Dave Chen. Could you remove -1 or other suggestions? Thanks.","accounts_in_message":[],"_revision_number":2},{"id":"666772f9979ad59acc41695be7c45ffae0eda10d","author":{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},"date":"2014-11-18 12:39:37.000000000","message":"Patch Set 2: Code-Review+1\n\nThe changes is make sense.","accounts_in_message":[],"_revision_number":2},{"id":"26d5149d0338b228a6e9d84fdbf2a2aee2777d62","author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"date":"2014-11-26 04:55:51.000000000","message":"Patch Set 2: Code-Review-1\n\n(2 comments)","accounts_in_message":[],"_revision_number":2},{"id":"e669dbeeb4dbf03a98b58f0e7e01abd4a162f8a0","author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"date":"2014-11-26 07:34:14.000000000","message":"Patch Set 2:\n\n(2 comments)","accounts_in_message":[],"_revision_number":2},{"id":"22b1de369eeba16ea8e84961b77755deb3622087","author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"date":"2014-11-27 03:44:06.000000000","message":"Patch Set 3: Commit message was updated","accounts_in_message":[],"_revision_number":3},{"id":"2070960e83911d8469dffc1f399058d63948370e","author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"date":"2014-11-27 03:44:37.000000000","message":"Patch Set 4: Commit message was updated","accounts_in_message":[],"_revision_number":4},{"id":"935e95c1a4885781cbc1ad18dc11692a50050ba9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-11-27 04:30:20.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-keystone-pep8 http://logs.openstack.org/52/132152/4/check/gate-keystone-pep8/bd97995 : SUCCESS in 2m 58s\n- gate-keystone-docs http://docs-draft.openstack.org/52/132152/4/check/gate-keystone-docs/689c4ba/doc/build/html/ : SUCCESS in 4m 28s\n- gate-keystone-python27 http://logs.openstack.org/52/132152/4/check/gate-keystone-python27/9080e25 : SUCCESS in 9m 23s\n- check-tempest-dsvm-full http://logs.openstack.org/52/132152/4/check/check-tempest-dsvm-full/de22098 : SUCCESS in 37m 11s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/52/132152/4/check/check-tempest-dsvm-postgres-full/b7c21c6 : SUCCESS in 34m 21s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/52/132152/4/check/check-tempest-dsvm-neutron-full/9de481d : SUCCESS in 45m 39s\n- check-grenade-dsvm http://logs.openstack.org/52/132152/4/check/check-grenade-dsvm/3fd57e3 : SUCCESS in 27m 12s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/52/132152/4/check/gate-tempest-dsvm-large-ops/0a2d7e3 : SUCCESS in 18m 53s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/52/132152/4/check/gate-tempest-dsvm-neutron-large-ops/270d79a : SUCCESS in 16m 58s\n- check-devstack-dsvm-cells http://logs.openstack.org/52/132152/4/check/check-devstack-dsvm-cells/262f4b9 : SUCCESS in 14m 19s\n- check-swift-dsvm-functional http://logs.openstack.org/52/132152/4/check/check-swift-dsvm-functional/5abd771 : SUCCESS in 19m 36s\n- gate-rally-dsvm-keystone http://logs.openstack.org/52/132152/4/check/gate-rally-dsvm-keystone/882de0e : SUCCESS in 14m 43s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"66c4ff35a0016d93f7c27a75b2993adb62809048","author":{"_account_id":13055,"name":"Alexander Makarov","email":"ajiekcahdp.makapob@gmail.com","username":"amakarov"},"date":"2014-12-03 17:28:48.000000000","message":"Patch Set 4: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"2640d3fee3b699694c0e4b1af47eed05121a9fcc","author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"date":"2014-12-04 15:25:09.000000000","message":"Patch Set 4: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"afd36d9bcbc3a6f9cfbd6573d08af980edc0cc1f","author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"date":"2014-12-08 02:46:35.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"689eb8b3c520f2d66f78375e34a5bc05fc4b17d8","revisions":{"7f4fbec969d7b6ccd6780b50a774c4194c3209d4":{"kind":"REWORK","_number":1,"created":"2014-10-31 03:14:35.000000000","uploader":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"ref":"refs/changes/52/132152/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/52/132152/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/52/132152/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/52/132152/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/52/132152/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/52/132152/1"}}},"commit":{"parents":[{"commit":"3d9184b6f5860f0b56091a326ed41d2a4c29fbe4","subject":"Merge \"Updated from global requirements\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/3d9184b6f5860f0b56091a326ed41d2a4c29fbe4"}]}],"author":{"name":"wanghong","email":"w.wanghong@huawei.com","date":"2014-10-31 03:07:51.000000000","tz":480},"committer":{"name":"wanghong","email":"w.wanghong@huawei.com","date":"2014-10-31 03:07:51.000000000","tz":480},"subject":"move matching id check in policy update into controller","message":"move matching id check in policy update into controller\n\nMake it consistent with other keystone objects.\n\nChange-Id: I86d4dd65f0eae8563723f6cbe02ecbc6962654e9\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/7f4fbec969d7b6ccd6780b50a774c4194c3209d4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/7f4fbec969d7b6ccd6780b50a774c4194c3209d4"}]},"branch":"refs/heads/master"},"1183b75dec35f3d15cf51bbc6e3555030fad52ff":{"kind":"REWORK","_number":2,"created":"2014-10-31 06:30:33.000000000","uploader":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"ref":"refs/changes/52/132152/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/52/132152/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/52/132152/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/52/132152/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/52/132152/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/52/132152/2"}}},"commit":{"parents":[{"commit":"3d9184b6f5860f0b56091a326ed41d2a4c29fbe4","subject":"Merge \"Updated from global requirements\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/3d9184b6f5860f0b56091a326ed41d2a4c29fbe4"}]}],"author":{"name":"wanghong","email":"w.wanghong@huawei.com","date":"2014-10-31 03:07:51.000000000","tz":480},"committer":{"name":"wanghong","email":"w.wanghong@huawei.com","date":"2014-10-31 06:30:08.000000000","tz":480},"subject":"move matching id check in policy update into controller","message":"move matching id check in policy update into controller\n\nMake it consistent with other keystone objects.\n\nChange-Id: I86d4dd65f0eae8563723f6cbe02ecbc6962654e9\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/1183b75dec35f3d15cf51bbc6e3555030fad52ff"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/1183b75dec35f3d15cf51bbc6e3555030fad52ff"}]},"branch":"refs/heads/master"},"330ef87826fbb6df47592b5266789ef62761f6df":{"kind":"NO_CODE_CHANGE","_number":3,"created":"2014-11-27 03:44:06.000000000","uploader":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"ref":"refs/changes/52/132152/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/52/132152/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/52/132152/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/52/132152/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/52/132152/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/52/132152/3"}}},"commit":{"parents":[{"commit":"3d9184b6f5860f0b56091a326ed41d2a4c29fbe4","subject":"Merge \"Updated from global requirements\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/3d9184b6f5860f0b56091a326ed41d2a4c29fbe4"}]}],"author":{"name":"wanghong","email":"w.wanghong@huawei.com","date":"2014-10-31 03:07:51.000000000","tz":480},"committer":{"name":"wanghong","email":"w.wanghong@huawei.com","date":"2014-11-27 03:44:06.000000000","tz":0},"subject":"move matching id check in policy update into controller","message":"move matching id check in policy update into controller\n\nThere is _require_matching_id method to do this already, and the parameter check should be in controller layer too.\nMake it consistent with other keystone objects.\n\nChange-Id: I86d4dd65f0eae8563723f6cbe02ecbc6962654e9\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/330ef87826fbb6df47592b5266789ef62761f6df"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/330ef87826fbb6df47592b5266789ef62761f6df"}]},"branch":"refs/heads/master"},"689eb8b3c520f2d66f78375e34a5bc05fc4b17d8":{"kind":"NO_CODE_CHANGE","_number":4,"created":"2014-11-27 03:44:37.000000000","uploader":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"ref":"refs/changes/52/132152/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/52/132152/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/52/132152/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/52/132152/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/52/132152/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/52/132152/4"}}},"commit":{"parents":[{"commit":"3d9184b6f5860f0b56091a326ed41d2a4c29fbe4","subject":"Merge \"Updated from global requirements\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/3d9184b6f5860f0b56091a326ed41d2a4c29fbe4"}]}],"author":{"name":"wanghong","email":"w.wanghong@huawei.com","date":"2014-10-31 03:07:51.000000000","tz":480},"committer":{"name":"wanghong","email":"w.wanghong@huawei.com","date":"2014-11-27 03:44:37.000000000","tz":0},"subject":"move matching id check in policy update into controller","message":"move matching id check in policy update into controller\n\nThere is _require_matching_id method to do this already,\nand the parameter check should be in controller layer too.\nMake it consistent with other keystone objects.\n\nChange-Id: I86d4dd65f0eae8563723f6cbe02ecbc6962654e9\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/689eb8b3c520f2d66f78375e34a5bc05fc4b17d8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/689eb8b3c520f2d66f78375e34a5bc05fc4b17d8"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}