Why does the nginx sidecar mount data and config?
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 55/100
- Issue-Typ
- Refactoring
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Ruhig
- Tech-Stack
- helm, kubernetes, nginx
- Bereich
- devops, infrastructure
Rechercherichtung
Beginne mit den nginx-Sidecar-Mount-Definitionen und files/nginx.config.tpl und verfolge dann, ob die gemounteten data-, config- und /var/www/tmp-Pfade verwendet werden. Vergleiche das Ergebnis mit der dataVolumeMount-Änderung in #816 und dem Problem des Besitzes in #335. Erledigt bedeutet, den Bedarf für jeden Mount zu dokumentieren und das Chart zu aktualisieren, falls einige davon nicht erforderlich sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Go Template
- Sterne
- 536
- Forks
- 316
- Ø Merge
- 4 T. 16 Std.
- Gemergte PRs (30 T.)
- 2
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus nextcloud/helm
-
No native support for REDIS_USER enviroment varEvtl. vergeben @jholmes802 hat das vor 2 Tagen übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
-
Failed to inspect imageOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
-
nextcloud.openmetrics.allowedClients is silently ignored unless nextcloud.configs is setEvtl. vergeben @JanWelker hat das vor 18 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 78/100
-
External Redis not working: redis-session.ini: Permission deniedEvtl. vergeben @antoinetran hat das vor 23 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 55/100
Ähnliche Issues
-
github_actions
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
Hochfrequenz/aibap.mcp#578 ·
Maintainer antworten meist innerhalb von 1 Tag
-
triage:accepted
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
open-telemetry/otel-arrow#4343 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
mishraprafful/multihull#147 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
paperclipai/paperclip#15751 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
Maintainer antworten meist innerhalb von 1 Tag