[ARVADOS] created: e8d11e963248fada51bdb1ecc59446adccff440f

Git user git at public.curoverse.com
Fri Jun 30 13:51:30 EDT 2017


        at  e8d11e963248fada51bdb1ecc59446adccff440f (commit)


commit e8d11e963248fada51bdb1ecc59446adccff440f
Author: Tom Clegg <tom at curoverse.com>
Date:   Fri Jun 30 10:17:02 2017 -0400

    11917: Do not clear rails cache at boot time.
    
    This seems like a good idea when there is only one process that has
    access to the cache. However, in real life the crunch1 dispatcher,
    rake tasks, and the apiserver all share the cache, and they don't all
    start up at the same time. This means clear() can run while other
    operations like fetch() are in progress, which is apparently not a
    supported use. With the Rails file cache, if clear() removes a
    directory at an inopportune moment, fetch() crashes (see
    https://dev.arvados.org/issues/11917).
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curoverse.com>

diff --git a/services/api/config/environment.rb b/services/api/config/environment.rb
index d4b606d..e24eee0 100644
--- a/services/api/config/environment.rb
+++ b/services/api/config/environment.rb
@@ -8,9 +8,3 @@ require 'josh_id'
 
 # Initialize the rails application
 Server::Application.initialize!
-begin
-  Rails.cache.clear
-rescue Errno::ENOENT => e
-  # Cache directory does not exist? Then cache is clear, proceed.
-  Rails.logger.warn "In Rails.cache.clear, ignoring #{e.inspect}"
-end

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list