Feature #7330
closed[SSO] Convert arvados-sso-server-upgrade.sh to fpm after-install/before-remove scripts
Description
The after-install script should, in the usual case, perform all the same actions that arvados-sso-server-upgrade.sh currently performs. However, there are some expected implementation differences:
- Be prepared to be called with dpkg's calling arguments or rpm's calling arguments. If some actions don't make sense in some calling scenarios, make sure they're skipped.
- Instead of introspecting the system to determine whether we're on Debian or Red Hat, use the calling arguments so we get that information directly from the packaging system, which should be more reliable.
- Instead of copying
/etc/arvados/sso/*.yml
to/var/www/arvados-sso/current/config
, we should make symlinks in/var
to files in/etc
, so the copies don't get out of sync with the canonical versions. If there's a way to do this directly with fpm, feel free to do that, since it will probably be more reliable. But if nothing else, you can make the symlinks in the after-install script, and add a before-remove script to remove them when the last version of the package is being removed (so they don't interfere with removing all of/var/www/arvados-sso
). - Update our install guide to remove references to running
arvados-sso-server-upgrade.sh
.
Updated by Brett Smith over 9 years ago
- Subject changed from [SSO] Convert arvados-sso-server-upgrade.sh to a postinst script in Debian packages to [SSO] Convert arvados-sso-server-upgrade.sh to an fpm after-install script
- Description updated (diff)
Updated by Brett Smith over 9 years ago
- Subject changed from [SSO] Convert arvados-sso-server-upgrade.sh to an fpm after-install script to [SSO] Convert arvados-sso-server-upgrade.sh to fpm after-install/before-remove scripts
- Description updated (diff)
Updated by Brett Smith over 9 years ago
- Description updated (diff)
We are kicking downgrades out of this ticket. We'll deal with them separately, later.
Updated by Brett Smith over 9 years ago
- Target version changed from Arvados Future Sprints to 2015-09-30 sprint
Updated by Ward Vandewege over 9 years ago
- Tracker changed from Bug to Feature
- Status changed from New to In Progress
Updated by Ward Vandewege over 9 years ago
- Assigned To changed from Nico César to Ward Vandewege
Updated by Ward Vandewege over 9 years ago
Branches to review:
arvados repository: 7330-improved-sso-package
arvados-dev repository: 7330-postinst-for-sso-server
Notes for review of 7330-postinst-for-sso-server:
Build package for one target:
WORKSPACE=path/to/sso-provider ./run-build-packages-one-target.sh --target ubuntu1204 --command run-build-packages-sso.sh
I've also created a 'run-test-packages-sso.sh' script, which runs a few basic tests. It makes sure the generated package can actually be installed and uninstalled without errors (and without leaving stray files behind):
WORKSPACE=path/to/sso-provider ./run-build-packages-one-target.sh --target ubuntu1204 --command run-test-packages-sso.sh
All targets:
WORKSPACE=path/to/sso-provider ./run-build-packages-all-targets.sh --command run-build-packages-sso.sh
Test all targets:
WORKSPACE=path/to/sso-provider ./run-build-packages-all-targets.sh --command run-test-packages-sso.sh
Notes for review of 7330-improved-sso-package:
This is a documentation branch. It updates the SSO server installation instructions. I encourage you to pick one of our supported distributions and do a manual installation according to the instructions (for example, in a Docker container).
Updated by Nico César over 9 years ago
I reviewed both branches.
arvados c6d10ad822ac94453d74371af0bf57072c43f018
1c82c1f875ca2681ff36e804fde8f6beb8f36c77
arvados-dev
I created my own deb for the installation and followed all the instructions from the documentation.
pull 'arvados' up to d084c7b404fa0cb597fbba4dcb007c087da49ba5 there are minor changes
LGTM!
Updated by Ward Vandewege over 9 years ago
- Status changed from In Progress to Resolved
- % Done changed from 0 to 100
Applied in changeset arvados|commit:4e8c321b344d8ec23de7ee7c68b2553d87defb67.