)]}'
{"keystone/assignment/core.py":[{"author":{"_account_id":11045,"name":"Bob Thyne","email":"bob.thyne@hp.com","username":"bobt"},"change_message_id":"d26c5e27631f8beb79dda0534bbb16103d07972a","unresolved":false,"context_lines":[{"line_number":93,"context_line":"        if not domain.get(\u0027enabled\u0027, True):"},{"line_number":94,"context_line":"            raise AssertionError(_(\u0027Domain is disabled: %s\u0027) % domain_id)"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    def assert_project_enabled(self, project_id, project\u003dNone):"},{"line_number":97,"context_line":"        \"\"\"Assert the project is enabled and it\u0027s associated domain is enabled."},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"        :raise AssertionError if the project or domain is disabled."}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_d30f1c87","line":96,"updated":"2014-07-16 03:14:16.000000000","message":"Why do we also check whether the domain is enabled inside assert_project_enabled?","commit_id":"83bd04cd905ccf4e4cc9b59fbef607db7d085d74"},{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"432a39b28a1b3e60056adc6dc4737b02ec7b4682","unresolved":false,"context_lines":[{"line_number":93,"context_line":"        if not domain.get(\u0027enabled\u0027, True):"},{"line_number":94,"context_line":"            raise AssertionError(_(\u0027Domain is disabled: %s\u0027) % domain_id)"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    def assert_project_enabled(self, project_id, project\u003dNone):"},{"line_number":97,"context_line":"        \"\"\"Assert the project is enabled and it\u0027s associated domain is enabled."},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"        :raise AssertionError if the project or domain is disabled."}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_7961d1a7","line":96,"in_reply_to":"baada198_d30f1c87","updated":"2014-07-16 03:47:41.000000000","message":"If the domain is disabled, the project is disabled. However, disabling a domain does not set \u0027disabled\u0027 on all projects, it would be a lot of extra overhead we can avoid by just checking the domain as needed.","commit_id":"83bd04cd905ccf4e4cc9b59fbef607db7d085d74"},{"author":{"_account_id":11045,"name":"Bob Thyne","email":"bob.thyne@hp.com","username":"bobt"},"change_message_id":"d26c5e27631f8beb79dda0534bbb16103d07972a","unresolved":false,"context_lines":[{"line_number":94,"context_line":"            raise AssertionError(_(\u0027Domain is disabled: %s\u0027) % domain_id)"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    def assert_project_enabled(self, project_id, project\u003dNone):"},{"line_number":97,"context_line":"        \"\"\"Assert the project is enabled and it\u0027s associated domain is enabled."},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"        :raise AssertionError if the project or domain is disabled."},{"line_number":100,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_9315a49d","line":97,"updated":"2014-07-16 03:14:16.000000000","message":"s/it\u0027s/its","commit_id":"83bd04cd905ccf4e4cc9b59fbef607db7d085d74"},{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"432a39b28a1b3e60056adc6dc4737b02ec7b4682","unresolved":false,"context_lines":[{"line_number":94,"context_line":"            raise AssertionError(_(\u0027Domain is disabled: %s\u0027) % domain_id)"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    def assert_project_enabled(self, project_id, project\u003dNone):"},{"line_number":97,"context_line":"        \"\"\"Assert the project is enabled and it\u0027s associated domain is enabled."},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"        :raise AssertionError if the project or domain is disabled."},{"line_number":100,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_9956a5cd","line":97,"in_reply_to":"baada198_9315a49d","updated":"2014-07-16 03:47:41.000000000","message":"Will fix when the next patchset is uploaded if needed.","commit_id":"83bd04cd905ccf4e4cc9b59fbef607db7d085d74"}],"keystone/auth/plugins/password.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"b57629074d442f5c4f8d9f32f46993a85fbea3b5","unresolved":false,"context_lines":[{"line_number":46,"context_line":"                domain_id\u003ddomain_ref[\u0027id\u0027],"},{"line_number":47,"context_line":"                domain\u003ddomain_ref)"},{"line_number":48,"context_line":"        except AssertionError as e:"},{"line_number":49,"context_line":"            LOG.warning(e)"},{"line_number":50,"context_line":"            six.reraise(exception.Unauthorized, exception.Unauthorized(e),"},{"line_number":51,"context_line":"                        sys.exc_info()[2])"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_f5a2ab3d","line":49,"updated":"2014-07-20 03:51:34.000000000","message":"out of scope, but i\u0027m thinking all these should all be INFO","commit_id":"ac31133fba0c1a0d0442a0a3e61196a9491da603"}]}