Why does the nginx sidecar mount data and config?
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 55/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- helm, kubernetes, nginx
- Lĩnh vực
- devops, infrastructure
Hướng nghiên cứu
Bắt đầu với các định nghĩa mount của nginx sidecar và files/nginx.config.tpl, sau đó kiểm tra xem data, config và các đường dẫn /var/www/tmp được mount có được sử dụng hay không. So sánh kết quả với thay đổi dataVolumeMount trong #816 và mối lo ngại về quyền sở hữu trong #335. Hoàn tất nghĩa là ghi lại nhu cầu đối với từng mount và cập nhật chart nếu có mount nào không cần thiết.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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.
- Ngôn ngữ chính
- Go Template
- Star
- 537
- Fork
- 317
- Merge trung bình
- 2 ngày 3 giờ
- Pull request đã merge (30 ngày)
- 1
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của nextcloud/helm
-
No native support for REDIS_USER enviroment varCó thể đã có người làm @jholmes802 đã nhận 8 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
Failed to inspect imageĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
nextcloud.openmetrics.allowedClients is silently ignored unless nextcloud.configs is setCó thể đã có người làm @JanWelker đã nhận 25 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 78/100
-
External Redis not working: redis-session.ini: Permission deniedCó thể đã có người làm @antoinetran đã nhận 30 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
Tất cả issue của nextcloud/helm
Issue tương tự
-
ci: nothing regenerates the Go contract types, so a hand-edit to types.gen.go passesCó thể đã có người làm @shaiknoorullah đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
shaiknoorullah/hg-mono#993 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[Bug] web.deploymentStrategy and admintools.deploymentStrategy are ignored, and component ServiceAccounts have no namespaceCó thể đã có người làm @wankhede04 đã nhận hôm nay. Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
temporalio/helm-charts#1006 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 63/100
opdev/vuln-scan-cert#44 ·
-
agent:needs-review documentation feature good first issue P3 · 低 隐私与合规
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
ASDW51/DanmuFloat#102 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100