-
Notifications
You must be signed in to change notification settings - Fork 490
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Install script should use environment variables where possible #3390
Comments
@telnoratti you seem to be saying you don't like the approach at https://github.com/IQSS/dataverse/blob/v4.5/scripts/deploy/phoenix.dataverse.org/install which is a bash script that defines a bunch of variables and then calls the At https://github.com/IQSS/dataverse/blob/v4.5/scripts/vagrant/install-dataverse.sh#L13 I call |
That script runs the glassfish-setup.sh and that works fine, especially I doubt users are trying to run glassfish-setup.sh in a non-automated way. The problem is with the install script. Being able to specify all arguments as cli options is useful, but I think doesn't cover the use case of the Hope this clears up my issue. |
@telnoratti does the work done by @pameyer in #3799 help? |
My guess would be that it helps, but does not solve. The advantage of the approach outlined above is that defaults would be specified as key-value pairs, which causes things to work better if the fields are changed in the future. |
@oscardssmith there already key value pairs at http://guides.dataverse.org/en/4.7/_downloads/default.config This is documented at http://guides.dataverse.org/en/4.7/installation/installation-main.html#running-the-dataverse-installer |
@telnoratti did you get a chance to look at #3799 or the docs linked above for running the installer non-interactively? Also, there was some discussion today having to do with #4763 how at least in an OpenShift (and maybe Kubernetes) environment, there's a preference for not specifying secrets explicitly and only referencing them as environment variables. Also, please feel free to weigh in on the future of the dataverse-aws repo at IQSS/dataverse-aws#11 Thanks! |
@telnoratti I'd like to encourage you to review the installer redesign in pull request #4805 for #4764. Closing. |
Currently the install script prompts on every variable that needs to be set, usually with some default. This is not particularly conducive for automated deployments as inserting a new input would break any scripts used to configure it now in unpredictable ways. For the dataverse-aws project I need to provide a newline delimited list of options and I'm concerned about this breaking in future updates that add new variables. I suggest the following behaviour:
Step 2 would fix the issue of unpredictable builds and potentially breaking deployments, but since it requires the user consciously trying to deploy in an automated fashion it would not break the user experience when simply running the install script.
The text was updated successfully, but these errors were encountered: