Skip to content

fix(docker): fix Nginx path #66#69

Open
timoa wants to merge 1 commit into
offendingcommit:mainfrom
timoa:fix/66-docker-helm
Open

fix(docker): fix Nginx path #66#69
timoa wants to merge 1 commit into
offendingcommit:mainfrom
timoa:fix/66-docker-helm

Conversation

@timoa

@timoa timoa commented Jun 16, 2026

Copy link
Copy Markdown

Summary

Root Cause

The Helm chart deploys with readOnlyRootFilesystem: true, but the tmpfsMounts was missing two directories that nginx's entrypoint and the openconcho config script need to write to at startup:

/etc/nginx/conf.d/ — where nginx renders the template and the config script writes 00-resolver.conf / allowlist_map.conf
/usr/share/nginx/html/ — where the config script was writing config.js

Fix Applied

Three files changed:

charts/openconcho/values.yaml:55-59: Added /etc/nginx/conf.d to tmpfsMounts

docker/40-openconcho-config.sh:10: Changed output path from /usr/share/nginx/html/config.js to /tmp/openconcho-config.js

docker/nginx.conf.template:40-43: Updated the /config.js location to serve from /tmp/openconcho-config.js via alias instead of try_files

Type

  • Bug fix
  • New feature
  • Refactor
  • Docs / chore

Test plan

  • pnpm lint passes
  • pnpm test passes
  • pnpm build succeeds
  • Tested in browser (describe what you verified)

Related issues

#66

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant