[ARVADOS] updated: 1.3.0-2937-g3bcdb6249

Git user git at public.arvados.org
Wed Aug 19 01:56:01 UTC 2020


Summary of changes:
 build/rails-package-scripts/README.md             | 2 +-
 build/rails-package-scripts/arvados-api-server.sh | 3 ---
 build/rails-package-scripts/prerm.sh              | 1 -
 services/api/.gitignore                           | 3 +--
 4 files changed, 2 insertions(+), 7 deletions(-)

       via  3bcdb62498359a51bb45aa0e163faebf2803b802 (commit)
      from  eee302a3157296bccf40e9b83f52c2952294a04e (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 3bcdb62498359a51bb45aa0e163faebf2803b802
Author: Ward Vandewege <ward at curii.com>
Date:   Tue Aug 18 21:54:20 2020 -0400

    Stop installing initializers/omniauth.rb in our rails package for the
    API server. It has been a dangling symlink for a long time, and Rails
    5.2 is no longer forgiving of that.
    
    refs #16470
    
    Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <ward at curii.com>

diff --git a/build/rails-package-scripts/README.md b/build/rails-package-scripts/README.md
index 0d720bde1..2930957b9 100644
--- a/build/rails-package-scripts/README.md
+++ b/build/rails-package-scripts/README.md
@@ -12,7 +12,7 @@ Since our build process is a tower of shell scripts, concatenating files seemed
 
 postinst.sh lets the early parts define a few hooks to control behavior:
 
-* After it installs the core configuration files (database.yml, application.yml, and production.rb) to /etc/arvados/server, it calls setup_extra_conffiles.  By default this is a noop function (in step2.sh).  API server defines this to set up the old omniauth.rb conffile.
+* After it installs the core configuration files (database.yml, application.yml, and production.rb) to /etc/arvados/server, it calls setup_extra_conffiles.  By default this is a noop function (in step2.sh).
 * Before it restarts nginx, it calls setup_before_nginx_restart.  By default this is a noop function (in step2.sh).  API server defines this to set up the internal git repository, if necessary.
 * $RAILSPKG_DATABASE_LOAD_TASK defines the Rake task to load the database.  API server uses db:structure:load.  SSO server uses db:schema:load.  Workbench doesn't set this, which causes the postinst to skip all database work.
 * If $RAILSPKG_SUPPORTS_CONFIG_CHECK != 1, it won't run the config:check rake task.  SSO clears this flag (it doesn't have that task code).
diff --git a/build/rails-package-scripts/arvados-api-server.sh b/build/rails-package-scripts/arvados-api-server.sh
index 82bc9898a..2504ff85d 100644
--- a/build/rails-package-scripts/arvados-api-server.sh
+++ b/build/rails-package-scripts/arvados-api-server.sh
@@ -11,9 +11,6 @@ CONFIG_PATH=/etc/arvados/api
 DOC_URL="http://doc.arvados.org/install/install-api-server.html#configure"
 
 RAILSPKG_DATABASE_LOAD_TASK=db:structure:load
-setup_extra_conffiles() {
-    setup_conffile initializers/omniauth.rb
-}
 
 setup_before_nginx_restart() {
   # initialize git_internal_dir
diff --git a/build/rails-package-scripts/prerm.sh b/build/rails-package-scripts/prerm.sh
index 9816b14c6..6773b6f30 100644
--- a/build/rails-package-scripts/prerm.sh
+++ b/build/rails-package-scripts/prerm.sh
@@ -10,7 +10,6 @@ remove () {
   rm -f $RELEASE_PATH/config/environments/production.rb
   rm -f $RELEASE_PATH/config/application.yml
   # Old API server configuration file.
-  rm -f $RELEASE_PATH/config/initializers/omniauth.rb
   rm -rf $RELEASE_PATH/public/assets/
   rm -rf $RELEASE_PATH/tmp
   rm -rf $RELEASE_PATH/.bundle
diff --git a/services/api/.gitignore b/services/api/.gitignore
index 793e981b5..096277965 100644
--- a/services/api/.gitignore
+++ b/services/api/.gitignore
@@ -7,7 +7,6 @@
 
 # Sensitive files and local configuration
 /config/database.yml
-/config/initializers/omniauth.rb
 /config/application.yml
 
 # asset cache
@@ -35,4 +34,4 @@
 /package-build.version
 
 # Debugger history
-.byebug_history
\ No newline at end of file
+.byebug_history

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list