[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