-
Notifications
You must be signed in to change notification settings - Fork 6
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
Fix site install #58
Fix site install #58
Conversation
@joecorall @aOelschlager Are you able to log in to a new site with this PR merged? My site seems to install ok, but I can't log in using admin/password. If I revert this PR, the install seems to fail here: https://github.com/Islandora-Devops/isle-buildkit/blob/297e2fcab18b1f10da8d4f265e9cfe5ce1a64a32/drupal/rootfs/etc/islandora/utilities.sh#L242 Trying that command in a working container gives this error:
I suspect this is a Drush 13 issue that needs to be fixed (maybe in starter site?) then we can roll this PR back. I think this is what @aOelschlager was seeing here Islandora-Devops/islandora-starter-site#162, but the error message was misleading. It looks like Drush is actually not working at all:
|
@joshdentremont - I am able to with this PR #63 |
Still not working for me with buildkit 3.4.0. It looks like there is something weird going on with the search api module in starter site. My Drush seems to be totally unusable, and it looks like the install is not creating the admin account. |
@joshdentremont - can you list your steps how you're doing a fresh install? Are you doing a fresh git clone of isle-site-template, and removing any/all old containers/volumes from your previous attempts. It's working for me locally and it's working in GitHub Actions CI tests on #63 so wondering what's going on for you locally.
The traditional step (which requires user interaction so we can't do it in CI) also works for me locally. Though it takes a few minutes after
|
@joecorall Are you able to log in as admin/password once the site is up? What I did was:
After doing this, my site is set up, but I can't log in using admin/password. It does look like drush is working though, so maybe something in the install_site function is causing the drush errors. I then reverted the changes in this PR and tried rebuilding the drupal container and it is throwing the same error:
By skipping the install_site function we are missing a lot of stuff, including the call to this - https://github.com/Islandora-Devops/isle-buildkit/blob/main/drupal/rootfs/usr/local/bin/install-drupal-site.sh, which I suspect might be why I can't log in with this PR. |
oh i see. i always just use |
I tried replacing the install_site command with just the drush command to get some debug info, like this
This is what gets spit out to the docker log
|
I just confirmed that this commit (Islandora-Devops/islandora-starter-site@d8ec762) is what broke the ability to use the install_site function. My guess is that Drush 13 changed something so that you can't run I think the way to fix this is to replace this line: https://github.com/Islandora-Devops/isle-buildkit/blob/main/drupal/rootfs/etc/islandora/utilities.sh#L242 with something that doesn't use Drush. I have just put in a PR for this - Islandora-Devops/isle-buildkit#349 Assuming that PR goes through, we should be able to revert this PR and go back to |
No description provided.