Skip to content
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

[bitnami/moodle] Fix moodle pv and pvc templates #31026

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

mdochdev
Copy link

This patch is to fix the typo in the moodle templates for volumes.
pvc has been updated to add the 's' on accessModes. pv has been updated to match the pvc.
IE: Support both arrays and a single string.

Description of the change

Update the moodle Volume templates

Benefits

Fix a bug for referencing local disk

Possible drawbacks

None that I can see

Applicable issues

#30976

Additional information

N/A

Checklist

Fix a typo

Signed-off-by: mDoch Dev <[email protected]>
Fixed pv like pvc

Signed-off-by: mDoch Dev <[email protected]>
@github-actions github-actions bot added moodle triage Triage is needed labels Dec 13, 2024
@github-actions github-actions bot requested a review from carrodher December 13, 2024 01:21
@carrodher
Copy link
Member

Thanks for your contribution! Could you please bump the chart version in the Chart.yaml? This is necessary to test the changes and cut a new release.

mdochdev and others added 2 commits December 14, 2024 08:34
Updated chart version

Signed-off-by: mDoch Dev <[email protected]>
Signed-off-by: Bitnami Containers <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
moodle triage Triage is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

accessMode vs accessModes
3 participants