)]}'
{"keystone/common/config.py":[{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"ce7d2f6bc6a3a31b0a4949be7e8eefbafdc3603f","unresolved":false,"context_lines":[{"line_number":405,"context_line":"                        \u0027keystone.assignment namespace. Only an SQL driver is \u0027"},{"line_number":406,"context_line":"                        \u0027supplied.\u0027,"},{"line_number":407,"context_line":"                   default\u003d\u0027sql\u0027),"},{"line_number":408,"context_line":"        cfg.ListOpt(\u0027root_role\u0027, default\u003d[\u0027admin\u0027],"},{"line_number":409,"context_line":"                    help\u003d\u0027A list of role names which are prohibited from \u0027"},{"line_number":410,"context_line":"                         \u0027being an implied role.\u0027),"},{"line_number":411,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a57f1b5_df02c7b3","line":408,"range":{"start_line":408,"start_character":21,"end_line":408,"end_character":30},"updated":"2016-02-12 17:53:02.000000000","message":"root_roles","commit_id":"893349e9bfc64f35b0cf798ec94f4f802e3007c7"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"988e3ed3a45929295506d5223c6e896b9f606edf","unresolved":false,"context_lines":[{"line_number":405,"context_line":"                        \u0027keystone.assignment namespace. Only an SQL driver is \u0027"},{"line_number":406,"context_line":"                        \u0027supplied.\u0027,"},{"line_number":407,"context_line":"                   default\u003d\u0027sql\u0027),"},{"line_number":408,"context_line":"        cfg.ListOpt(\u0027root_role\u0027, default\u003d[\u0027admin\u0027],"},{"line_number":409,"context_line":"                    help\u003d\u0027A list of role names which are prohibited from \u0027"},{"line_number":410,"context_line":"                         \u0027being an implied role.\u0027),"},{"line_number":411,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a57f1b5_85b2c0e3","line":408,"range":{"start_line":408,"start_character":21,"end_line":408,"end_character":30},"in_reply_to":"3a57f1b5_c288ae73","updated":"2016-02-12 18:36:02.000000000","message":"i\u0027d go with \"prohibited_implied_role\"\n\nfor the other ListOpt options, we don\u0027t pluralize the word (see notification_driver)","commit_id":"893349e9bfc64f35b0cf798ec94f4f802e3007c7"},{"author":{"_account_id":8116,"name":"John Dennis","email":"jdennis@redhat.com","username":"jdennis"},"change_message_id":"46cbc22b393e9bcf320d5cdc4b5e56d5c2e7f3b8","unresolved":false,"context_lines":[{"line_number":405,"context_line":"                        \u0027keystone.assignment namespace. Only an SQL driver is \u0027"},{"line_number":406,"context_line":"                        \u0027supplied.\u0027,"},{"line_number":407,"context_line":"                   default\u003d\u0027sql\u0027),"},{"line_number":408,"context_line":"        cfg.ListOpt(\u0027root_role\u0027, default\u003d[\u0027admin\u0027],"},{"line_number":409,"context_line":"                    help\u003d\u0027A list of role names which are prohibited from \u0027"},{"line_number":410,"context_line":"                         \u0027being an implied role.\u0027),"},{"line_number":411,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a57f1b5_ff4303ad","line":408,"range":{"start_line":408,"start_character":21,"end_line":408,"end_character":30},"in_reply_to":"3a57f1b5_df02c7b3","updated":"2016-02-12 18:05:44.000000000","message":"My assumption was we could not change the name of the configuration variable because it may be in use. If it is OK to change the variable name than I would call it something else entirely because root_role and the whole discussion of directed acyclic graphs is confusing. Given my druthers I would probably have called the variable \"prohibited_implied_roles\".","commit_id":"893349e9bfc64f35b0cf798ec94f4f802e3007c7"},{"author":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"change_message_id":"e3d9e6f6ffaa4f5983cb65543cbade91f969e986","unresolved":false,"context_lines":[{"line_number":405,"context_line":"                        \u0027keystone.assignment namespace. Only an SQL driver is \u0027"},{"line_number":406,"context_line":"                        \u0027supplied.\u0027,"},{"line_number":407,"context_line":"                   default\u003d\u0027sql\u0027),"},{"line_number":408,"context_line":"        cfg.ListOpt(\u0027root_role\u0027, default\u003d[\u0027admin\u0027],"},{"line_number":409,"context_line":"                    help\u003d\u0027A list of role names which are prohibited from \u0027"},{"line_number":410,"context_line":"                         \u0027being an implied role.\u0027),"},{"line_number":411,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a57f1b5_c288ae73","line":408,"range":{"start_line":408,"start_character":21,"end_line":408,"end_character":30},"in_reply_to":"3a57f1b5_ff4303ad","updated":"2016-02-12 18:10:43.000000000","message":"prohibited_implied_roles  is fine.  We can change this now until milestone 3 which is coming up","commit_id":"893349e9bfc64f35b0cf798ec94f4f802e3007c7"}]}