)]}'
{"id":"openstack%2Fkeystone~225738","triplet_id":"openstack%2Fkeystone~master~I73af6b80fa75286e96943026b7b71ad23dc1786b","project":"openstack/keystone","branch":"master","topic":"bandit","hashtags":[],"change_id":"I73af6b80fa75286e96943026b7b71ad23dc1786b","subject":"Enable try_except_pass Bandit test","status":"MERGED","created":"2015-09-21 14:27:39.000000000","updated":"2015-11-06 10:52:02.000000000","submitted":"2015-11-06 10:52:00.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":30,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"9b8b881d73e9c422641d51a6fd3c716aed09916c","_number":225738,"virtual_id_number":225738,"owner":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},{"value":0,"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},{"value":0,"_account_id":16523,"name":"ZhangLei","display_name":"ZhangLei","email":"zhang.lei@99cloud.net","username":"zhang.lei"},{"value":0,"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},{"value":0,"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},{"value":0,"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},{"value":0,"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"value":0,"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},{"value":0,"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},{"value":0,"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},{"value":2,"date":"2015-11-06 10:52:00.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"all":[{"value":0,"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},{"value":2,"date":"2015-10-16 10:53:42.000000000","_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},{"value":0,"_account_id":16523,"name":"ZhangLei","display_name":"ZhangLei","email":"zhang.lei@99cloud.net","username":"zhang.lei"},{"value":1,"date":"2015-11-06 10:20:46.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},{"value":0,"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},{"value":0,"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},{"value":0,"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"value":1,"date":"2015-10-16 11:30:32.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},{"value":1,"date":"2015-10-15 18:01:34.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},{"value":2,"date":"2015-11-06 09:53:40.000000000","_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"all":[{"value":0,"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},{"value":0,"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},{"value":0,"_account_id":16523,"name":"ZhangLei","display_name":"ZhangLei","email":"zhang.lei@99cloud.net","username":"zhang.lei"},{"value":0,"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},{"value":0,"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},{"value":0,"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},{"value":0,"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"value":0,"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},{"value":0,"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},{"value":1,"date":"2015-11-06 09:53:40.000000000","_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},{"_account_id":16523,"name":"ZhangLei","display_name":"ZhangLei","email":"zhang.lei@99cloud.net","username":"zhang.lei"},{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-09-24 08:02:23.000000000","updated_by":{"_account_id":16523,"name":"ZhangLei","display_name":"ZhangLei","email":"zhang.lei@99cloud.net","username":"zhang.lei"},"reviewer":{"_account_id":16523,"name":"ZhangLei","display_name":"ZhangLei","email":"zhang.lei@99cloud.net","username":"zhang.lei"},"state":"REVIEWER"},{"updated":"2015-09-24 15:23:59.000000000","updated_by":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"reviewer":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"state":"REVIEWER"},{"updated":"2015-09-24 19:42:30.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"},{"updated":"2015-10-08 16:40:58.000000000","updated_by":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"reviewer":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"state":"REVIEWER"},{"updated":"2015-10-09 13:35:19.000000000","updated_by":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"reviewer":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"state":"REVIEWER"},{"updated":"2015-10-16 10:53:42.000000000","updated_by":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"reviewer":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"state":"REVIEWER"},{"updated":"2015-10-16 11:30:32.000000000","updated_by":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"reviewer":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"state":"REVIEWER"},{"updated":"2015-11-06 09:53:40.000000000","updated_by":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"reviewer":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"state":"REVIEWER"},{"updated":"2015-11-06 10:20:46.000000000","updated_by":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"reviewer":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"state":"REVIEWER"},{"updated":"2015-11-06 10:52:00.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"290f76ec743b835ac47bbc7cf6d1dc1de241a1a9","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2015-09-21 14:27:39.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"5c70cf049323538d64f6b31681d41f06428146e8","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2015-09-21 14:56:04.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"b0061a033e4dc064962608d233e7181d5b42673a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-21 16:39:25.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-keystone-pep8 http://logs.openstack.org/38/225738/2/check/gate-keystone-pep8/6a68e4a/ : SUCCESS in 4m 52s\n- gate-keystone-docs http://docs-draft.openstack.org/38/225738/2/check/gate-keystone-docs/f9e8afe//doc/build/html/ : SUCCESS in 4m 50s\n- gate-keystone-python27 http://logs.openstack.org/38/225738/2/check/gate-keystone-python27/821ad7c/ : SUCCESS in 9m 44s\n- gate-keystone-python34 http://logs.openstack.org/38/225738/2/check/gate-keystone-python34/418a320/ : FAILURE in 4m 04s\n- gate-tempest-dsvm-full http://logs.openstack.org/38/225738/2/check/gate-tempest-dsvm-full/9f69c02/ : SUCCESS in 1h 03m 42s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/38/225738/2/check/gate-tempest-dsvm-postgres-full/f29e520/ : SUCCESS in 42m 51s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/38/225738/2/check/gate-tempest-dsvm-neutron-full/371d2a2/ : SUCCESS in 1h 03m 01s\n- gate-grenade-dsvm http://logs.openstack.org/38/225738/2/check/gate-grenade-dsvm/ff5cd43/ : SUCCESS in 52m 23s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/38/225738/2/check/gate-tempest-dsvm-large-ops/0be5b62/ : SUCCESS in 35m 07s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/38/225738/2/check/gate-tempest-dsvm-neutron-large-ops/408e0d4/ : SUCCESS in 34m 32s\n- gate-rally-dsvm-keystone http://logs.openstack.org/38/225738/2/check/gate-rally-dsvm-keystone/fdc39e3/ : SUCCESS in 37m 10s (non-voting)\n- gate-keystone-tox-bandit http://logs.openstack.org/38/225738/2/check/gate-keystone-tox-bandit/e87fc4f/ : SUCCESS in 2m 15s","accounts_in_message":[],"_revision_number":2},{"id":"1209091c9ef1913f9f19827c285b71f123352aef","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2015-09-21 22:34:15.000000000","message":"Patch Set 3: Patch Set 2 was rebased","accounts_in_message":[],"_revision_number":2},{"id":"e2e0206ed2850e7f50275f0da56ed283b07301e3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-21 23:59:47.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-keystone-pep8 http://logs.openstack.org/38/225738/3/check/gate-keystone-pep8/8e126d5/ : SUCCESS in 3m 50s\n- gate-keystone-docs http://docs-draft.openstack.org/38/225738/3/check/gate-keystone-docs/0d2463c//doc/build/html/ : SUCCESS in 6m 49s\n- gate-keystone-python27 http://logs.openstack.org/38/225738/3/check/gate-keystone-python27/5f5e127/ : SUCCESS in 7m 10s\n- gate-keystone-python34 http://logs.openstack.org/38/225738/3/check/gate-keystone-python34/e2cf4ca/ : SUCCESS in 5m 21s\n- gate-tempest-dsvm-full http://logs.openstack.org/38/225738/3/check/gate-tempest-dsvm-full/6de9ef0/ : SUCCESS in 42m 40s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/38/225738/3/check/gate-tempest-dsvm-postgres-full/dbb0183/ : SUCCESS in 55m 58s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/38/225738/3/check/gate-tempest-dsvm-neutron-full/7a96a4e/ : SUCCESS in 1h 20m 39s\n- gate-grenade-dsvm http://logs.openstack.org/38/225738/3/check/gate-grenade-dsvm/5a78d9e/ : SUCCESS in 37m 09s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/38/225738/3/check/gate-tempest-dsvm-large-ops/1e4024e/ : SUCCESS in 33m 00s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/38/225738/3/check/gate-tempest-dsvm-neutron-large-ops/b63e94d/ : SUCCESS in 31m 38s\n- gate-rally-dsvm-keystone http://logs.openstack.org/38/225738/3/check/gate-rally-dsvm-keystone/e7e0024/ : SUCCESS in 28m 04s (non-voting)\n- gate-keystone-tox-bandit http://logs.openstack.org/38/225738/3/check/gate-keystone-tox-bandit/92089a9/ : SUCCESS in 2m 33s","accounts_in_message":[],"_revision_number":3},{"id":"5bff8e0216245335c44abcfa56d76dc2f896680f","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2015-09-22 13:12:08.000000000","message":"Patch Set 4: Patch Set 3 was rebased","accounts_in_message":[],"_revision_number":3},{"id":"19701d2273aac246b8cef31c0d5b64755878c2fe","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2015-09-22 13:12:57.000000000","message":"Patch Set 3: Code-Review-1\n\n(2 comments)\n\nI have some comments around not documenting places where we have except/pass.\n\nI\u0027m wondering if we should at least provide a comment since it makes the justification easier to understand.","accounts_in_message":[],"_revision_number":3},{"id":"2f7963046fd3ffc3f9eb66d22dd1b63753abb464","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2015-09-22 13:13:12.000000000","message":"Patch Set 4: Code-Review-1","accounts_in_message":[],"_revision_number":4},{"id":"c47ff6d8b1ab78f1e91ec24eef81dc56178a7f2c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-22 14:12:23.000000000","message":"Patch Set 4: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-keystone-pep8 http://logs.openstack.org/38/225738/4/check/gate-keystone-pep8/8cdbaac/ : SUCCESS in 4m 03s\n- gate-keystone-docs http://docs-draft.openstack.org/38/225738/4/check/gate-keystone-docs/6034496//doc/build/html/ : SUCCESS in 5m 22s\n- gate-keystone-python27 http://logs.openstack.org/38/225738/4/check/gate-keystone-python27/b343dce/ : SUCCESS in 9m 25s\n- gate-keystone-python34 http://logs.openstack.org/38/225738/4/check/gate-keystone-python34/7387f1e/ : SUCCESS in 5m 01s\n- gate-tempest-dsvm-full http://logs.openstack.org/38/225738/4/check/gate-tempest-dsvm-full/c95c6da/ : SUCCESS in 45m 00s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/38/225738/4/check/gate-tempest-dsvm-postgres-full/8fbc9fe/ : SUCCESS in 49m 09s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/38/225738/4/check/gate-tempest-dsvm-neutron-full/1a979ef/ : SUCCESS in 56m 04s\n- gate-grenade-dsvm http://logs.openstack.org/38/225738/4/check/gate-grenade-dsvm/fd09f48/ : SUCCESS in 46m 45s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/38/225738/4/check/gate-tempest-dsvm-large-ops/1adeafc/ : SUCCESS in 32m 49s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/38/225738/4/check/gate-tempest-dsvm-neutron-large-ops/3ed3c5b/ : SUCCESS in 34m 01s\n- gate-rally-dsvm-keystone http://logs.openstack.org/38/225738/4/check/gate-rally-dsvm-keystone/6758bc0/ : SUCCESS in 26m 54s (non-voting)\n- gate-keystone-tox-bandit http://logs.openstack.org/38/225738/4/check/gate-keystone-tox-bandit/42f0756/ : FAILURE in 2m 40s","accounts_in_message":[],"_revision_number":4},{"id":"ed2c56e1e478a3a5cb0d891ad5527a901a66ec90","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2015-09-23 16:37:29.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"7e8dd846708a22f66f242cc4086bb869a1b8f4f0","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2015-09-23 16:37:47.000000000","message":"Patch Set 3:\n\n(2 comments)","accounts_in_message":[],"_revision_number":3},{"id":"aad8413c2c96fe064a1e16dc0595405910f178e0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-23 18:02:30.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-keystone-pep8 http://logs.openstack.org/38/225738/5/check/gate-keystone-pep8/27755b7/ : SUCCESS in 3m 18s\n- gate-keystone-docs http://docs-draft.openstack.org/38/225738/5/check/gate-keystone-docs/b9b3750//doc/build/html/ : SUCCESS in 7m 00s\n- gate-keystone-python27 http://logs.openstack.org/38/225738/5/check/gate-keystone-python27/351a413/ : SUCCESS in 9m 08s\n- gate-keystone-python34 http://logs.openstack.org/38/225738/5/check/gate-keystone-python34/cf933a1/ : SUCCESS in 5m 31s\n- gate-tempest-dsvm-full http://logs.openstack.org/38/225738/5/check/gate-tempest-dsvm-full/4b971df/ : SUCCESS in 48m 29s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/38/225738/5/check/gate-tempest-dsvm-postgres-full/7ce97eb/ : SUCCESS in 55m 48s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/38/225738/5/check/gate-tempest-dsvm-neutron-full/5fd9908/ : SUCCESS in 1h 20m 01s\n- gate-grenade-dsvm http://logs.openstack.org/38/225738/5/check/gate-grenade-dsvm/e5c1173/ : SUCCESS in 53m 46s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/38/225738/5/check/gate-tempest-dsvm-large-ops/cb62bec/ : SUCCESS in 27m 27s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/38/225738/5/check/gate-tempest-dsvm-neutron-large-ops/a8a2422/ : SUCCESS in 25m 00s\n- gate-rally-dsvm-keystone http://logs.openstack.org/38/225738/5/check/gate-rally-dsvm-keystone/e33bde7/ : SUCCESS in 26m 59s (non-voting)\n- gate-keystone-tox-bandit http://logs.openstack.org/38/225738/5/check/gate-keystone-tox-bandit/986599e/ : SUCCESS in 2m 30s","accounts_in_message":[],"_revision_number":5},{"id":"fdc227a3845f87c50fd93502ff7bf8bf47130163","author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"date":"2015-09-24 07:41:27.000000000","message":"Patch Set 5: Code-Review+2","accounts_in_message":[],"_revision_number":5},{"id":"1e95e686becab9854fa56f8b9691e2e6ac445479","author":{"_account_id":16523,"name":"ZhangLei","display_name":"ZhangLei","email":"zhang.lei@99cloud.net","username":"zhang.lei"},"date":"2015-09-24 08:02:23.000000000","message":"Patch Set 5: Code-Review+1","accounts_in_message":[],"_revision_number":5},{"id":"c6311ba68d1588b78990efe810ecdbc3f2d9a44b","author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"date":"2015-09-24 15:23:59.000000000","message":"Patch Set 5: Code-Review-1\n\n(2 comments)\n\nIf bandit is going to be adding value, then now is the time. Each of these except blocks that we\u0027re blindly adding #nosec to should be justified with an inline comment in the except block. Some already contain such comments, but many of them are empty. As a result, this patch would set a terrible precedent if it were to merge (inexplicably ignoring security concerns).\n\nIn fact, I don\u0027t think bandit should allow # nosec without any an adjacent inline comment at all...","accounts_in_message":[],"_revision_number":5},{"id":"52f9eac43f9c3b1af4edbd05fa0d1bf281f092ab","author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"date":"2015-09-24 19:42:30.000000000","message":"Patch Set 5: Code-Review-1\n\nvoting a -1 until dolph\u0027s comment is addressed","accounts_in_message":[],"_revision_number":5},{"id":"c8de885349bd8cc62f1782cb5534390dac2f26c3","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2015-09-28 18:54:02.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"4efdf10908034bdc336c2b0b30091fb636817248","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2015-09-28 19:02:54.000000000","message":"Patch Set 5:\n\n(2 comments)","accounts_in_message":[],"_revision_number":5},{"id":"5127e6e3a61df9ca30368ba730279ce1b4a324fb","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-09-28 20:11:39.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-keystone-pep8 http://logs.openstack.org/38/225738/6/check/gate-keystone-pep8/89bb4a7/ : SUCCESS in 4m 32s\n- gate-keystone-docs http://docs-draft.openstack.org/38/225738/6/check/gate-keystone-docs/f01a930//doc/build/html/ : SUCCESS in 5m 35s\n- gate-keystone-python27 http://logs.openstack.org/38/225738/6/check/gate-keystone-python27/22beb90/ : SUCCESS in 7m 43s\n- gate-keystone-python34 http://logs.openstack.org/38/225738/6/check/gate-keystone-python34/576ae0f/ : SUCCESS in 4m 55s\n- gate-tempest-dsvm-full http://logs.openstack.org/38/225738/6/check/gate-tempest-dsvm-full/044a4c7/ : SUCCESS in 48m 59s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/38/225738/6/check/gate-tempest-dsvm-postgres-full/f369b41/ : SUCCESS in 50m 41s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/38/225738/6/check/gate-tempest-dsvm-neutron-full/bde00ad/ : SUCCESS in 1h 15m 28s\n- gate-grenade-dsvm http://logs.openstack.org/38/225738/6/check/gate-grenade-dsvm/5b8bd86/ : SUCCESS in 41m 25s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/38/225738/6/check/gate-tempest-dsvm-large-ops/b2d79a8/ : SUCCESS in 45m 18s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/38/225738/6/check/gate-tempest-dsvm-neutron-large-ops/d4aea3d/ : SUCCESS in 34m 31s\n- gate-rally-dsvm-keystone http://logs.openstack.org/38/225738/6/check/gate-rally-dsvm-keystone/f63325c/ : SUCCESS in 34m 23s (non-voting)\n- gate-keystone-tox-bandit http://logs.openstack.org/38/225738/6/check/gate-keystone-tox-bandit/a1fe4b5/ : SUCCESS in 2m 27s","accounts_in_message":[],"_revision_number":6},{"id":"85865dde92209e6aeeedd4bd930beea5b346ba6c","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2015-09-30 14:35:24.000000000","message":"Patch Set 6: Code-Review-1\n\n(5 comments)","accounts_in_message":[],"_revision_number":6},{"id":"7f93d67466c764caf6bf3b85f19847d7d23d15d8","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2015-09-30 23:53:35.000000000","message":"Patch Set 6:\n\n(2 comments)","accounts_in_message":[],"_revision_number":6},{"id":"3808d024677215004c28dbc4b6ad244b3b5fa967","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2015-10-01 00:01:50.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"66d84b8c1741cdf6cdc76610bf1f917052194345","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-10-01 01:19:10.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-keystone-pep8 http://logs.openstack.org/38/225738/7/check/gate-keystone-pep8/4868312/ : SUCCESS in 3m 01s\n- gate-keystone-docs http://docs-draft.openstack.org/38/225738/7/check/gate-keystone-docs/af3b999//doc/build/html/ : SUCCESS in 5m 09s\n- gate-keystone-python27 http://logs.openstack.org/38/225738/7/check/gate-keystone-python27/7cc57f1/ : SUCCESS in 7m 07s\n- gate-keystone-python34 http://logs.openstack.org/38/225738/7/check/gate-keystone-python34/3ec077f/ : SUCCESS in 5m 33s\n- gate-tempest-dsvm-full http://logs.openstack.org/38/225738/7/check/gate-tempest-dsvm-full/ed233ce/ : SUCCESS in 53m 38s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/38/225738/7/check/gate-tempest-dsvm-postgres-full/4e7c2b9/ : SUCCESS in 55m 34s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/38/225738/7/check/gate-tempest-dsvm-neutron-full/bbf353a/ : SUCCESS in 1h 12m 42s\n- gate-grenade-dsvm http://logs.openstack.org/38/225738/7/check/gate-grenade-dsvm/597bda0/ : SUCCESS in 45m 54s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/38/225738/7/check/gate-tempest-dsvm-large-ops/5c452b4/ : SUCCESS in 37m 27s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/38/225738/7/check/gate-tempest-dsvm-neutron-large-ops/5df6e14/ : SUCCESS in 27m 45s\n- gate-rally-dsvm-keystone http://logs.openstack.org/38/225738/7/check/gate-rally-dsvm-keystone/7976053/ : SUCCESS in 24m 58s (non-voting)\n- gate-keystone-tox-bandit http://logs.openstack.org/38/225738/7/check/gate-keystone-tox-bandit/20e1358/ : SUCCESS in 1m 58s","accounts_in_message":[],"_revision_number":7},{"id":"9ae0a1699c7559b8544fceb3646efbde22074720","author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"date":"2015-10-01 08:11:57.000000000","message":"Patch Set 7: Code-Review+2\n\n(1 comment)\n\nOne minor nit, but otherwise good - thanks for taking the time to fill in all the comments so that each instance of this is clearly described as to why it\u0027s OK for #nosec.","accounts_in_message":[],"_revision_number":7},{"id":"a491d9ed06579cfc2f8eff4d87d42929613febc5","author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"date":"2015-10-04 02:39:08.000000000","message":"Patch Set 7: Code-Review+1\n\n(9 comments)\n\nLooks good, just some minor comments/nits inline","accounts_in_message":[],"_revision_number":7},{"id":"10bdb5c4d6a35738e6cefbf38f6f4a5c7060d1c7","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2015-10-06 13:48:22.000000000","message":"Patch Set 7:\n\n(5 comments)","accounts_in_message":[],"_revision_number":7},{"id":"0aeea5c945dab5957cd9ab6ac1101ee44481d823","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2015-10-06 13:50:34.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"e54e3f2c457be412737ffb59f270f0f8049d4638","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-10-06 15:24:59.000000000","message":"Patch Set 8: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-keystone-pep8 http://logs.openstack.org/38/225738/8/check/gate-keystone-pep8/32dc4da/ : SUCCESS in 4m 18s\n- gate-keystone-docs http://docs-draft.openstack.org/38/225738/8/check/gate-keystone-docs/f0f5e1a//doc/build/html/ : SUCCESS in 7m 44s\n- gate-keystone-python27 http://logs.openstack.org/38/225738/8/check/gate-keystone-python27/15fb2ab/ : SUCCESS in 9m 23s\n- gate-keystone-python34 http://logs.openstack.org/38/225738/8/check/gate-keystone-python34/7f97951/ : SUCCESS in 4m 10s\n- gate-tempest-dsvm-full http://logs.openstack.org/38/225738/8/check/gate-tempest-dsvm-full/82ebf6b/ : SUCCESS in 59m 36s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/38/225738/8/check/gate-tempest-dsvm-postgres-full/9b9f361/ : SUCCESS in 1h 04m 48s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/38/225738/8/check/gate-tempest-dsvm-neutron-full/6a9296d/ : SUCCESS in 1h 30m 12s\n- gate-grenade-dsvm http://logs.openstack.org/38/225738/8/check/gate-grenade-dsvm/df611d7/ : SUCCESS in 40m 17s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/38/225738/8/check/gate-tempest-dsvm-large-ops/df408a5/ : SUCCESS in 35m 40s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/38/225738/8/check/gate-tempest-dsvm-neutron-large-ops/1d689df/ : SUCCESS in 29m 41s\n- gate-rally-dsvm-keystone http://logs.openstack.org/38/225738/8/check/gate-rally-dsvm-keystone/a48d237/ : SUCCESS in 30m 10s (non-voting)\n- gate-keystone-tox-bandit http://logs.openstack.org/38/225738/8/check/gate-keystone-tox-bandit/f1f5179/ : SUCCESS in 1m 45s","accounts_in_message":[],"_revision_number":8},{"id":"904c7e7b8dbb181f165c9859133e3fbcf7efbf55","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2015-10-07 21:15:21.000000000","message":"Patch Set 8:\n\nneeds a rebase?","accounts_in_message":[],"_revision_number":8},{"id":"8c5575694d3c96d1dd9931f434c8d423f6d80af6","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2015-10-07 21:15:27.000000000","message":"Patch Set 9: Patch Set 8 was rebased","accounts_in_message":[],"_revision_number":8},{"id":"d168b072afac6254148ee677407851679702bfc8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-10-07 22:44:18.000000000","message":"Patch Set 9: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-keystone-pep8 http://logs.openstack.org/38/225738/9/check/gate-keystone-pep8/54c3ce4/ : SUCCESS in 3m 56s\n- gate-keystone-docs http://docs-draft.openstack.org/38/225738/9/check/gate-keystone-docs/204cd1d//doc/build/html/ : SUCCESS in 6m 51s\n- gate-keystone-python27 http://logs.openstack.org/38/225738/9/check/gate-keystone-python27/3ccef0a/ : SUCCESS in 9m 13s\n- gate-keystone-python34 http://logs.openstack.org/38/225738/9/check/gate-keystone-python34/2d3ff55/ : SUCCESS in 5m 52s\n- gate-tempest-dsvm-full http://logs.openstack.org/38/225738/9/check/gate-tempest-dsvm-full/3fa239a/ : SUCCESS in 40m 46s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/38/225738/9/check/gate-tempest-dsvm-postgres-full/aa6d8b6/ : SUCCESS in 39m 05s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/38/225738/9/check/gate-tempest-dsvm-neutron-full/3bf0a36/ : SUCCESS in 1h 09m 13s\n- gate-grenade-dsvm http://logs.openstack.org/38/225738/9/check/gate-grenade-dsvm/571b6c9/ : SUCCESS in 55m 51s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/38/225738/9/check/gate-tempest-dsvm-large-ops/08730c8/ : SUCCESS in 39m 05s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/38/225738/9/check/gate-tempest-dsvm-neutron-large-ops/6dc347f/ : SUCCESS in 33m 16s\n- gate-rally-dsvm-keystone http://logs.openstack.org/38/225738/9/check/gate-rally-dsvm-keystone/c373001/ : SUCCESS in 36m 02s (non-voting)\n- gate-keystone-tox-bandit http://logs.openstack.org/38/225738/9/check/gate-keystone-tox-bandit/1f77c8a/ : SUCCESS in 2m 31s","accounts_in_message":[],"_revision_number":9},{"id":"24f21541807d0cd5371d4f0e9e0571ef6c0a6875","author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"date":"2015-10-08 16:40:58.000000000","message":"Patch Set 9: Code-Review+1\n\nThese changes are fine, but I question the bandit test for try_except_pass as being some kind of security issue.  I think its a stretch.","accounts_in_message":[],"_revision_number":9},{"id":"21abcf217e1624bc5a063a2a971b1b41f8df8634","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2015-10-09 00:17:41.000000000","message":"Patch Set 9:\n\nI\u0027d be fine leaving this as a skipped test in keystone if others don\u0027t think this is useful. I like commenting why an exception is ignored but that\u0027s a code convention not a security issue.","accounts_in_message":[],"_revision_number":9},{"id":"e3f4bcf553e61364bd989a8ee9fe9970fc43c2da","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2015-10-09 13:35:19.000000000","message":"Patch Set 9:\n\nI agreed, I think having comments in the code explaining why we are passing on an exception is really useful.\n\nI don\u0027t think it\u0027s a bad idea to enable the check because it might catch unnecessary usage in the future. At least with the check, the future committer has to add #nosec and hopefully a comment before their patch merges.","accounts_in_message":[],"_revision_number":9},{"id":"13c46c86e6df72e69c6799f2e0160f3b0a6b2cd1","author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"date":"2015-10-15 18:01:34.000000000","message":"Uploaded patch set 10.","accounts_in_message":[],"_revision_number":10},{"id":"e78837567b7b689d89363a66fbc081d98e322059","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-10-15 19:07:04.000000000","message":"Patch Set 10: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-keystone-pep8 http://logs.openstack.org/38/225738/10/check/gate-keystone-pep8/5fe738e/ : SUCCESS in 4m 15s\n- gate-keystone-docs http://docs-draft.openstack.org/38/225738/10/check/gate-keystone-docs/1c7c8b2//doc/build/html/ : SUCCESS in 5m 53s\n- gate-keystone-python27 http://logs.openstack.org/38/225738/10/check/gate-keystone-python27/8dc3d3b/ : SUCCESS in 7m 15s\n- gate-keystone-python34 http://logs.openstack.org/38/225738/10/check/gate-keystone-python34/73ecac1/ : SUCCESS in 6m 25s\n- gate-tempest-dsvm-full http://logs.openstack.org/38/225738/10/check/gate-tempest-dsvm-full/53115da/ : SUCCESS in 1h 02m 03s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/38/225738/10/check/gate-tempest-dsvm-postgres-full/eabcefb/ : SUCCESS in 40m 10s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/38/225738/10/check/gate-tempest-dsvm-neutron-full/bd38662/ : SUCCESS in 52m 25s\n- gate-grenade-dsvm http://logs.openstack.org/38/225738/10/check/gate-grenade-dsvm/b2aedde/ : SUCCESS in 44m 37s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/38/225738/10/check/gate-tempest-dsvm-large-ops/50c2e93/ : SUCCESS in 38m 20s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/38/225738/10/check/gate-tempest-dsvm-neutron-large-ops/3182f72/ : SUCCESS in 25m 54s\n- gate-rally-dsvm-keystone http://logs.openstack.org/38/225738/10/check/gate-rally-dsvm-keystone/18c9c41/ : SUCCESS in 30m 00s (non-voting)\n- gate-keystone-tox-bandit http://logs.openstack.org/38/225738/10/check/gate-keystone-tox-bandit/959825f/ : SUCCESS in 2m 31s","accounts_in_message":[],"_revision_number":10},{"id":"b100e2460cc797f79cc86069f5cc4ab123d78291","author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"date":"2015-10-16 10:53:42.000000000","message":"Patch Set 10: Code-Review+2","accounts_in_message":[],"_revision_number":10},{"id":"36b5dd87773400da48acddb8a4d29683ebd836bc","author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"date":"2015-10-16 11:30:32.000000000","message":"Patch Set 10: Code-Review+1","accounts_in_message":[],"_revision_number":10},{"id":"83b4cd40f6efdb538febe1b5dda5ad0c93bde06c","author":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"date":"2015-11-06 09:53:40.000000000","message":"Patch Set 10: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":10},{"id":"1fad2bedd963c1a66a4cd864be92d08f6e953a33","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-11-06 09:53:46.000000000","message":"Patch Set 10: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":10},{"id":"9d1623ad9c25add15937add459951b92d3e6d765","author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"date":"2015-11-06 10:20:46.000000000","message":"Patch Set 10: Code-Review+1\n\nI am still fine with this as it is.","accounts_in_message":[],"_revision_number":10},{"id":"4852b26c61b1655ef876e8f57fb1a8d23b634154","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-11-06 10:52:00.000000000","message":"Patch Set 10: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-keystone-docs http://docs-draft.openstack.org/38/225738/10/gate/gate-keystone-docs/09ce584//doc/build/html/ : SUCCESS in 6m 41s\n- gate-keystone-pep8 http://logs.openstack.org/38/225738/10/gate/gate-keystone-pep8/065c319/ : SUCCESS in 3m 23s\n- gate-keystone-python27 http://logs.openstack.org/38/225738/10/gate/gate-keystone-python27/a58a189/ : SUCCESS in 6m 56s\n- gate-keystone-python34 http://logs.openstack.org/38/225738/10/gate/gate-keystone-python34/ea65b41/ : SUCCESS in 9m 50s\n- gate-tempest-dsvm-full http://logs.openstack.org/38/225738/10/gate/gate-tempest-dsvm-full/13db465/ : SUCCESS in 40m 53s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/38/225738/10/gate/gate-tempest-dsvm-postgres-full/9d9a4b8/ : SUCCESS in 39m 27s\n- gate-tempest-dsvm-neutron-full http://logs.openstack.org/38/225738/10/gate/gate-tempest-dsvm-neutron-full/1e5f8c8/ : SUCCESS in 57m 36s\n- gate-grenade-dsvm http://logs.openstack.org/38/225738/10/gate/gate-grenade-dsvm/e9be5a0/ : SUCCESS in 45m 08s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/38/225738/10/gate/gate-tempest-dsvm-large-ops/27fba25/ : SUCCESS in 30m 05s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/38/225738/10/gate/gate-tempest-dsvm-neutron-large-ops/52d2896/ : SUCCESS in 32m 51s\n- gate-keystone-tox-bandit http://logs.openstack.org/38/225738/10/gate/gate-keystone-tox-bandit/367b01d/ : SUCCESS in 3m 12s","accounts_in_message":[],"_revision_number":10},{"id":"c7f14b9fa4b795edb4395d88b166cfdc7c2a2430","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-11-06 10:52:02.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":10}],"current_revision_number":10,"current_revision":"068cba0047f56b300b5f20880a7f835a2f2342af","revisions":{"9431db856fa286b79d27383dfd83b9f0bc871dbe":{"kind":"REWORK","_number":1,"created":"2015-09-21 14:27:39.000000000","uploader":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"ref":"refs/changes/38/225738/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/38/225738/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/38/225738/1"}}},"commit":{"parents":[{"commit":"a1f584b4b2642cecd52712162ea401091d3faca9","subject":"Enable subprocess_without_shell_equals_true Bandit test","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/a1f584b4b2642cecd52712162ea401091d3faca9"}]}],"author":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-09-21 14:27:24.000000000","tz":-300},"committer":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-09-21 14:27:24.000000000","tz":-300},"subject":"Enable try_except_pass Bandit test","message":"Enable try_except_pass Bandit test\n\nThe try_except_pass test checks that pass isn\u0027t used in an except\nblock because it\u0027s the source of lots of security issues. The\ncurrent instances of pass in an except block are marked as nosec.\n\nChange-Id: I73af6b80fa75286e96943026b7b71ad23dc1786b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/9431db856fa286b79d27383dfd83b9f0bc871dbe"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/9431db856fa286b79d27383dfd83b9f0bc871dbe"}]},"branch":"refs/heads/master"},"ce9a84670585b3e8abf54d6fd51b6344160a8f03":{"kind":"NO_CHANGE","_number":2,"created":"2015-09-21 14:56:04.000000000","uploader":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"ref":"refs/changes/38/225738/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/38/225738/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/38/225738/2"}}},"commit":{"parents":[{"commit":"6a20be61b2a62209b7bcf02e4c9da576339d4974","subject":"Enable subprocess_without_shell_equals_true Bandit test","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/6a20be61b2a62209b7bcf02e4c9da576339d4974"}]}],"author":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-09-21 14:27:24.000000000","tz":-300},"committer":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-09-21 14:50:02.000000000","tz":-300},"subject":"Enable try_except_pass Bandit test","message":"Enable try_except_pass Bandit test\n\nThe try_except_pass test checks that pass isn\u0027t used in an except\nblock because it\u0027s the source of lots of security issues. The\ncurrent instances of pass in an except block are marked as nosec.\n\nChange-Id: I73af6b80fa75286e96943026b7b71ad23dc1786b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/ce9a84670585b3e8abf54d6fd51b6344160a8f03"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/ce9a84670585b3e8abf54d6fd51b6344160a8f03"}]},"branch":"refs/heads/master"},"25196f05858ef3915535b1045cf809236052e1f6":{"kind":"NO_CHANGE","_number":3,"created":"2015-09-21 22:34:15.000000000","uploader":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"ref":"refs/changes/38/225738/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/38/225738/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/38/225738/3"}}},"commit":{"parents":[{"commit":"62c74f20d181ee25ce3046dd4dbfde906cb2d8b1","subject":"Enable subprocess_without_shell_equals_true Bandit test","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/62c74f20d181ee25ce3046dd4dbfde906cb2d8b1"}]}],"author":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-09-21 14:27:24.000000000","tz":-300},"committer":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-09-21 22:34:15.000000000","tz":0},"subject":"Enable try_except_pass Bandit test","message":"Enable try_except_pass Bandit test\n\nThe try_except_pass test checks that pass isn\u0027t used in an except\nblock because it\u0027s the source of lots of security issues. The\ncurrent instances of pass in an except block are marked as nosec.\n\nChange-Id: I73af6b80fa75286e96943026b7b71ad23dc1786b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/25196f05858ef3915535b1045cf809236052e1f6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/25196f05858ef3915535b1045cf809236052e1f6"}]},"branch":"refs/heads/master"},"97d22071fbf37bdd6a4bfeaeb89d38566615851a":{"kind":"NO_CHANGE","_number":4,"created":"2015-09-22 13:12:08.000000000","uploader":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"ref":"refs/changes/38/225738/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/38/225738/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/38/225738/4"}}},"commit":{"parents":[{"commit":"efa26ca3162994c1fdf91efdba97a7e4244893cb","subject":"Enable subprocess_without_shell_equals_true Bandit test","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/efa26ca3162994c1fdf91efdba97a7e4244893cb"}]}],"author":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-09-21 14:27:24.000000000","tz":-300},"committer":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-09-22 13:12:07.000000000","tz":0},"subject":"Enable try_except_pass Bandit test","message":"Enable try_except_pass Bandit test\n\nThe try_except_pass test checks that pass isn\u0027t used in an except\nblock because it\u0027s the source of lots of security issues. The\ncurrent instances of pass in an except block are marked as nosec.\n\nChange-Id: I73af6b80fa75286e96943026b7b71ad23dc1786b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/97d22071fbf37bdd6a4bfeaeb89d38566615851a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/97d22071fbf37bdd6a4bfeaeb89d38566615851a"}]},"branch":"refs/heads/master"},"d5f1b61b7ae1a3f293a85237408d11c811ab86ad":{"kind":"REWORK","_number":5,"created":"2015-09-23 16:37:29.000000000","uploader":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"ref":"refs/changes/38/225738/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/38/225738/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/38/225738/5"}}},"commit":{"parents":[{"commit":"be0a827a48f15bdc1c303d200a3eb00879f542b4","subject":"Enable subprocess_without_shell_equals_true Bandit test","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/be0a827a48f15bdc1c303d200a3eb00879f542b4"}]}],"author":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-09-21 14:27:24.000000000","tz":-300},"committer":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-09-23 16:37:06.000000000","tz":-300},"subject":"Enable try_except_pass Bandit test","message":"Enable try_except_pass Bandit test\n\nThe try_except_pass test checks that pass isn\u0027t used in an except\nblock because it\u0027s the source of lots of security issues. The\ncurrent instances of pass in an except block are marked as nosec.\n\nChange-Id: I73af6b80fa75286e96943026b7b71ad23dc1786b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/d5f1b61b7ae1a3f293a85237408d11c811ab86ad"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/d5f1b61b7ae1a3f293a85237408d11c811ab86ad"}]},"branch":"refs/heads/master"},"6f7884c75bbdf49f6cd70b707544f353b9adda00":{"kind":"REWORK","_number":6,"created":"2015-09-28 18:54:02.000000000","uploader":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"ref":"refs/changes/38/225738/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/38/225738/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/38/225738/6"}}},"commit":{"parents":[{"commit":"47238765793fe81d06a9eb77188fd44b097bce5f","subject":"Enable subprocess_without_shell_equals_true Bandit test","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/47238765793fe81d06a9eb77188fd44b097bce5f"}]}],"author":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-09-21 14:27:24.000000000","tz":-300},"committer":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-09-28 18:53:09.000000000","tz":-300},"subject":"Enable try_except_pass Bandit test","message":"Enable try_except_pass Bandit test\n\nThe try_except_pass test checks that pass isn\u0027t used in an except\nblock because it\u0027s the source of lots of security issues. The\ncurrent instances of pass in an except block are marked as nosec.\n\nChange-Id: I73af6b80fa75286e96943026b7b71ad23dc1786b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/6f7884c75bbdf49f6cd70b707544f353b9adda00"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/6f7884c75bbdf49f6cd70b707544f353b9adda00"}]},"branch":"refs/heads/master"},"299e33a5acf729424d09c8f366e6e93997616b80":{"kind":"REWORK","_number":7,"created":"2015-10-01 00:01:50.000000000","uploader":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"ref":"refs/changes/38/225738/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/38/225738/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/38/225738/7"}}},"commit":{"parents":[{"commit":"84fd6f8bfa031d08160fe8723c2f3b2453f652d0","subject":"Enable subprocess_without_shell_equals_true Bandit test","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/84fd6f8bfa031d08160fe8723c2f3b2453f652d0"}]}],"author":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-09-21 14:27:24.000000000","tz":-300},"committer":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-10-01 00:00:53.000000000","tz":-300},"subject":"Enable try_except_pass Bandit test","message":"Enable try_except_pass Bandit test\n\nThe try_except_pass test checks that pass isn\u0027t used in an except\nblock because it\u0027s the source of lots of security issues. The\ncurrent instances of pass in an except block are marked as nosec.\n\nChange-Id: I73af6b80fa75286e96943026b7b71ad23dc1786b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/299e33a5acf729424d09c8f366e6e93997616b80"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/299e33a5acf729424d09c8f366e6e93997616b80"}]},"branch":"refs/heads/master"},"3acb3382ab809f8648ebc4e80da0608f66001f0d":{"kind":"REWORK","_number":8,"created":"2015-10-06 13:50:34.000000000","uploader":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"ref":"refs/changes/38/225738/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/38/225738/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/38/225738/8"}}},"commit":{"parents":[{"commit":"b4b6c699d822b2f0c47ca34ee5c95118780c59a3","subject":"Enable subprocess_without_shell_equals_true Bandit test","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/b4b6c699d822b2f0c47ca34ee5c95118780c59a3"}]}],"author":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-09-21 14:27:24.000000000","tz":-300},"committer":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-10-06 13:48:31.000000000","tz":-300},"subject":"Enable try_except_pass Bandit test","message":"Enable try_except_pass Bandit test\n\nThe try_except_pass test checks that pass isn\u0027t used in an except\nblock because it\u0027s the source of lots of security issues. The\ncurrent instances of pass in an except block are marked as nosec.\n\nChange-Id: I73af6b80fa75286e96943026b7b71ad23dc1786b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/3acb3382ab809f8648ebc4e80da0608f66001f0d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/3acb3382ab809f8648ebc4e80da0608f66001f0d"}]},"branch":"refs/heads/master"},"5f710aca280601f4a6b6dccfa839ba2a0faa3f44":{"kind":"TRIVIAL_REBASE","_number":9,"created":"2015-10-07 21:15:27.000000000","uploader":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"ref":"refs/changes/38/225738/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/38/225738/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/38/225738/9"}}},"commit":{"parents":[{"commit":"fe7d7818152df4bab73a50df56a0b9ecc5fccbec","subject":"Enable subprocess_without_shell_equals_true Bandit test","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/fe7d7818152df4bab73a50df56a0b9ecc5fccbec"}]}],"author":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-09-21 14:27:24.000000000","tz":-300},"committer":{"name":"Lance Bragstad","email":"lbragstad@gmail.com","date":"2015-10-07 21:15:27.000000000","tz":0},"subject":"Enable try_except_pass Bandit test","message":"Enable try_except_pass Bandit test\n\nThe try_except_pass test checks that pass isn\u0027t used in an except\nblock because it\u0027s the source of lots of security issues. The\ncurrent instances of pass in an except block are marked as nosec.\n\nChange-Id: I73af6b80fa75286e96943026b7b71ad23dc1786b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/5f710aca280601f4a6b6dccfa839ba2a0faa3f44"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/5f710aca280601f4a6b6dccfa839ba2a0faa3f44"}]},"branch":"refs/heads/master"},"068cba0047f56b300b5f20880a7f835a2f2342af":{"kind":"TRIVIAL_REBASE","_number":10,"created":"2015-10-15 18:01:34.000000000","uploader":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"ref":"refs/changes/38/225738/10","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/38/225738/10","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/10 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/10 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/38/225738/10 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/38/225738/10"}}},"commit":{"parents":[{"commit":"61397486a3ec2be36e1359cfff616327fc904155","subject":"Enable subprocess_without_shell_equals_true Bandit test","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/61397486a3ec2be36e1359cfff616327fc904155"}]}],"author":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-09-21 14:27:24.000000000","tz":-300},"committer":{"name":"Brant Knudson","email":"bknudson@us.ibm.com","date":"2015-10-15 17:56:35.000000000","tz":-300},"subject":"Enable try_except_pass Bandit test","message":"Enable try_except_pass Bandit test\n\nThe try_except_pass test checks that pass isn\u0027t used in an except\nblock because it\u0027s the source of lots of security issues. The\ncurrent instances of pass in an except block are marked as nosec.\n\nChange-Id: I73af6b80fa75286e96943026b7b71ad23dc1786b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/068cba0047f56b300b5f20880a7f835a2f2342af"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/068cba0047f56b300b5f20880a7f835a2f2342af"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}