)]}'
{"keystone/common/config.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"3d7d79d2b609251eb567b7ef669349ca1d7bbf1a","unresolved":false,"context_lines":[{"line_number":230,"context_line":"    \u0027os_inherit\u0027: ["},{"line_number":231,"context_line":"        cfg.BoolOpt(\u0027enabled\u0027, default\u003dTrue,"},{"line_number":232,"context_line":"                    deprecated_for_removal\u003dTrue,"},{"line_number":233,"context_line":"                    deprecated_reason\u003d\u0027In the future, this option will be \u0027"},{"line_number":234,"context_line":"                                      \u0027removed and this portion of the API \u0027"},{"line_number":235,"context_line":"                                      \u0027will be always enabled.\u0027,"},{"line_number":236,"context_line":"                    help\u003d\u0027role-assignment inheritance to projects from \u0027"},{"line_number":237,"context_line":"                         \u0027owning domain or from projects higher in the \u0027"},{"line_number":238,"context_line":"                         \u0027hierarchy can be optionally disabled. In the \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_0fbd5bd7","line":235,"range":{"start_line":233,"start_character":20,"end_line":235,"end_character":64},"updated":"2016-01-07 01:59:00.000000000","message":"this will be removed in the \u0027O\u0027 release","commit_id":"dbbd6738a3558239bbcaffcb59a7a63b06a72f5c"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"3d7d79d2b609251eb567b7ef669349ca1d7bbf1a","unresolved":false,"context_lines":[{"line_number":522,"context_line":"        cfg.BoolOpt(\u0027enabled\u0027,"},{"line_number":523,"context_line":"                    default\u003dTrue,"},{"line_number":524,"context_line":"                    deprecated_for_removal\u003dTrue,"},{"line_number":525,"context_line":"                    deprecated_reason\u003d\u0027The endpoint policy extension should \u0027"},{"line_number":526,"context_line":"                                      \u0027always be enabled.\u0027,"},{"line_number":527,"context_line":"                    help\u003d\u0027Enable endpoint_policy functionality.\u0027),"},{"line_number":528,"context_line":"        cfg.StrOpt(\u0027driver\u0027,"},{"line_number":529,"context_line":"                   default\u003d\u0027sql\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_efb567ec","line":526,"range":{"start_line":525,"start_character":20,"end_line":526,"end_character":59},"updated":"2016-01-07 01:59:00.000000000","message":"this will be removed in the \u0027O\u0027 release","commit_id":"dbbd6738a3558239bbcaffcb59a7a63b06a72f5c"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"3d7d79d2b609251eb567b7ef669349ca1d7bbf1a","unresolved":false,"context_lines":[{"line_number":650,"context_line":"                   deprecated_opts\u003d[cfg.DeprecatedOpt("},{"line_number":651,"context_line":"                       \u0027tenant_tree_dn\u0027, group\u003d\u0027ldap\u0027)],"},{"line_number":652,"context_line":"                   deprecated_for_removal\u003dTrue,"},{"line_number":653,"context_line":"                   deprecated_reason\u003d\u0027The LDAP assignment driver and its \u0027"},{"line_number":654,"context_line":"                                     \u0027configuration options are deprecated \u0027"},{"line_number":655,"context_line":"                                     \u0027for removal.\u0027,"},{"line_number":656,"context_line":"                   help\u003d\u0027Search base for projects. \u0027"},{"line_number":657,"context_line":"                        \u0027Defaults to the suffix value.\u0027),"},{"line_number":658,"context_line":"        cfg.StrOpt(\u0027project_filter\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_afcc4f89","line":655,"range":{"start_line":653,"start_character":19,"end_line":655,"end_character":52},"updated":"2016-01-07 01:59:00.000000000","message":"this will be removed in the \u0027M\u0027 release","commit_id":"dbbd6738a3558239bbcaffcb59a7a63b06a72f5c"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"7c66241f3c6c7dab9114207246718bc509a37ddc","unresolved":false,"context_lines":[{"line_number":650,"context_line":"                   deprecated_opts\u003d[cfg.DeprecatedOpt("},{"line_number":651,"context_line":"                       \u0027tenant_tree_dn\u0027, group\u003d\u0027ldap\u0027)],"},{"line_number":652,"context_line":"                   deprecated_for_removal\u003dTrue,"},{"line_number":653,"context_line":"                   deprecated_reason\u003d\u0027The LDAP assignment driver and its \u0027"},{"line_number":654,"context_line":"                                     \u0027configuration options are deprecated \u0027"},{"line_number":655,"context_line":"                                     \u0027for removal.\u0027,"},{"line_number":656,"context_line":"                   help\u003d\u0027Search base for projects. \u0027"},{"line_number":657,"context_line":"                        \u0027Defaults to the suffix value.\u0027),"},{"line_number":658,"context_line":"        cfg.StrOpt(\u0027project_filter\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_494828b3","line":655,"range":{"start_line":653,"start_character":19,"end_line":655,"end_character":52},"in_reply_to":"fa69d971_95a63cac","updated":"2016-01-07 07:30:15.000000000","message":"Nevermind, I see https://review.openstack.org/#/c/231872/ covers that.  So no need to add the deprecated_reason for anything getting removed in M.","commit_id":"dbbd6738a3558239bbcaffcb59a7a63b06a72f5c"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"f016fd29104f1ac0f6aabd8336734621885483f2","unresolved":false,"context_lines":[{"line_number":650,"context_line":"                   deprecated_opts\u003d[cfg.DeprecatedOpt("},{"line_number":651,"context_line":"                       \u0027tenant_tree_dn\u0027, group\u003d\u0027ldap\u0027)],"},{"line_number":652,"context_line":"                   deprecated_for_removal\u003dTrue,"},{"line_number":653,"context_line":"                   deprecated_reason\u003d\u0027The LDAP assignment driver and its \u0027"},{"line_number":654,"context_line":"                                     \u0027configuration options are deprecated \u0027"},{"line_number":655,"context_line":"                                     \u0027for removal.\u0027,"},{"line_number":656,"context_line":"                   help\u003d\u0027Search base for projects. \u0027"},{"line_number":657,"context_line":"                        \u0027Defaults to the suffix value.\u0027),"},{"line_number":658,"context_line":"        cfg.StrOpt(\u0027project_filter\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_95a63cac","line":655,"range":{"start_line":653,"start_character":19,"end_line":655,"end_character":52},"in_reply_to":"fa69d971_afcc4f89","updated":"2016-01-07 03:03:17.000000000","message":"I guess these I can just create another patch to remove.","commit_id":"dbbd6738a3558239bbcaffcb59a7a63b06a72f5c"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"7b957e46a7f5b17208fa8dd57982ddc30871fdbf","unresolved":false,"context_lines":[{"line_number":795,"context_line":""},{"line_number":796,"context_line":"        cfg.StrOpt(\u0027role_tree_dn\u0027,"},{"line_number":797,"context_line":"                   deprecated_for_removal\u003dTrue,"},{"line_number":798,"context_line":"                   deprecated_reason\u003d\u0027The LDAP assignment driver and its \u0027"},{"line_number":799,"context_line":"                                     \u0027configuration options are deprecated \u0027"},{"line_number":800,"context_line":"                                     \u0027for removal.\u0027,"},{"line_number":801,"context_line":"                   help\u003d\u0027Search base for roles. \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_af1e8f09","line":798,"range":{"start_line":798,"start_character":47,"end_line":798,"end_character":64},"updated":"2016-01-07 01:57:14.000000000","message":"this should be \u0027role driver\u0027, same with all the other options in this group below","commit_id":"dbbd6738a3558239bbcaffcb59a7a63b06a72f5c"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"f016fd29104f1ac0f6aabd8336734621885483f2","unresolved":false,"context_lines":[{"line_number":795,"context_line":""},{"line_number":796,"context_line":"        cfg.StrOpt(\u0027role_tree_dn\u0027,"},{"line_number":797,"context_line":"                   deprecated_for_removal\u003dTrue,"},{"line_number":798,"context_line":"                   deprecated_reason\u003d\u0027The LDAP assignment driver and its \u0027"},{"line_number":799,"context_line":"                                     \u0027configuration options are deprecated \u0027"},{"line_number":800,"context_line":"                                     \u0027for removal.\u0027,"},{"line_number":801,"context_line":"                   help\u003d\u0027Search base for roles. \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_15b9eccb","line":798,"range":{"start_line":798,"start_character":47,"end_line":798,"end_character":64},"in_reply_to":"fa69d971_af1e8f09","updated":"2016-01-07 03:03:17.000000000","message":"Oops, got copy and paste happy.  Thanks!","commit_id":"dbbd6738a3558239bbcaffcb59a7a63b06a72f5c"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"3d7d79d2b609251eb567b7ef669349ca1d7bbf1a","unresolved":false,"context_lines":[{"line_number":795,"context_line":""},{"line_number":796,"context_line":"        cfg.StrOpt(\u0027role_tree_dn\u0027,"},{"line_number":797,"context_line":"                   deprecated_for_removal\u003dTrue,"},{"line_number":798,"context_line":"                   deprecated_reason\u003d\u0027The LDAP assignment driver and its \u0027"},{"line_number":799,"context_line":"                                     \u0027configuration options are deprecated \u0027"},{"line_number":800,"context_line":"                                     \u0027for removal.\u0027,"},{"line_number":801,"context_line":"                   help\u003d\u0027Search base for roles. \u0027"},{"line_number":802,"context_line":"                        \u0027Defaults to the suffix value.\u0027),"},{"line_number":803,"context_line":"        cfg.StrOpt(\u0027role_filter\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_8f986b79","line":800,"range":{"start_line":798,"start_character":19,"end_line":800,"end_character":52},"updated":"2016-01-07 01:59:00.000000000","message":"these will be removed in the \u0027O\u0027 release, we forgot to deprecate this driver earlier when we split assignment into assignment and role","commit_id":"dbbd6738a3558239bbcaffcb59a7a63b06a72f5c"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"8a4efcf7b9a39ab248f4c2c8abecf34590fec352","unresolved":false,"context_lines":[{"line_number":232,"context_line":"                    deprecated_for_removal\u003dTrue,"},{"line_number":233,"context_line":"                    deprecated_reason\u003d\u0027In the O release, this option will be \u0027"},{"line_number":234,"context_line":"                                      \u0027removed and this portion of the API \u0027"},{"line_number":235,"context_line":"                                      \u0027will be always enabled.\u0027,"},{"line_number":236,"context_line":"                    help\u003d\u0027role-assignment inheritance to projects from \u0027"},{"line_number":237,"context_line":"                         \u0027owning domain or from projects higher in the \u0027"},{"line_number":238,"context_line":"                         \u0027hierarchy can be optionally disabled. In the \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_20edb325","line":235,"range":{"start_line":235,"start_character":54,"end_line":235,"end_character":61},"updated":"2016-01-07 14:59:52.000000000","message":"\"enabled by default.\" \n\nThis mostly says \"what\" is happening versus giving the reason. How much reason do we need to give?","commit_id":"2e171ef795921c6de418a973147ae60e0b238beb"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"8a4efcf7b9a39ab248f4c2c8abecf34590fec352","unresolved":false,"context_lines":[{"line_number":522,"context_line":"        cfg.BoolOpt(\u0027enabled\u0027,"},{"line_number":523,"context_line":"                    default\u003dTrue,"},{"line_number":524,"context_line":"                    deprecated_for_removal\u003dTrue,"},{"line_number":525,"context_line":"                    deprecated_reason\u003d\u0027The endpoint policy extension should \u0027"},{"line_number":526,"context_line":"                                      \u0027always be enabled. This option will be \u0027"},{"line_number":527,"context_line":"                                      \u0027removed in the O release.\u0027,"},{"line_number":528,"context_line":"                    help\u003d\u0027Enable endpoint_policy functionality.\u0027),"},{"line_number":529,"context_line":"        cfg.StrOpt(\u0027driver\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_80c33f8a","line":526,"range":{"start_line":525,"start_character":39,"end_line":526,"end_character":56},"updated":"2016-01-07 14:59:52.000000000","message":"As a deployer, why should I have the endpoint policy extension enabled?","commit_id":"2e171ef795921c6de418a973147ae60e0b238beb"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"8a4efcf7b9a39ab248f4c2c8abecf34590fec352","unresolved":false,"context_lines":[{"line_number":748,"context_line":""},{"line_number":749,"context_line":"        cfg.StrOpt(\u0027role_tree_dn\u0027,"},{"line_number":750,"context_line":"                   deprecated_for_removal\u003dTrue,"},{"line_number":751,"context_line":"                   deprecated_reason\u003d\u0027The LDAP role driver and its \u0027"},{"line_number":752,"context_line":"                                     \u0027configuration options are deprecated \u0027"},{"line_number":753,"context_line":"                                     \u0027for removal. This option will be \u0027"},{"line_number":754,"context_line":"                                     \u0027removed in the O release.\u0027,"},{"line_number":755,"context_line":"                   help\u003d\u0027Search base for roles. \u0027"},{"line_number":756,"context_line":"                        \u0027Defaults to the suffix value.\u0027),"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_406b0786","line":753,"range":{"start_line":751,"start_character":38,"end_line":753,"end_character":49},"updated":"2016-01-07 14:59:52.000000000","message":"As an operator, this doesn\u0027t give me much of a reason *why* the LDAP role driver is being deprecated. I think this comment applies to most of these additions.","commit_id":"2e171ef795921c6de418a973147ae60e0b238beb"}]}