inveniosoftware / helm-invenio

Helm charts for deploying an Invenio instance
https://helm-invenio.readthedocs.io
7 stars 19 forks source link

Fix issues in `volumeMounts` #124

Closed lindhe closed 3 months ago

lindhe commented 3 months ago

Description

This PR does three things:

  1. Reverts an earlier commit that removed subPath from the volumeMounts.
  2. Adds subPath to the worker pod, where it was missing.
  3. Fixes a typo for the volume name.

The reverted commit tried to fix an issue where we got duplicate directory names (data/data/), but the issue was not that web used subPath but actually that worker did not.

Checklist

Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

Frontend

Reminder

By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

  1. You license your contribution under the same terms as the current repository’s license.
  2. You agree that you have the right to license your contribution under the current repository’s license.