)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"5eace52fdb7ef3a73a34ec89c584c9ef08573700","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Enable trusts for federated users"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"1. Add federated shadow user to groups on user creation"},{"line_number":10,"context_line":"2. Do not always check domains because federated user doesn\u0027t belong"},{"line_number":11,"context_line":"to a domain"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This should probably be broken into 2 patches"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba5201f7_e93884a2","line":11,"range":{"start_line":10,"start_character":0,"end_line":11,"end_character":11},"updated":"2017-01-06 01:32:28.000000000","message":"They will as soon as https://review.openstack.org/#/c/399684/ merges","commit_id":"8524f1fdb934fdfeed98de71e4241cf4733e54c5"},{"author":{"_account_id":18338,"name":"Ron De Rose","email":"ronald.de.rose@intel.com","username":"derosenet"},"change_message_id":"ca6d20caea8e400a1a5cf2754f4f3a71f7dbc1a6","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Enable trusts for federated users"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"1. Add federated shadow user to groups on user creation"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I2dd601b7bdc56f2c19628304095d0f7be0a33d87"},{"line_number":12,"context_line":"Closes-Bug: 1589993"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3a461143_1f1e73ba","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":55},"updated":"2017-01-26 03:38:13.000000000","message":"Why is this needed?","commit_id":"e7d59d6d647b4b5499cbde4bf49217e65d1073c8"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"6cf46d83318a5e8e0e20b6bf41e557f942f400e5","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Enable trusts for federated users"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add federated shadow user to groups on user creation and remove from"},{"line_number":10,"context_line":"old groups"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I2dd601b7bdc56f2c19628304095d0f7be0a33d87"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"1a430d35_1d507f31","line":10,"updated":"2017-02-08 22:59:20.000000000","message":"Per David\u0027s previous comment, did we come to consensus on how to move forward with this? I know we talked about it extensively in a recent meeting, but whatever the outcome was we should document it here somehow (so we can find it and reference it later).","commit_id":"dac6415fb66f99f134388efa274ebf0dca6fc932"}],"keystone/auth/plugins/mapped.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"c7b000a24bc082ffc1b015abd6664e461ab54fe1","unresolved":false,"context_lines":[{"line_number":157,"context_line":"                                                      display_name)"},{"line_number":158,"context_line":"            user_id \u003d user[\u0027id\u0027]"},{"line_number":159,"context_line":"            group_ids \u003d mapped_properties[\u0027group_ids\u0027]"},{"line_number":160,"context_line":"            for group_id in group_ids:"},{"line_number":161,"context_line":"                identity_api.add_user_to_group(user_id, group_id)"},{"line_number":162,"context_line":"            build_ephemeral_user_context(auth_context, user,"},{"line_number":163,"context_line":"                                         mapped_properties,"},{"line_number":164,"context_line":"                                         identity_provider, protocol)"}],"source_content_type":"text/x-python","patch_set":1,"id":"da4df55a_ad083d29","line":161,"range":{"start_line":160,"start_character":12,"end_line":161,"end_character":65},"updated":"2016-12-29 00:35:14.000000000","message":"eek, can\u0027t believe we weren\u0027t doing that","commit_id":"8524f1fdb934fdfeed98de71e4241cf4733e54c5"},{"author":{"_account_id":18338,"name":"Ron De Rose","email":"ronald.de.rose@intel.com","username":"derosenet"},"change_message_id":"ca6d20caea8e400a1a5cf2754f4f3a71f7dbc1a6","unresolved":false,"context_lines":[{"line_number":258,"context_line":""},{"line_number":259,"context_line":"            user_id \u003d user[\u0027id\u0027]"},{"line_number":260,"context_line":"            group_ids \u003d mapped_properties[\u0027group_ids\u0027]"},{"line_number":261,"context_line":"            for group_id in group_ids:"},{"line_number":262,"context_line":"                identity_api.add_user_to_group(user_id, group_id)"},{"line_number":263,"context_line":"            build_ephemeral_user_context(auth_context, user,"},{"line_number":264,"context_line":"                                         mapped_properties,"},{"line_number":265,"context_line":"                                         identity_provider, protocol)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a461143_fff28799","line":262,"range":{"start_line":261,"start_character":12,"end_line":262,"end_character":65},"updated":"2017-01-26 03:38:13.000000000","message":"What happens when the mapping changes, such as changing the groups? Will we remove group membership?","commit_id":"e7d59d6d647b4b5499cbde4bf49217e65d1073c8"}]}