)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"51f014d2c8c342dbec38ba1239597aeacad974b6","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Henry Nash \u003chenryn@linux.vnet.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-02-21 11:32:38 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Refactor domain specific loading"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In preparation for the support of loading domain specific"},{"line_number":10,"context_line":"confoguration options from a database backend, the existing"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba7be1f8_07369752","line":7,"updated":"2015-02-22 07:33:43.000000000","message":"you\u0027re doing a bit more than just refactoring here","commit_id":"226e5ad8723ddf8761e606f49d23be669ee9ae39"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"effc4888674ea0cef94a66f73fb0c45efc5012e5","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Henry Nash \u003chenryn@linux.vnet.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-02-21 11:32:38 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Refactor domain specific loading"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In preparation for the support of loading domain specific"},{"line_number":10,"context_line":"confoguration options from a database backend, the existing"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba7be1f8_c260ed37","line":7,"in_reply_to":"ba7be1f8_07369752","updated":"2015-02-22 10:12:26.000000000","message":"It\u0027s a fair cop, guv....I\u0027ll change m\u0027ways","commit_id":"226e5ad8723ddf8761e606f49d23be669ee9ae39"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"51f014d2c8c342dbec38ba1239597aeacad974b6","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Refactor domain specific loading"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In preparation for the support of loading domain specific"},{"line_number":10,"context_line":"confoguration options from a database backend, the existing"},{"line_number":11,"context_line":"code structure for loadin from files is refactored to make"},{"line_number":12,"context_line":"this easier."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba7be1f8_c72b9fe9","line":10,"updated":"2015-02-22 07:33:43.000000000","message":"configuration*","commit_id":"226e5ad8723ddf8761e606f49d23be669ee9ae39"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"effc4888674ea0cef94a66f73fb0c45efc5012e5","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Refactor domain specific loading"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In preparation for the support of loading domain specific"},{"line_number":10,"context_line":"confoguration options from a database backend, the existing"},{"line_number":11,"context_line":"code structure for loadin from files is refactored to make"},{"line_number":12,"context_line":"this easier."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba7be1f8_e265f145","line":10,"in_reply_to":"ba7be1f8_c72b9fe9","updated":"2015-02-22 10:12:26.000000000","message":"Done","commit_id":"226e5ad8723ddf8761e606f49d23be669ee9ae39"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"51f014d2c8c342dbec38ba1239597aeacad974b6","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In preparation for the support of loading domain specific"},{"line_number":10,"context_line":"confoguration options from a database backend, the existing"},{"line_number":11,"context_line":"code structure for loadin from files is refactored to make"},{"line_number":12,"context_line":"this easier."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Partially Implements: blueprint domain-config-ext"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba7be1f8_e730a358","line":11,"updated":"2015-02-22 07:33:43.000000000","message":"loading*","commit_id":"226e5ad8723ddf8761e606f49d23be669ee9ae39"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"effc4888674ea0cef94a66f73fb0c45efc5012e5","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In preparation for the support of loading domain specific"},{"line_number":10,"context_line":"confoguration options from a database backend, the existing"},{"line_number":11,"context_line":"code structure for loadin from files is refactored to make"},{"line_number":12,"context_line":"this easier."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Partially Implements: blueprint domain-config-ext"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba7be1f8_025be587","line":11,"in_reply_to":"ba7be1f8_e730a358","updated":"2015-02-22 10:12:26.000000000","message":"Done","commit_id":"226e5ad8723ddf8761e606f49d23be669ee9ae39"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"42f9f05bc29ba42779550d8a0c31c3af544ddfc4","unresolved":false,"context_lines":[{"line_number":11,"context_line":"code structure for loading from files is refactored to make"},{"line_number":12,"context_line":"this easier. In addition the config option to support"},{"line_number":13,"context_line":"domain specific configurations loading from a database is"},{"line_number":14,"context_line":"also added (whiih will result in a NotImplemented error if"},{"line_number":15,"context_line":"it is enabled)."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Partially Implements: blueprint domain-config-ext"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ba7be1f8_a68cdf36","line":14,"updated":"2015-02-23 12:28:16.000000000","message":"nit: which","commit_id":"33866003f05a0592782fbec54b606c0b01e3a454"},{"author":{"_account_id":9142,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@lsd.ufcg.edu.br","username":"samuel-z","inactive":true},"change_message_id":"920d6afa5bd278cc04e7be64a59b5a7293356aea","unresolved":false,"context_lines":[{"line_number":11,"context_line":"code structure for loading from files is refactored to make"},{"line_number":12,"context_line":"this easier. In addition the config option to support"},{"line_number":13,"context_line":"domain specific configurations loading from a database is"},{"line_number":14,"context_line":"also added (whiih will result in a NotImplemented error if"},{"line_number":15,"context_line":"it is enabled)."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Partially Implements: blueprint domain-config-ext"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"ba7be1f8_fbcd62b8","line":14,"updated":"2015-03-02 17:49:28.000000000","message":"(nit) s/whiih/which ?","commit_id":"2c4edf068cdc4fe7ba9ba305d439d486a3e767fe"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"d6928df68b0f6abfb8927fac2fa63dd66ccb4344","unresolved":false,"context_lines":[{"line_number":11,"context_line":"code structure for loading from files is refactored to make"},{"line_number":12,"context_line":"this easier. In addition the config option to support"},{"line_number":13,"context_line":"domain specific configurations loading from a database is"},{"line_number":14,"context_line":"also added (whiih will result in a NotImplemented error if"},{"line_number":15,"context_line":"it is enabled)."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Partially Implements: blueprint domain-config-ext"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"ba7be1f8_9ba1c6ab","line":14,"updated":"2015-03-02 17:49:26.000000000","message":"s/whiih/which/","commit_id":"2c4edf068cdc4fe7ba9ba305d439d486a3e767fe"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"64ba89cb86612d63b55bac75f316479a835fc92b","unresolved":false,"context_lines":[{"line_number":11,"context_line":"code structure for loading from files is refactored to make"},{"line_number":12,"context_line":"this easier. In addition the config option to support"},{"line_number":13,"context_line":"domain specific configurations loading from a database is"},{"line_number":14,"context_line":"also added (whiih will result in a NotImplemented error if"},{"line_number":15,"context_line":"it is enabled)."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Partially Implements: blueprint domain-config-ext"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"ba7be1f8_ef859c07","line":14,"in_reply_to":"ba7be1f8_fbcd62b8","updated":"2015-03-02 18:49:04.000000000","message":"Done","commit_id":"2c4edf068cdc4fe7ba9ba305d439d486a3e767fe"}],"keystone/common/config.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"51f014d2c8c342dbec38ba1239597aeacad974b6","unresolved":false,"context_lines":[{"line_number":133,"context_line":"                         \u0027values specific to the domain need to be specified \u0027"},{"line_number":134,"context_line":"                         \u0027in this manner. This feature is disabled by \u0027"},{"line_number":135,"context_line":"                         \u0027default; set to true to enable.\u0027),"},{"line_number":136,"context_line":"        cfg.BoolOpt(\u0027domain_configurations_from_database\u0027,"},{"line_number":137,"context_line":"                    default\u003dFalse,"},{"line_number":138,"context_line":"                    help\u003d\u0027Extract the domain specific configuration options \u0027"},{"line_number":139,"context_line":"                         \u0027from the resource database where they have been \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_a7ef8baf","line":136,"updated":"2015-02-22 07:33:43.000000000","message":"adding a new option isn\u0027t really a refactor","commit_id":"226e5ad8723ddf8761e606f49d23be669ee9ae39"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"effc4888674ea0cef94a66f73fb0c45efc5012e5","unresolved":false,"context_lines":[{"line_number":133,"context_line":"                         \u0027values specific to the domain need to be specified \u0027"},{"line_number":134,"context_line":"                         \u0027in this manner. This feature is disabled by \u0027"},{"line_number":135,"context_line":"                         \u0027default; set to true to enable.\u0027),"},{"line_number":136,"context_line":"        cfg.BoolOpt(\u0027domain_configurations_from_database\u0027,"},{"line_number":137,"context_line":"                    default\u003dFalse,"},{"line_number":138,"context_line":"                    help\u003d\u0027Extract the domain specific configuration options \u0027"},{"line_number":139,"context_line":"                         \u0027from the resource database where they have been \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_824ef5bf","line":136,"in_reply_to":"ba7be1f8_a7ef8baf","updated":"2015-02-22 10:12:26.000000000","message":"fair point - added explanation to commit comment to explain","commit_id":"226e5ad8723ddf8761e606f49d23be669ee9ae39"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"42f9f05bc29ba42779550d8a0c31c3af544ddfc4","unresolved":false,"context_lines":[{"line_number":133,"context_line":"                         \u0027values specific to the domain need to be specified \u0027"},{"line_number":134,"context_line":"                         \u0027in this manner. This feature is disabled by \u0027"},{"line_number":135,"context_line":"                         \u0027default; set to true to enable.\u0027),"},{"line_number":136,"context_line":"        cfg.BoolOpt(\u0027domain_configurations_from_database\u0027,"},{"line_number":137,"context_line":"                    default\u003dFalse,"},{"line_number":138,"context_line":"                    help\u003d\u0027Extract the domain specific configuration options \u0027"},{"line_number":139,"context_line":"                         \u0027from the resource database where they have been \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba7be1f8_452f96f6","line":136,"updated":"2015-02-23 12:28:16.000000000","message":"maybe you should say that if you are using LDAP for assignment, this will need a SQL backend.","commit_id":"33866003f05a0592782fbec54b606c0b01e3a454"},{"author":{"_account_id":9142,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@lsd.ufcg.edu.br","username":"samuel-z","inactive":true},"change_message_id":"920d6afa5bd278cc04e7be64a59b5a7293356aea","unresolved":false,"context_lines":[{"line_number":137,"context_line":"        cfg.BoolOpt(\u0027domain_configurations_from_database\u0027,"},{"line_number":138,"context_line":"                    default\u003dFalse,"},{"line_number":139,"context_line":"                    help\u003d\u0027Extract the domain specific configuration options \u0027"},{"line_number":140,"context_line":"                         \u0027from the resource database where they have been \u0027"},{"line_number":141,"context_line":"                         \u0027stored with the domain data. This feature is \u0027"},{"line_number":142,"context_line":"                         \u0027disabled by default (in which case the domain \u0027"},{"line_number":143,"context_line":"                         \u0027specific options will be loaded from files in the \u0027"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba7be1f8_5b9f4e5f","line":140,"updated":"2015-03-02 17:49:28.000000000","message":"resource backend ?","commit_id":"2c4edf068cdc4fe7ba9ba305d439d486a3e767fe"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"64ba89cb86612d63b55bac75f316479a835fc92b","unresolved":false,"context_lines":[{"line_number":137,"context_line":"        cfg.BoolOpt(\u0027domain_configurations_from_database\u0027,"},{"line_number":138,"context_line":"                    default\u003dFalse,"},{"line_number":139,"context_line":"                    help\u003d\u0027Extract the domain specific configuration options \u0027"},{"line_number":140,"context_line":"                         \u0027from the resource database where they have been \u0027"},{"line_number":141,"context_line":"                         \u0027stored with the domain data. This feature is \u0027"},{"line_number":142,"context_line":"                         \u0027disabled by default (in which case the domain \u0027"},{"line_number":143,"context_line":"                         \u0027specific options will be loaded from files in the \u0027"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba7be1f8_cfca98a0","line":140,"in_reply_to":"ba7be1f8_5b9f4e5f","updated":"2015-03-02 18:49:04.000000000","message":"Done","commit_id":"2c4edf068cdc4fe7ba9ba305d439d486a3e767fe"}],"keystone/identity/core.py":[{"author":{"_account_id":9142,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@lsd.ufcg.edu.br","username":"samuel-z","inactive":true},"change_message_id":"920d6afa5bd278cc04e7be64a59b5a7293356aea","unresolved":false,"context_lines":[{"line_number":78,"context_line":"    setting of the \u0027domain_configurations_from_database\u0027 config option."},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    The result will be that for each domain with a specific configuration,"},{"line_number":81,"context_line":"    this class this hold a reference to a ConfigOpts and driver object that"},{"line_number":82,"context_line":"    the identity manager and driver can use."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":5,"id":"ba7be1f8_9b04667f","line":81,"updated":"2015-03-02 17:49:28.000000000","message":"(nit) s/this/will ? (the second \u0027this\u0027)","commit_id":"2c4edf068cdc4fe7ba9ba305d439d486a3e767fe"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"64ba89cb86612d63b55bac75f316479a835fc92b","unresolved":false,"context_lines":[{"line_number":78,"context_line":"    setting of the \u0027domain_configurations_from_database\u0027 config option."},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    The result will be that for each domain with a specific configuration,"},{"line_number":81,"context_line":"    this class this hold a reference to a ConfigOpts and driver object that"},{"line_number":82,"context_line":"    the identity manager and driver can use."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":5,"id":"ba7be1f8_cf41b80c","line":81,"in_reply_to":"ba7be1f8_9b04667f","updated":"2015-03-02 18:49:04.000000000","message":"Done","commit_id":"2c4edf068cdc4fe7ba9ba305d439d486a3e767fe"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"d6928df68b0f6abfb8927fac2fa63dd66ccb4344","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    The result will be that for each domain with a specific configuration,"},{"line_number":81,"context_line":"    this class this hold a reference to a ConfigOpts and driver object that"},{"line_number":82,"context_line":"    the identity manager and driver can use."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    \"\"\""},{"line_number":85,"context_line":"    # TODO(henry-nash): Support for reading the configuration options from"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba7be1f8_bb668a27","line":82,"updated":"2015-03-02 17:49:26.000000000","message":"Do we need to explain how the sensitive data such as passwords are protected?","commit_id":"2c4edf068cdc4fe7ba9ba305d439d486a3e767fe"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"64ba89cb86612d63b55bac75f316479a835fc92b","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    The result will be that for each domain with a specific configuration,"},{"line_number":81,"context_line":"    this class this hold a reference to a ConfigOpts and driver object that"},{"line_number":82,"context_line":"    the identity manager and driver can use."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    \"\"\""},{"line_number":85,"context_line":"    # TODO(henry-nash): Support for reading the configuration options from"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba7be1f8_6f50ccd4","line":82,"in_reply_to":"ba7be1f8_bb668a27","updated":"2015-03-02 18:49:04.000000000","message":"That comes is later patches, so I would say we don\u0027t need to describe it here....look at the next two patches up the chain...","commit_id":"2c4edf068cdc4fe7ba9ba305d439d486a3e767fe"},{"author":{"_account_id":9142,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@lsd.ufcg.edu.br","username":"samuel-z","inactive":true},"change_message_id":"920d6afa5bd278cc04e7be64a59b5a7293356aea","unresolved":false,"context_lines":[{"line_number":82,"context_line":"    the identity manager and driver can use."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    \"\"\""},{"line_number":85,"context_line":"    # TODO(henry-nash): Support for reading the configuration options from"},{"line_number":86,"context_line":"    # the resource database is not yet supported, and will raise a"},{"line_number":87,"context_line":"    # NotImplemented error."},{"line_number":88,"context_line":"    configured \u003d False"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba7be1f8_bbbf6a5f","line":85,"updated":"2015-03-02 17:49:28.000000000","message":"(nit) \u0027Support for reading the configuration [...] is not yet supported\u0027 sounds weird.\n\nMaybe we could remove \u0027Support for\u0027 and just keep \u0027Reading the configuration [...] is not yet supported, ...","commit_id":"2c4edf068cdc4fe7ba9ba305d439d486a3e767fe"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"64ba89cb86612d63b55bac75f316479a835fc92b","unresolved":false,"context_lines":[{"line_number":82,"context_line":"    the identity manager and driver can use."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"    \"\"\""},{"line_number":85,"context_line":"    # TODO(henry-nash): Support for reading the configuration options from"},{"line_number":86,"context_line":"    # the resource database is not yet supported, and will raise a"},{"line_number":87,"context_line":"    # NotImplemented error."},{"line_number":88,"context_line":"    configured \u003d False"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba7be1f8_8f5ee0e7","line":85,"in_reply_to":"ba7be1f8_bbbf6a5f","updated":"2015-03-02 18:49:04.000000000","message":"Done","commit_id":"2c4edf068cdc4fe7ba9ba305d439d486a3e767fe"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"d6928df68b0f6abfb8927fac2fa63dd66ccb4344","unresolved":false,"context_lines":[{"line_number":180,"context_line":""},{"line_number":181,"context_line":"        if CONF.identity.domain_configurations_from_database:"},{"line_number":182,"context_line":"            self._setup_domain_drivers_from_database(standard_driver,"},{"line_number":183,"context_line":"                                                     resource_api)"},{"line_number":184,"context_line":"        else:"},{"line_number":185,"context_line":"            self._setup_domain_drivers_from_files(standard_driver,"},{"line_number":186,"context_line":"                                                  resource_api)"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba7be1f8_3eaa88d2","line":183,"updated":"2015-03-02 17:49:26.000000000","message":"This means we either going to load domain config from database or file, but not both. How do we address backward compatibility? Explicit one-time migration via keystone-manage?","commit_id":"2c4edf068cdc4fe7ba9ba305d439d486a3e767fe"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"64ba89cb86612d63b55bac75f316479a835fc92b","unresolved":false,"context_lines":[{"line_number":180,"context_line":""},{"line_number":181,"context_line":"        if CONF.identity.domain_configurations_from_database:"},{"line_number":182,"context_line":"            self._setup_domain_drivers_from_database(standard_driver,"},{"line_number":183,"context_line":"                                                     resource_api)"},{"line_number":184,"context_line":"        else:"},{"line_number":185,"context_line":"            self._setup_domain_drivers_from_files(standard_driver,"},{"line_number":186,"context_line":"                                                  resource_api)"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba7be1f8_cfdd3844","line":183,"in_reply_to":"ba7be1f8_3eaa88d2","updated":"2015-03-02 18:49:04.000000000","message":"Correct....yes, further up the chain of patches you will see a patch for keystone-manage that provides exactly that capability","commit_id":"2c4edf068cdc4fe7ba9ba305d439d486a3e767fe"}],"keystone/tests/unit/test_backend_ldap.py":[{"author":{"_account_id":9142,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@lsd.ufcg.edu.br","username":"samuel-z","inactive":true},"change_message_id":"920d6afa5bd278cc04e7be64a59b5a7293356aea","unresolved":false,"context_lines":[{"line_number":2998,"context_line":"        # which should fail."},{"line_number":2999,"context_line":"        self.assertRaises("},{"line_number":3000,"context_line":"            exception.MultipleSQLDriversInConfig,"},{"line_number":3001,"context_line":"            self.identity_api.domain_configs._load_config_from_file,"},{"line_number":3002,"context_line":"            self.resource_api,"},{"line_number":3003,"context_line":"            [tests.TESTCONF + \u0027/domain_configs_one_extra_sql/\u0027 +"},{"line_number":3004,"context_line":"             \u0027keystone.domain2.conf\u0027],"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba7be1f8_3e0e0833","line":3001,"updated":"2015-03-02 17:49:28.000000000","message":"It think it would be better to call the non-private  domain_config.setup_domain_drivers, with domain_configurations_from_database as False.\n\nInstead of calling internal private methods. However, this is not being introduced by this patch.","commit_id":"2c4edf068cdc4fe7ba9ba305d439d486a3e767fe"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"64ba89cb86612d63b55bac75f316479a835fc92b","unresolved":false,"context_lines":[{"line_number":2998,"context_line":"        # which should fail."},{"line_number":2999,"context_line":"        self.assertRaises("},{"line_number":3000,"context_line":"            exception.MultipleSQLDriversInConfig,"},{"line_number":3001,"context_line":"            self.identity_api.domain_configs._load_config_from_file,"},{"line_number":3002,"context_line":"            self.resource_api,"},{"line_number":3003,"context_line":"            [tests.TESTCONF + \u0027/domain_configs_one_extra_sql/\u0027 +"},{"line_number":3004,"context_line":"             \u0027keystone.domain2.conf\u0027],"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba7be1f8_8fa180ab","line":3001,"in_reply_to":"ba7be1f8_3e0e0833","updated":"2015-03-02 18:49:04.000000000","message":"Agreed...In fact, once all this is in, we\u0027l likely to another refactor to try and stop the delayed loading of configs...and I\u0027ll include this suggestion in that clean up","commit_id":"2c4edf068cdc4fe7ba9ba305d439d486a3e767fe"}]}