Why does the nginx sidecar mount data and config?
还没有人认领这个 Issue。
评估
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 新手友好度
- 55/100
- Issue 类型
- 重构
- 描述清晰度
- 基本清楚
- 活跃度
- 冷清
- 技术栈
- helm, kubernetes, nginx
- 领域
- devops, infrastructure
调研方向
从 nginx sidecar 的挂载定义和 files/nginx.config.tpl 开始，然后追踪已挂载的 data、config 和 /var/www/tmp 路径是否被使用。将结果与 #816 中的 dataVolumeMount 变更以及 #335 中关于所有权的问题进行比较。完成的标准是记录每个挂载的必要性，并在有任何挂载不必要时更新 chart。
由索引模型根据 Issue 内容生成。
描述
The nginx sidecar gets data, config and /var/www/tmp mounted, and I can't work out what it uses them for.
The default server block already refuses both paths (files/nginx.config.tpl):
location ~ ^/(?:build|tests|config|lib|3rdparty|templates|data)(?:$|/) { return 404; }
That's a regex location declared before the static file ones, so it matches first and nginx never touches the filesystem for those URLs. And /var/www/tmp sits outside root /var/www/html, so it isn't reachable at all.
Am I missing a case where nginx actually needs to read them? Asking partly because #816 is adding a dataVolumeMount flag that also applies to the nginx container, and if the mount is never used there then it does not need the flag either. Dropping the three mounts would also mean the internet-facing container no longer has the user data directory attached, and would sidestep the /var/www/html/config ownership trouble in #335.
- 主要语言
- Go Template
- 星标
- 537
- 派生
- 317
- 平均合并
- 2 天 3 小时
- 30 天内合并 PR
- 1
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
nextcloud/helm 的其他 Issue
-
No native support for REDIS_USER enviroment var可能已有人在做 @jholmes802 于 10 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 70/100
-
难度 2/5 1-3 小时 新手友好度 72/100
-
难度 2/5 1-3 小时 新手友好度 68/100
-
Nextcloud pods run as the namespace's default service account: the chart's account is only created with rbac.enabled, which binds the removed PodSecurityPolicy可能已有人在做 @martin-mat 于 2 天前认领。 未关闭
难度 3/5 半天 新手友好度 55/100
-
nextcloud.openmetrics.allowedClients is silently ignored unless nextcloud.configs is set可能已有人在做 @JanWelker 于 27 天前认领。 未关闭
难度 3/5 1-2 天 新手友好度 78/100
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 66/100
-
good first issue
难度 2/5 半天 新手友好度 78/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 76/100
oposs/repo-infra#93 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 83/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
cloudflare/terraform-provider-cloudflare#7423 ·
维护者通常 1 天内回复