)]}'
{"barbican/api/controllers/cas.py":[{"author":{"_account_id":7789,"name":"John Wood","email":"john.wood@rackspace.com","username":"woodster"},"change_message_id":"3b8c737fe5b019f7ffec623448b620d7e18bf1ae","unresolved":false,"context_lines":[{"line_number":247,"context_line":"        self.project_repo \u003d repo.get_project_repository()"},{"line_number":248,"context_line":"        self.validator \u003d validators.NewCAValidator()"},{"line_number":249,"context_line":"        self.quota_enforcer \u003d quota.QuotaEnforcer(\u0027cas\u0027, self.ca_repo)"},{"line_number":250,"context_line":"        # Populate the CA table at start up"},{"line_number":251,"context_line":"        cert_resources.refresh_certificate_resources()"},{"line_number":252,"context_line":""},{"line_number":253,"context_line":"    def __getattr__(self, name):"}],"source_content_type":"text/x-python","patch_set":6,"id":"ba15a1d1_6842baaf","line":250,"updated":"2015-09-18 17:08:44.000000000","message":"Does this have to be done here? I\u0027m concerned that this comes up before the database is initialized.","commit_id":"91a561dd9adc44b89ada27104e27649e2c726a5a"},{"author":{"_account_id":11561,"name":"Dave McCowan","email":"dmccowan@cisco.com","username":"dave-mccowan"},"change_message_id":"2dffed817509fdc04a67106f094da34c90931ece","unresolved":false,"context_lines":[{"line_number":247,"context_line":"        self.project_repo \u003d repo.get_project_repository()"},{"line_number":248,"context_line":"        self.validator \u003d validators.NewCAValidator()"},{"line_number":249,"context_line":"        self.quota_enforcer \u003d quota.QuotaEnforcer(\u0027cas\u0027, self.ca_repo)"},{"line_number":250,"context_line":"        # Populate the CA table at start up"},{"line_number":251,"context_line":"        cert_resources.refresh_certificate_resources()"},{"line_number":252,"context_line":""},{"line_number":253,"context_line":"    def __getattr__(self, name):"}],"source_content_type":"text/x-python","patch_set":6,"id":"ba15a1d1_949de30b","line":250,"in_reply_to":"ba15a1d1_6842baaf","updated":"2015-09-18 18:40:54.000000000","message":"In app.py, controllers are started after the database is initialized.  The calls above are connecting to the repositories.  I can\u0027t find any code that (automatically) runs after this.  Waiting for a user to make a call would leave the same theoretical timing problem, since a user could issue a GET /cas/ at the same time as this routine is run.","commit_id":"91a561dd9adc44b89ada27104e27649e2c726a5a"}],"barbican/plugin/interface/certificate_manager.py":[{"author":{"_account_id":7789,"name":"John Wood","email":"john.wood@rackspace.com","username":"woodster"},"change_message_id":"3b8c737fe5b019f7ffec623448b620d7e18bf1ae","unresolved":false,"context_lines":[{"line_number":622,"context_line":""},{"line_number":623,"context_line":"    def refresh_ca_table(self):"},{"line_number":624,"context_line":"        \"\"\"Refreshes the CertificateAuthority table.\"\"\""},{"line_number":625,"context_line":"        session \u003d self.ca_repo.get_session()"},{"line_number":626,"context_line":"        updates_made \u003d False"},{"line_number":627,"context_line":"        for plugin in plugin_utils.get_active_plugins(self):"},{"line_number":628,"context_line":"            plugin_name \u003d utils.generate_fullname_for(plugin)"}],"source_content_type":"text/x-python","patch_set":6,"id":"ba15a1d1_3b2e4a8c","line":625,"updated":"2015-09-18 17:08:44.000000000","message":"Yeah, you really shouldn\u0027t have to do these extra steps. I wonder if the call from the initializer is where you see issues? Also that session is established at the request start for a given request process/thread, so you shouldn\u0027t have to pass it into the repo calls below. If you aren\u0027t seeing this, then something else might be causing issues.","commit_id":"91a561dd9adc44b89ada27104e27649e2c726a5a"},{"author":{"_account_id":11561,"name":"Dave McCowan","email":"dmccowan@cisco.com","username":"dave-mccowan"},"change_message_id":"2dffed817509fdc04a67106f094da34c90931ece","unresolved":false,"context_lines":[{"line_number":622,"context_line":""},{"line_number":623,"context_line":"    def refresh_ca_table(self):"},{"line_number":624,"context_line":"        \"\"\"Refreshes the CertificateAuthority table.\"\"\""},{"line_number":625,"context_line":"        session \u003d self.ca_repo.get_session()"},{"line_number":626,"context_line":"        updates_made \u003d False"},{"line_number":627,"context_line":"        for plugin in plugin_utils.get_active_plugins(self):"},{"line_number":628,"context_line":"            plugin_name \u003d utils.generate_fullname_for(plugin)"}],"source_content_type":"text/x-python","patch_set":6,"id":"ba15a1d1_9458838b","line":625,"in_reply_to":"ba15a1d1_3b2e4a8c","updated":"2015-09-18 18:40:54.000000000","message":"I just took them out, and the code stopped working.  The updates to the database were not saved.","commit_id":"91a561dd9adc44b89ada27104e27649e2c726a5a"}]}