Why does the nginx sidecar mount data and config?
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 55/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- helm, kubernetes, nginx
- Área
- devops, infrastructure
Línea de trabajo
Comienza con las definiciones de montaje del sidecar de nginx y files/nginx.config.tpl; después, sigue el rastro para comprobar si se utilizan los datos montados, la configuración y las rutas /var/www/tmp. Compara el resultado con el cambio de dataVolumeMount en #816 y con la preocupación sobre la propiedad en #335. Se considera terminado cuando se documente la necesidad de cada montaje y se actualice el chart si alguno es innecesario.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Go Template
- Estrellas
- 536
- Forks
- 316
- Merge medio
- 4 d 16 h
- PR fusionados (30 d)
- 2
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de nextcloud/helm
-
No native support for REDIS_USER enviroment varPosiblemente ocupada @jholmes802 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Failed to inspect imageAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
nextcloud.openmetrics.allowedClients is silently ignored unless nextcloud.configs is setPosiblemente ocupada @JanWelker la tomó hace 17 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 78/100
-
External Redis not working: redis-session.ini: Permission deniedPosiblemente ocupada @antoinetran la tomó hace 21 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
Todos los issues de nextcloud/helm
Issues similares
-
Sanity on ansible-core devel fails: ignore-2.23.txt references the removed import-3.9 testPosiblemente ocupada @yurnov la tomó hoy. Abiertoneeds_triage
Dificultad 1/5 Menos de una hora Aptitud para principiantes 91/100
ansible-collections/kubernetes.core#1275 ·
Los mantenedores suelen responder en 1 día
-
bug milestone-qa
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
lognorman20/monaco#3995 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
gnosis/gnosis_vpn#540 ·
Los mantenedores suelen responder en 1 día
-
[fullsend] E2E: rhdh-version-override — run-e2e.sh overrides RHDH_VERSION to non-existent 2.1Abiertoe2e-failure ready-to-code
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
redhat-developer/rhdh-plugin-export-overlays#4261 · 1 comentario ·
Los mantenedores suelen responder en 1 día