search

drupalwxt / helm-drupal

Helm Chart for deploying an enterprise-grade Drupal environment.
https://drupalwxt.github.io/helm-drupal/index.yaml
MIT License
31 stars 22 forks source link

feat(drupal): Allow default install to work single node #143

Closed sylus closed 1 year ago

sylus commented 1 year ago

Currently the default install won't fully work until Azure Files are setup so that the /var/www/html/sites/default/files are shared between the drupal and nginx pods.

Depending on the cloud this can be setup in 1...N ways, however we do provide examples for how this is specifically done either with Azure Files or Azure Shared Disk.

You can see a full example using Azure Files using the CSI Driver here:

I was thinking we should make the default install work with the caveat it will only work when you only have 1 node in the cluster because the PVC can only attach to multiple nodes if it is the one same node or you will get a multi attach error.

This technically isn't best practice so wanted to see if this is something we should add as a default.

PatrickMessierGoC commented 1 year ago

If you decide to move forward with this change, I would suggest to add some context (test purpose), limitation (single node k8s cluster) and instructions (helm command) on the readme page.