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/apache] Fix htdocs with readOnlyRootFilesystem #26304

Merged
merged 2 commits into from
May 29, 2024

Conversation

migruiz4
Copy link
Member

Description of the change

Fixes an issue with apache, where deployment will fail when htdocs are mounted.

apache 09:12:49.11 INFO  ==> ** Starting Apache setup **
apache 09:12:49.22 INFO  ==> Mounting application files from '/app'
rm: cannot remove '/opt/bitnami/apache/htdocs/index.html': Read-only file system

This issue is caused by this section of the image logic:

if ! is_dir_empty "/app"; then
    info "Mounting application files from '/app'"
    rm -rf "$APACHE_HTDOCS_DIR"
    ln -sf "/app" "$APACHE_HTDOCS_DIR"
fi

Mounting the htdocs at /app will attempt to remove /opt/bitnami/apache/htdocs, causing the above permission error.

Possible drawbacks

This PR mounts the htdocs directly at /opt/bitnami/apache/htdocs, causing the /app folder to remain empty from now on.

Applicable issues

Checklist

  • Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
  • Variables are documented in the values.yaml and added to the README.md using readme-generator-for-helm
  • Title of the pull request follows this pattern [bitnami/<name_of_the_chart>] Descriptive title
  • All commits signed off and in agreement of Developer Certificate of Origin (DCO)

Signed-off-by: Miguel Ruiz <miruiz@vmware.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
@javsalgar javsalgar removed their assignment May 27, 2024
@migruiz4 migruiz4 merged commit 243446b into main May 29, 2024
8 checks passed
@migruiz4 migruiz4 deleted the apache-fix-htdocs branch May 29, 2024 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apache bitnami solved verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bitnami/apache] htdocsConfigMap is incompatible with containerSecurityContext
4 participants