)]}'
{"keystone/common/config.py":[{"author":{"_account_id":11045,"name":"Bob Thyne","email":"bob.thyne@hp.com","username":"bobt"},"change_message_id":"ed0c12712fccaa1d63c48f4bc2dc1bcb8012ded5","unresolved":false,"context_lines":[{"line_number":92,"context_line":"                        \u0027exception for token values.\u0027),"},{"line_number":93,"context_line":"        cfg.StrOpt(\u0027member_role_id\u0027,"},{"line_number":94,"context_line":"                   default\u003d\u00279fe2ff9ee4384b1894a90878d3e92bab\u0027,"},{"line_number":95,"context_line":"                   help\u003d\u0027Similar to the member_role_name option, this \u0027"},{"line_number":96,"context_line":"                        \u0027represents the default role ID used to associate \u0027"},{"line_number":97,"context_line":"                        \u0027users with their default projects in the v2 API. \u0027"},{"line_number":98,"context_line":"                        \u0027During a database migration, this will be used to \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_df6433b5","line":95,"updated":"2014-07-31 04:57:33.000000000","message":"Instead of using \"this\" it would be clearer to explicitly reference member_role_id.","commit_id":"dd6b0089b7943c0d7dd716120e9def2db5dac2df"},{"author":{"_account_id":11333,"name":"David J Hu","email":"david.hu@hpe.com","username":"dhu"},"change_message_id":"ce77888108397f4e82f79cd0c9d8b186d254a87b","unresolved":false,"context_lines":[{"line_number":94,"context_line":"                   default\u003d\u00279fe2ff9ee4384b1894a90878d3e92bab\u0027,"},{"line_number":95,"context_line":"                   help\u003d\u0027Similar to the member_role_name option, this \u0027"},{"line_number":96,"context_line":"                        \u0027represents the default role ID used to associate \u0027"},{"line_number":97,"context_line":"                        \u0027users with their default projects in the v2 API. \u0027"},{"line_number":98,"context_line":"                        \u0027During a database migration, this will be used to \u0027"},{"line_number":99,"context_line":"                        \u0027create a new role that will replace records in the \u0027"},{"line_number":100,"context_line":"                        \u0027assignment table with explicit role assignments. \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_c5ef04b0","line":97,"updated":"2014-07-31 06:49:40.000000000","message":"So, v3 API as well?","commit_id":"dd6b0089b7943c0d7dd716120e9def2db5dac2df"},{"author":{"_account_id":11045,"name":"Bob Thyne","email":"bob.thyne@hp.com","username":"bobt"},"change_message_id":"ed0c12712fccaa1d63c48f4bc2dc1bcb8012ded5","unresolved":false,"context_lines":[{"line_number":95,"context_line":"                   help\u003d\u0027Similar to the member_role_name option, this \u0027"},{"line_number":96,"context_line":"                        \u0027represents the default role ID used to associate \u0027"},{"line_number":97,"context_line":"                        \u0027users with their default projects in the v2 API. \u0027"},{"line_number":98,"context_line":"                        \u0027During a database migration, this will be used to \u0027"},{"line_number":99,"context_line":"                        \u0027create a new role that will replace records in the \u0027"},{"line_number":100,"context_line":"                        \u0027assignment table with explicit role assignments. \u0027"},{"line_number":101,"context_line":"                        \u0027After that migration, this will be used as the \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_ff02d7b8","line":98,"updated":"2014-07-31 04:57:33.000000000","message":"Same as above","commit_id":"dd6b0089b7943c0d7dd716120e9def2db5dac2df"},{"author":{"_account_id":11045,"name":"Bob Thyne","email":"bob.thyne@hp.com","username":"bobt"},"change_message_id":"ed0c12712fccaa1d63c48f4bc2dc1bcb8012ded5","unresolved":false,"context_lines":[{"line_number":98,"context_line":"                        \u0027During a database migration, this will be used to \u0027"},{"line_number":99,"context_line":"                        \u0027create a new role that will replace records in the \u0027"},{"line_number":100,"context_line":"                        \u0027assignment table with explicit role assignments. \u0027"},{"line_number":101,"context_line":"                        \u0027After that migration, this will be used as the \u0027"},{"line_number":102,"context_line":"                        \u0027explicit role where one is not specified by the v2 \u0027"},{"line_number":103,"context_line":"                        \u0027API.\u0027),"},{"line_number":104,"context_line":"        cfg.StrOpt(\u0027member_role_name\u0027, default\u003d\u0027_member_\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_1f06abcd","line":101,"updated":"2014-07-31 04:57:33.000000000","message":"After the migration ...\n\n\"this\" \u003d\u003e member_role_id","commit_id":"dd6b0089b7943c0d7dd716120e9def2db5dac2df"},{"author":{"_account_id":11333,"name":"David J Hu","email":"david.hu@hpe.com","username":"dhu"},"change_message_id":"ce77888108397f4e82f79cd0c9d8b186d254a87b","unresolved":false,"context_lines":[{"line_number":98,"context_line":"                        \u0027During a database migration, this will be used to \u0027"},{"line_number":99,"context_line":"                        \u0027create a new role that will replace records in the \u0027"},{"line_number":100,"context_line":"                        \u0027assignment table with explicit role assignments. \u0027"},{"line_number":101,"context_line":"                        \u0027After that migration, this will be used as the \u0027"},{"line_number":102,"context_line":"                        \u0027explicit role where one is not specified by the v2 \u0027"},{"line_number":103,"context_line":"                        \u0027API.\u0027),"},{"line_number":104,"context_line":"        cfg.StrOpt(\u0027member_role_name\u0027, default\u003d\u0027_member_\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_85798c32","line":101,"in_reply_to":"baada198_1f06abcd","updated":"2014-07-31 06:49:40.000000000","message":"s/that/the","commit_id":"dd6b0089b7943c0d7dd716120e9def2db5dac2df"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"42d63072fcefc539420dc0cc93a3d40b9108aafc","unresolved":false,"context_lines":[{"line_number":95,"context_line":"                   help\u003d\u0027Similar to the member_role_name option, this \u0027"},{"line_number":96,"context_line":"                        \u0027represents the default role ID used to associate \u0027"},{"line_number":97,"context_line":"                        \u0027users with their default projects in the v2 API. \u0027"},{"line_number":98,"context_line":"                        \u0027During a database migration, this will be used to \u0027"},{"line_number":99,"context_line":"                        \u0027create a new role that will replace records in the \u0027"},{"line_number":100,"context_line":"                        \u0027assignment table with explicit role assignments. \u0027"},{"line_number":101,"context_line":"                        \u0027After that migration, this will be used as the \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"da9df570_bcddfa44","line":98,"updated":"2014-09-22 11:48:27.000000000","message":"Maybe I\u0027m just missing it, but I can\u0027t see where we do this.  All I can see member_role_id/name be used for is if we call the v2 update_user method (and we are changing default tenant), or we call the v2 create_user method for for a new user. What\u0027s database migration got to do with it?","commit_id":"ad5cdaee560d4559b3f7931925b850f5dc3d3413"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"e906f724b2ec64b37fb8330550b516dd921c4e20","unresolved":false,"context_lines":[{"line_number":95,"context_line":"                   help\u003d\u0027Similar to the member_role_name option, this \u0027"},{"line_number":96,"context_line":"                        \u0027represents the default role ID used to associate \u0027"},{"line_number":97,"context_line":"                        \u0027users with their default projects in the v2 API. \u0027"},{"line_number":98,"context_line":"                        \u0027During a database migration, this will be used to \u0027"},{"line_number":99,"context_line":"                        \u0027create a new role that will replace records in the \u0027"},{"line_number":100,"context_line":"                        \u0027assignment table with explicit role assignments. \u0027"},{"line_number":101,"context_line":"                        \u0027After that migration, this will be used as the \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"baa201ad_49ffb767","line":98,"in_reply_to":"baa201ad_336312b3","updated":"2014-10-01 21:08:06.000000000","message":"yes it is still needed as it\u0027s used during v2 user creation, but apparently not during a migration any longer (which is unnecessary, as it\u0027s created on demand anyway)","commit_id":"ad5cdaee560d4559b3f7931925b850f5dc3d3413"},{"author":{"_account_id":6772,"name":"Stephen Gordon","email":"sgordon@redhat.com","username":"sgordon"},"change_message_id":"83572d95388320db6ba969a0165b8e569b4c4ca8","unresolved":false,"context_lines":[{"line_number":95,"context_line":"                   help\u003d\u0027Similar to the member_role_name option, this \u0027"},{"line_number":96,"context_line":"                        \u0027represents the default role ID used to associate \u0027"},{"line_number":97,"context_line":"                        \u0027users with their default projects in the v2 API. \u0027"},{"line_number":98,"context_line":"                        \u0027During a database migration, this will be used to \u0027"},{"line_number":99,"context_line":"                        \u0027create a new role that will replace records in the \u0027"},{"line_number":100,"context_line":"                        \u0027assignment table with explicit role assignments. \u0027"},{"line_number":101,"context_line":"                        \u0027After that migration, this will be used as the \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"da9df570_c0a63fc1","line":98,"in_reply_to":"da9df570_bcddfa44","updated":"2014-09-22 22:03:52.000000000","message":"I think it was originally done in https://review.openstack.org/#/c/21327/ and has since been removed again.","commit_id":"ad5cdaee560d4559b3f7931925b850f5dc3d3413"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"b5760d91e33b3bee53a42ce77ecefc440b3c0b7e","unresolved":false,"context_lines":[{"line_number":95,"context_line":"                   help\u003d\u0027Similar to the member_role_name option, this \u0027"},{"line_number":96,"context_line":"                        \u0027represents the default role ID used to associate \u0027"},{"line_number":97,"context_line":"                        \u0027users with their default projects in the v2 API. \u0027"},{"line_number":98,"context_line":"                        \u0027During a database migration, this will be used to \u0027"},{"line_number":99,"context_line":"                        \u0027create a new role that will replace records in the \u0027"},{"line_number":100,"context_line":"                        \u0027assignment table with explicit role assignments. \u0027"},{"line_number":101,"context_line":"                        \u0027After that migration, this will be used as the \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"baa201ad_336312b3","line":98,"in_reply_to":"da9df570_c0a63fc1","updated":"2014-09-29 01:56:26.000000000","message":"so is the config option needed still?","commit_id":"ad5cdaee560d4559b3f7931925b850f5dc3d3413"}]}