)]}'
{"id":"openstack%2Fkeystone~674782","triplet_id":"openstack%2Fkeystone~master~I7874a70e6109219baee80309c3a27f8af9905a6d","project":"openstack/keystone","branch":"master","topic":"bug/1839133","hashtags":[],"change_id":"I7874a70e6109219baee80309c3a27f8af9905a6d","subject":"Honor group_members_are_ids for user_enabled_emulation","status":"MERGED","created":"2019-08-06 11:35:46.000000000","updated":"2019-09-20 07:18:57.000000000","submitted":"2019-09-20 01:50:21.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":12,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"674782-1568944221850-f2a16230","meta_rev_id":"30e2d73365f45a412dae6ed09b85dafed74e241a","_number":674782,"virtual_id_number":674782,"owner":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":8866,"name":"Raildo Mascena de Sousa Filho","email":"rmascena@redhat.com","username":"raildo"},{"value":0,"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},{"value":0,"date":"2019-08-14 19:09:05.000000000","_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},{"value":2,"date":"2019-09-20 01:50:21.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"}],"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":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"recommended":{"_account_id":8866,"name":"Raildo Mascena de Sousa Filho","email":"rmascena@redhat.com","username":"raildo"},"all":[{"value":1,"date":"2019-08-13 13:56:48.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":8866,"name":"Raildo Mascena de Sousa Filho","email":"rmascena@redhat.com","username":"raildo"},{"value":2,"date":"2019-09-19 17:48:07.000000000","_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},{"value":0,"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2019-08-14 18:32:56.000000000","_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"}],"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":{"approved":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"all":[{"value":0,"_account_id":8866,"name":"Raildo Mascena de Sousa Filho","email":"rmascena@redhat.com","username":"raildo"},{"value":0,"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},{"value":0,"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2019-09-19 22:42:42.000000000","_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},{"_account_id":8866,"name":"Raildo Mascena de Sousa Filho","email":"rmascena@redhat.com","username":"raildo"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-08-13 13:56:48.000000000","updated_by":{"_account_id":8866,"name":"Raildo Mascena de Sousa Filho","email":"rmascena@redhat.com","username":"raildo"},"reviewer":{"_account_id":8866,"name":"Raildo Mascena de Sousa Filho","email":"rmascena@redhat.com","username":"raildo"},"state":"REVIEWER"},{"updated":"2019-09-19 17:48:07.000000000","updated_by":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"reviewer":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"state":"REVIEWER"},{"updated":"2019-09-19 22:42:42.000000000","updated_by":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"reviewer":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"state":"REVIEWER"},{"updated":"2019-09-20 01:50:21.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":"a330c6588b2742435d4b66a64f947af5cb7c5ba4","author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"date":"2019-08-06 11:35:46.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"3ec171e11a50933d724b40184639827affee5e94","author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"date":"2019-08-06 11:41:09.000000000","message":"Patch Set 1:\n\nI\u0027m aware of missing covering unit test but need some guidance with it. I have to go deeper into the established testing infrastructure.","accounts_in_message":[],"_revision_number":1},{"id":"3a390a87a2439df5ebdcf3a7950477d2584c0cd6","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-06 16:54:23.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- openstack-tox-cover https://logs.opendev.org/82/674782/1/check/openstack-tox-cover/36973cd/cover/ : SUCCESS in 38m 41s\n- openstack-tox-lower-constraints https://logs.opendev.org/82/674782/1/check/openstack-tox-lower-constraints/b4d9cd7/ : SUCCESS in 37m 22s\n- openstack-tox-pep8 https://logs.opendev.org/82/674782/1/check/openstack-tox-pep8/4c91c56/ : SUCCESS in 5m 38s\n- openstack-tox-py27 https://logs.opendev.org/82/674782/1/check/openstack-tox-py27/63b9bb4/ : SUCCESS in 26m 21s\n- openstack-tox-py36 https://logs.opendev.org/82/674782/1/check/openstack-tox-py36/8927efb/ : SUCCESS in 29m 35s\n- openstack-tox-py37 https://logs.opendev.org/82/674782/1/check/openstack-tox-py37/c63de61/ : SUCCESS in 29m 28s\n- openstack-tox-docs https://logs.opendev.org/82/674782/1/check/openstack-tox-docs/b377a79/html/ : SUCCESS in 8m 09s\n- tempest-full https://logs.opendev.org/82/674782/1/check/tempest-full/36d672c/ : FAILURE in 1h 11m 02s\n- neutron-grenade https://logs.opendev.org/82/674782/1/check/neutron-grenade/a1a2adc/ : SUCCESS in 1h 05m 09s\n- grenade-py3 https://logs.opendev.org/82/674782/1/check/grenade-py3/9ace45e/ : SUCCESS in 1h 04m 45s\n- tempest-full-py3 https://logs.opendev.org/82/674782/1/check/tempest-full-py3/c9466aa/ : POST_FAILURE in 1h 34m 28s\n- build-openstack-releasenotes https://logs.opendev.org/82/674782/1/check/build-openstack-releasenotes/a12db08/html/ : SUCCESS in 12m 32s\n- keystone-dsvm-functional https://logs.opendev.org/82/674782/1/check/keystone-dsvm-functional/1f8be60/ : SUCCESS in 43m 15s\n- keystone-dsvm-py3-functional https://logs.opendev.org/82/674782/1/check/keystone-dsvm-py3-functional/066bf1a/ : SUCCESS in 35m 14s\n- keystone-dsvm-functional-federation-opensuse15 https://logs.opendev.org/82/674782/1/check/keystone-dsvm-functional-federation-opensuse15/41ed576/ : SUCCESS in 45m 38s (non-voting)\n- keystone-dsvm-py3-functional-federation-opensuse15 https://logs.opendev.org/82/674782/1/check/keystone-dsvm-py3-functional-federation-opensuse15/8918948/ : SUCCESS in 48m 42s (non-voting)\n- keystoneclient-devstack-functional https://logs.opendev.org/82/674782/1/check/keystoneclient-devstack-functional/fe3ce3a/ : SUCCESS in 17m 34s (non-voting)\n- legacy-tempest-dsvm-ldap-domain-specific-driver https://logs.opendev.org/82/674782/1/check/legacy-tempest-dsvm-ldap-domain-specific-driver/4dd6eb9/ : FAILURE in 1h 13m 02s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"da1c1d3cc042d37910b32b085661559dcf80f22e","author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"date":"2019-08-06 17:37:24.000000000","message":"Patch Set 1:\n\nrecheck\n\nirrelevant failures","accounts_in_message":[],"_revision_number":1},{"id":"76e2712d0e70bcb669fc759808399f0005c9e34d","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-06 22:35:28.000000000","message":"Patch Set 1:\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- openstack-tox-cover https://logs.opendev.org/82/674782/1/check/openstack-tox-cover/63bdbe6/cover/ : SUCCESS in 44m 35s\n- openstack-tox-lower-constraints https://logs.opendev.org/82/674782/1/check/openstack-tox-lower-constraints/0a03372/ : FAILURE in 38m 00s\n- openstack-tox-pep8 https://logs.opendev.org/82/674782/1/check/openstack-tox-pep8/5d2913b/ : SUCCESS in 7m 04s\n- openstack-tox-py27 https://logs.opendev.org/82/674782/1/check/openstack-tox-py27/b35fad5/ : TIMED_OUT in 41m 38s\n- openstack-tox-py36 https://logs.opendev.org/82/674782/1/check/openstack-tox-py36/1d0c9a4/ : TIMED_OUT in 42m 37s\n- openstack-tox-py37 https://logs.opendev.org/82/674782/1/check/openstack-tox-py37/d153509/ : SUCCESS in 35m 53s\n- openstack-tox-docs https://logs.opendev.org/82/674782/1/check/openstack-tox-docs/d1663f4/html/ : SUCCESS in 12m 15s\n- tempest-full https://logs.opendev.org/82/674782/1/check/tempest-full/9667ba6/ : SUCCESS in 1h 48m 30s\n- neutron-grenade https://logs.opendev.org/82/674782/1/check/neutron-grenade/18e664e/ : SUCCESS in 58m 02s\n- grenade-py3 https://logs.opendev.org/82/674782/1/check/grenade-py3/6e8aa30/ : SUCCESS in 1h 03m 03s\n- tempest-full-py3 https://logs.opendev.org/82/674782/1/check/tempest-full-py3/5560399/ : SUCCESS in 1h 23m 34s\n- build-openstack-releasenotes https://logs.opendev.org/82/674782/1/check/build-openstack-releasenotes/1d22851/html/ : SUCCESS in 10m 20s\n- keystone-dsvm-functional https://logs.opendev.org/82/674782/1/check/keystone-dsvm-functional/7bca159/ : SUCCESS in 41m 43s\n- keystone-dsvm-py3-functional https://logs.opendev.org/82/674782/1/check/keystone-dsvm-py3-functional/fae3e19/ : SUCCESS in 39m 10s\n- keystone-dsvm-functional-federation-opensuse15 https://logs.opendev.org/82/674782/1/check/keystone-dsvm-functional-federation-opensuse15/41dc68b/ : SUCCESS in 51m 41s (non-voting)\n- keystone-dsvm-py3-functional-federation-opensuse15 https://logs.opendev.org/82/674782/1/check/keystone-dsvm-py3-functional-federation-opensuse15/00d916f/ : SUCCESS in 48m 27s (non-voting)\n- keystoneclient-devstack-functional https://logs.opendev.org/82/674782/1/check/keystoneclient-devstack-functional/395de92/ : SUCCESS in 16m 55s (non-voting)\n- legacy-tempest-dsvm-ldap-domain-specific-driver https://logs.opendev.org/82/674782/1/check/legacy-tempest-dsvm-ldap-domain-specific-driver/bbaff06/ : SUCCESS in 1h 26m 07s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"f7e3a70141f203d9780078610ecefee975a8d1eb","author":{"_account_id":8866,"name":"Raildo Mascena de Sousa Filho","email":"rmascena@redhat.com","username":"raildo"},"date":"2019-08-08 11:58:16.000000000","message":"Patch Set 1: Code-Review-1\n\nHey Radosław, first of all, thanks for the patch, We have a ldap mock server on this file https://github.com/openstack/keystone/blob/master/keystone/tests/unit/test_backend_ldap.py and there are a lot of cases similar to what you need to test for this case. If you need some help implementing it, please let me know.","accounts_in_message":[],"_revision_number":1},{"id":"4da86b51afa7fc59af694354890c7c560ddbcc5c","author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"date":"2019-08-12 14:43:03.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"396902fdecdd21290c92534de4e1b8a5e2ffd9e1","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-12 18:45:04.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/a443a84b0ddb492689358241846411fb : SUCCESS in 30m 06s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/2c059dfa49a54e3b988681f55ba79480 : SUCCESS in 32m 31s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/8142094c144945268300e231e123c437 : SUCCESS in 6m 59s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/e9ab5e28721e40099eae2114f3b1ccf6 : SUCCESS in 30m 34s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/c91b900128eb45219617b13ba26dd9ec : SUCCESS in 28m 40s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/92f82f7ffdbf4ffbac44867dc22a322f : SUCCESS in 26m 25s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/d6a0a3a683e147aeab206d699e776eae : SUCCESS in 7m 50s\n- tempest-full https://zuul.opendev.org/t/openstack/build/8bfcdde02a5540c8a419127d0e228b32 : SUCCESS in 1h 30m 29s\n- neutron-grenade https://zuul.opendev.org/t/openstack/build/d44b705d7c9344bb8105d801ce787bd1 : SUCCESS in 53m 17s\n- grenade-py3 https://zuul.opendev.org/t/openstack/build/5943e9c2341e4b94b89a4e892dbb4df4 : SUCCESS in 57m 15s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/ef5b9c0b2fec4ce9ac4d69e390f9e1f7 : SUCCESS in 1h 35m 28s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/b8e9b4761eac4b3ca7f0f92537d6df71 : SUCCESS in 10m 13s\n- keystone-dsvm-functional https://zuul.opendev.org/t/openstack/build/d14fbe6c41ae4dd7b5064bc0a46de1a4 : SUCCESS in 32m 16s\n- keystone-dsvm-py3-functional https://zuul.opendev.org/t/openstack/build/e5c2c17adc324483939c480f67906930 : SUCCESS in 32m 39s\n- keystone-dsvm-functional-federation-opensuse15 https://zuul.opendev.org/t/openstack/build/68d52af6c7794deab02b90b88b059f9a : SUCCESS in 37m 04s (non-voting)\n- keystone-dsvm-py3-functional-federation-opensuse15 https://zuul.opendev.org/t/openstack/build/8e2e1030adce415cb2903751457eb420 : SUCCESS in 41m 29s (non-voting)\n- keystoneclient-devstack-functional https://zuul.opendev.org/t/openstack/build/e77361e80ae24d8ca08c061b193f3d53 : FAILURE in 18m 36s (non-voting)\n- legacy-tempest-dsvm-ldap-domain-specific-driver https://zuul.opendev.org/t/openstack/build/409c5ccea4ae428197cd5508b10f90af : SUCCESS in 1h 45m 59s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"b5b2d301900bd3e1df87e2f9c4c516bec4241aa1","author":{"_account_id":8866,"name":"Raildo Mascena de Sousa Filho","email":"rmascena@redhat.com","username":"raildo"},"date":"2019-08-13 13:56:48.000000000","message":"Patch Set 2: Code-Review+1\n\nI\u0027m fine with this patch, I\u0027m just wondering if we need to update our docs explaining how to use this configuration, or things will be fine just with the release note, since, I suppose this option was already supported before.","accounts_in_message":[],"_revision_number":2},{"id":"7e9ddf75e5deff2865b3587d2e41cab3567c5bad","author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"date":"2019-08-13 17:24:58.000000000","message":"Patch Set 2:\n\n\u003e I\u0027m fine with this patch, I\u0027m just wondering if we need to update\n \u003e our docs explaining how to use this configuration, or things will\n \u003e be fine just with the release note, since, I suppose this option\n \u003e was already supported before.\n\nThe docs are already suggesting user_enabled_emulation_use_group_config would honor any relevant group config. They don\u0027t make exceptions.","accounts_in_message":[],"_revision_number":2},{"id":"345e28d829427b9ed66565d9688a27fd1f437998","author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"date":"2019-08-13 17:45:05.000000000","message":"Patch Set 2: Code-Review-1\n\n(4 comments)","accounts_in_message":[],"_revision_number":2},{"id":"0ff05d3bcd6866e6d726a603e2cfddb757ce3155","author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"date":"2019-08-13 17:56:41.000000000","message":"Patch Set 2:\n\n(4 comments)","accounts_in_message":[],"_revision_number":2},{"id":"af66cad64617c960a01b9733668c2bb1a35e6e78","author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"date":"2019-08-13 19:47:10.000000000","message":"Patch Set 2:\n\n(3 comments)\n\nAlso, we don\u0027t need the logic pertaining to writable LDAP as we don\u0027t support writable LDAP anymore. Probably should remove those code in a follow-on patch.","accounts_in_message":[],"_revision_number":2},{"id":"9969364c1d59af91879d1748de721a1409b2d8e5","author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"date":"2019-08-14 06:23:10.000000000","message":"Patch Set 2:\n\n(1 comment)\n\nGuang, I agree with all your statements in general. This patch is prepared as a minimal fix so that it can be backported pretty easily to all supported branches with this functionality. I don\u0027t think there is anything for me to change in it despite your standing CR-1. Please clarify whether you\u0027d like me to change something in this one, or you just want some follow-ups to come. Please rate accordingly. Thank you.","accounts_in_message":[],"_revision_number":2},{"id":"76dae7edacae98a6ae260e5aaf4b411ffc8c4d68","author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"date":"2019-08-14 18:32:56.000000000","message":"Patch Set 2: Code-Review+2\n\nChanging my vote. Yes I agreed those are outside the scope of this particular fix. Though I think there are definitely rooms for improvement regarding the feature itself. We can do those as follow-ons.","accounts_in_message":[],"_revision_number":2},{"id":"6b921ccdffc1ba01e45fa855904b65377dd99a82","author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"date":"2019-08-14 19:09:05.000000000","message":"Patch Set 2:\n\n\u003e Changing my vote. Yes I agreed those are outside the scope of this\n \u003e particular fix. Though I think there are definitely rooms for\n \u003e improvement regarding the feature itself. We can do those as\n \u003e follow-ons.\n\nThanks. Please let me know where we could coordinate this. I am lurking on #keystone as yoctozepto. UTC+2 now","accounts_in_message":[],"_revision_number":2},{"id":"73196d921f1472a42acec5ab001e519c229a2349","author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"date":"2019-09-19 17:48:07.000000000","message":"Patch Set 2: Code-Review+2\n\nReally ok with this.","accounts_in_message":[],"_revision_number":2},{"id":"a924d008f4f8a9566f3bcbb60f69b99208830356","author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"date":"2019-09-19 22:42:42.000000000","message":"Patch Set 2: Workflow+1","accounts_in_message":[],"_revision_number":2},{"id":"ba55fe5de999a71de7cd0ce6181ba6c59a366fd8","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-19 22:43:05.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"7f7fbe69bb3f3dfe6521668a5b0396fdd68c3a2e","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-20 01:50:21.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2},{"id":"6656e26ca227255f2db78888be17d7211acf2596","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-20 01:50:21.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/4fd5ae594d354bc782d92eceb0a11356 : SUCCESS in 13m 45s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/33d52dadceb24d518b4036b008a9cda2 : SUCCESS in 6m 40s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/c9b32cc4d49c4576a80c0bebee71f567 : SUCCESS in 15m 49s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/cc0d8daceba3447086b5da45f6b4d920 : SUCCESS in 32m 20s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/982432165270462fa89f16a5612dece5 : SUCCESS in 15m 15s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/a639103802e244e7957853bba7644f62 : SUCCESS in 20m 47s\n- tempest-full https://zuul.opendev.org/t/openstack/build/56c4a2add6f34d54a7e0e4f3f01498d1 : SUCCESS in 1h 46m 02s\n- neutron-grenade https://zuul.opendev.org/t/openstack/build/64aed8b52fb34d37b938b425b6e1012f : SUCCESS in 57m 44s\n- grenade-py3 https://zuul.opendev.org/t/openstack/build/f8140c3f1a4140a5a3d639b86c90d18f : SUCCESS in 51m 19s\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/6dbc9532b2cc4894909f2e3bdf38e159 : SUCCESS in 1h 25m 22s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/68ccb1e90d774535aea6649e96af11b6 : SUCCESS in 13m 33s\n- keystone-dsvm-functional https://zuul.opendev.org/t/openstack/build/6963e1b1362e4241b8ff0a7faa02b853 : SUCCESS in 40m 58s\n- keystone-dsvm-py3-functional https://zuul.opendev.org/t/openstack/build/40ce5dcb22b54a8fa89c686db9d4073a : SUCCESS in 28m 42s\n- tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/c41a3dbaa5f744d2ba44523bfd6ffe7e : SUCCESS in 1h 13m 07s\n- keystone-tox-protection https://zuul.opendev.org/t/openstack/build/3561aa037fa54077bf20bf4cbd93a4e0 : SUCCESS in 29m 11s","accounts_in_message":[],"_revision_number":2},{"id":"e01be35e186de0bc82516b61aadcfe8d100615e8","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-20 01:54:17.000000000","message":"Patch Set 2:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/e9adf8b6b0a247c38bc1f2d19a00c1f5 : SUCCESS in 1m 34s\n- promote-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/7612179d963944e0bfe85f96736ac301 : SUCCESS in 51s","accounts_in_message":[],"_revision_number":2},{"id":"6d7f3b9971898dbff437a2b3b0af0be01959442f","author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"date":"2019-09-20 07:18:57.000000000","message":"Patch Set 2: Cherry Picked\n\nThis patchset was cherry picked to branch stable/stein as commit 2a4d48f3670bf11513ad8a9c50850469684c29c8","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"c7fae97d873f72068ca65538ec5b5919c0ac7d5a","revisions":{"1567501f5a46401f8c9cadb89946c7a6a20dc836":{"kind":"REWORK","_number":1,"created":"2019-08-06 11:35:46.000000000","uploader":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"ref":"refs/changes/82/674782/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/82/674782/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/82/674782/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/82/674782/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/82/674782/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/82/674782/1"}}},"commit":{"parents":[{"commit":"73045dfbcaeeb32a4ff7a026e19e8796e7261246","subject":"Merge \"Followup for remove signing[config].\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/73045dfbcaeeb32a4ff7a026e19e8796e7261246"}]}],"author":{"name":"Radosław Piliszek","email":"radoslaw.piliszek@gmail.com","date":"2019-08-06 11:25:17.000000000","tz":120},"committer":{"name":"Radosław Piliszek","email":"radoslaw.piliszek@gmail.com","date":"2019-08-06 11:35:44.000000000","tz":120},"subject":"Honor group_members_are_ids for user_enabled_emulation","message":"Honor group_members_are_ids for user_enabled_emulation\n\nApplied when group config is to be honored\n(i.e. set user_enabled_emulation_use_group_config).\nConditionals follow usage of group_members_are_ids.\n\nAdditionally some TODOs are added for observed potential issues.\n\nChange-Id: I7874a70e6109219baee80309c3a27f8af9905a6d\nCloses-Bug: #1839133\nSigned-off-by: Radosław Piliszek \u003cradoslaw.piliszek@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/1567501f5a46401f8c9cadb89946c7a6a20dc836"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/1567501f5a46401f8c9cadb89946c7a6a20dc836"}]},"branch":"refs/heads/master"},"c7fae97d873f72068ca65538ec5b5919c0ac7d5a":{"kind":"REWORK","_number":2,"created":"2019-08-12 14:43:03.000000000","uploader":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"ref":"refs/changes/82/674782/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/keystone","ref":"refs/changes/82/674782/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/keystone refs/changes/82/674782/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/keystone refs/changes/82/674782/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/keystone refs/changes/82/674782/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/keystone refs/changes/82/674782/2"}}},"commit":{"parents":[{"commit":"73045dfbcaeeb32a4ff7a026e19e8796e7261246","subject":"Merge \"Followup for remove signing[config].\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/73045dfbcaeeb32a4ff7a026e19e8796e7261246"}]}],"author":{"name":"Radosław Piliszek","email":"radoslaw.piliszek@gmail.com","date":"2019-08-06 11:25:17.000000000","tz":120},"committer":{"name":"Radosław Piliszek","email":"radoslaw.piliszek@gmail.com","date":"2019-08-12 14:41:56.000000000","tz":120},"subject":"Honor group_members_are_ids for user_enabled_emulation","message":"Honor group_members_are_ids for user_enabled_emulation\n\nApplied when group config is to be honored\n(i.e. set user_enabled_emulation_use_group_config).\nConditionals follow usage of group_members_are_ids.\n\nAdded new test for the case with ids.\nIt fails without fix.\nThe original test expanded to ensure the change did not\nbreak its internals either.\nIt passes without fix as well.\n\nAdditionally some TODOs are added for observed potential issues.\n\nChange-Id: I7874a70e6109219baee80309c3a27f8af9905a6d\nCloses-Bug: #1839133\nSigned-off-by: Radosław Piliszek \u003cradoslaw.piliszek@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/c7fae97d873f72068ca65538ec5b5919c0ac7d5a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/keystone/commit/c7fae97d873f72068ca65538ec5b5919c0ac7d5a"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}