-
-
Notifications
You must be signed in to change notification settings - Fork 260
Build initial Drupal 8 codebase if not present #104
Comments
New docker image for building off of: https://hub.docker.com/r/geerlingguy/drupal/ |
D'oh, I could just recommend people set the environment vars in settings.php instead, e.g.: How to Use Environment Variables for your Drupal Settings with Docksal. Code example: $databases['default']['default'] = [
'driver' => 'mysql',
'database' => getenv('DRUPAL_DATABASE'),
'username' => getenv('DRUPAL_DATABASE_USER'),
'password' => getenv('DRUPAL_DATABASE_PASSWORD'),
'host' => getenv('DRUPAL_DATABASE_HOST'),
]; |
As part of geerlingguy/drupal-container#3, I'm adding a 'download Drupal core if it's not already present' feature to the entrypoint. |
Blocked by two things:
|
Worked on #2 above... we'll see if that works. |
Was having a 'fun' time debugging why Posted follow-up comment in kubernetes/kubernetes#60835 (comment) to help if I run into this in the future :) |
One good thing... I can confirm that even if you blow away the |
Fixed via above commit. |
So, right now the Kubernetes branch is using the drupal repository, which includes a copy of the Drupal codebase, but is not specifically intended for production usage (though it could be hacked around to do so...).
One annoying side effect is that on every reboot or Pod replacement of the
drupal8
service, we end up with a codebase divorced from the database!So, instead, I'd like to:
For all of these steps, they should happen after Kubernetes is started and stabilized.
I haven't yet decided if I want to just host a separate Git repo with the Drupal codebase in it on GitHub, so I don't have to rebuild it forever and ever for testing purposes... it's kind of annoying and takes a bit of time. But it might be better for educational purposes anyways :/
The text was updated successfully, but these errors were encountered: