)]}'
{"id":"openstack%2Fkeystone~690156","triplet_id":"openstack%2Fkeystone~stable%2Fqueens~I176bbbda9658a54f6873a4009938f140a5b1a33e","project":"openstack/keystone","branch":"stable/queens","topic":"test-implied-roles","hashtags":[],"change_id":"I176bbbda9658a54f6873a4009938f140a5b1a33e","subject":"Add test case for expanding implied roles in system tokens","status":"MERGED","created":"2019-10-22 18:49:45.000000000","updated":"2019-10-23 14:55:24.000000000","submitted":"2019-10-23 14:55:24.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":"690156-1571842524984-ed58b23b","meta_rev_id":"b1faf3d489a59b1b9a76aa84d14eb2593995f861","_number":690156,"virtual_id_number":690156,"owner":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":9954,"name":"Lance Bragstad","username":"lbragstad","inactive":true},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2019-10-23 14:55:24.000000000","post_submit":true,"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":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":2,"date":"2019-10-23 10:43:24.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":9954,"name":"Lance Bragstad","username":"lbragstad","inactive":true},{"value":2,"date":"2019-10-22 21:52:08.000000000","_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_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":{"approved":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":1,"date":"2019-10-23 10:43:24.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"date":"2019-10-22 18:49:45.000000000","_account_id":9954,"name":"Lance Bragstad","username":"lbragstad","inactive":true},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_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}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"_account_id":9954,"name":"Lance Bragstad","username":"lbragstad","inactive":true},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-10-22 18:49:45.000000000","updated_by":{"_account_id":9954,"name":"Lance Bragstad","username":"lbragstad","inactive":true},"reviewer":{"_account_id":9954,"name":"Lance Bragstad","username":"lbragstad","inactive":true},"state":"REVIEWER"},{"updated":"2019-10-22 21:52:08.000000000","updated_by":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"reviewer":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"state":"REVIEWER"},{"updated":"2019-10-23 10:43:24.000000000","updated_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"reviewer":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"state":"REVIEWER"},{"updated":"2019-10-23 14:55:24.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":"04ca86bb75bf1eb9a6b94cefd88c4298df0fe829","author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"date":"2019-10-22 18:49:45.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"65f8316b2472fe186135ef9c2cc999efc2ebcac6","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-10-22 19:12:54.000000000","message":"Topic changed from test-implied-roles-stable/queens to test-implied-roles","accounts_in_message":[],"_revision_number":1},{"id":"a7542e076b5f9af84bde94f1a032f614464ced5e","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-10-22 19:13:22.000000000","message":"Patch Set 1: Code-Review+1\n\nClean backport, lgtm, will upgrade to a +2 when the stable/rocky change merges.","accounts_in_message":[],"_revision_number":1},{"id":"1c00828beed884c81d90c80c6efd1e69d433603e","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-22 20:24:15.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/4257074600c049ffaece7548f8103ca3 : SUCCESS in 8m 40s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/00a2a4c728194de78f004e331055046b : SUCCESS in 11m 57s\n- openstack-tox-py35 https://zuul.opendev.org/t/openstack/build/032a4a8e3312492fbe52dffd6ddb204f : SUCCESS in 20m 29s\n- build-openstack-sphinx-docs https://zuul.opendev.org/t/openstack/build/718743704494412bb936e7da4ea93346 : SUCCESS in 7m 38s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/b28b29d0b3464f8d863e76b8f49b96f1 : SUCCESS in 1h 33m 47s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/56c3339f32eb465bbbb66d341a36ec47 : SUCCESS in 18m 32s","accounts_in_message":[],"_revision_number":1},{"id":"1c292d7c696997630f57409ef44d6059243a7e02","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-10-22 21:52:08.000000000","message":"Patch Set 1: Code-Review+2\n\nThe stable/rocky change is merged now.","accounts_in_message":[],"_revision_number":1},{"id":"670cdcadf64cb0bf2b423fbf9ad73157157b5da1","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2019-10-23 10:43:24.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1\n\nRocky merged, gtg.","accounts_in_message":[],"_revision_number":1},{"id":"69ecedae77ec4c6d08c73fee43e14ae9f5b962ef","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-23 10:45:39.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"0d8fa2483b4aaec5db391caf069f4d4ccd2bb6e2","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-23 14:55:24.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/86e9e671c1534c2991dca0b2bc376108 : SUCCESS in 9m 08s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/d293b351fb404402898c50e5fa7c29a2 : SUCCESS in 21m 45s\n- openstack-tox-py35 https://zuul.opendev.org/t/openstack/build/0da2b54ea12e4216841be8bf42c5c2e4 : SUCCESS in 18m 55s\n- build-openstack-sphinx-docs https://zuul.opendev.org/t/openstack/build/79b417828d244963b2ee0de3974a7565 : SUCCESS in 9m 26s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/4338152cf8284967bedec7452e5b2a5a : SUCCESS in 1h 24m 57s","accounts_in_message":[],"_revision_number":1},{"id":"8aecefbb7fe221d4490049b02b85bdbe60cd09fa","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-10-23 14:55:24.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"d57733f4e8849331935951e8a6c3f93d755fea3e","revisions":{"d57733f4e8849331935951e8a6c3f93d755fea3e":{"kind":"REWORK","_number":1,"created":"2019-10-22 18:49:45.000000000","uploader":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"ref":"refs/changes/56/690156/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/56/690156/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/56/690156/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/56/690156/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/56/690156/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/56/690156/1"}}},"commit":{"parents":[{"commit":"a131622de4c0317fb6fa56fdc395539b35caf083","subject":"Merge \"Add retry for DBDeadlock in credential delete\" into stable/queens","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/a131622de4c0317fb6fa56fdc395539b35caf083"}]}],"author":{"name":"Lance Bragstad","email":"lbragstad@gmail.com","date":"2018-08-24 13:56:37.000000000","tz":0},"committer":{"name":"Colleen Murphy","email":"colleen.murphy@suse.de","date":"2019-10-22 18:49:29.000000000","tz":-420},"subject":"Add test case for expanding implied roles in system tokens","message":"Add test case for expanding implied roles in system tokens\n\nThis change is being backported because it provides a helper method in\nthe unit tests that another change relies on.\n\nIf a user has a role assignment on the system, which implies another\nrole assignment, the system-scoped token response should include\nboth role assignments.\n\nThis patch exposes a bug in the system-scoped token implementation\nwhere implied roles aren\u0027t expanded out before returning the\ntoken response to the user.\n\nChange-Id: I176bbbda9658a54f6873a4009938f140a5b1a33e\nRelated-Bug: 1788694\n(cherry picked from commit 6d7cfdb4ba5b8ce81d656dd22316505af6d382b8)\n(cherry picked from commit 1403a9645d3dca20a681e0ffee3f5ac3a36fe0c6)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/d57733f4e8849331935951e8a6c3f93d755fea3e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/d57733f4e8849331935951e8a6c3f93d755fea3e"}]},"branch":"refs/heads/stable/queens"}},"requirements":[],"submit_records":[],"submit_requirements":[]}