Why does the nginx sidecar mount data and config?
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 55/100
- Tipo de issue
- Refatoração
- Clareza
- Razoavelmente clara
- Status de atividade
- Pouca atividade
- Stack de tecnologia
- helm, kubernetes, nginx
- Domínio
- devops, infrastructure
Direção de pesquisa
Comece pelas definições de montagem do sidecar nginx e por files/nginx.config.tpl; em seguida, verifique se os dados montados, a configuração e os caminhos /var/www/tmp são usados. Compare o resultado com a alteração de dataVolumeMount em #816 e com a preocupação relacionada à propriedade em #335. Considera-se concluído quando a necessidade de cada montagem estiver documentada e o chart for atualizado caso alguma seja desnecessária.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- Go Template
- Estrelas
- 536
- Forks
- 316
- Merge médio
- 4d 16h
- PRs com merge (30d)
- 2
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de nextcloud/helm
-
No native support for REDIS_USER enviroment varTalvez já em andamento @jholmes802 assumiu há 2 dias. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
-
Failed to inspect imageAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
-
nextcloud.openmetrics.allowedClients is silently ignored unless nextcloud.configs is setTalvez já em andamento @JanWelker assumiu há 18 dias. Aberta
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 78/100
-
External Redis not working: redis-session.ini: Permission deniedTalvez já em andamento @antoinetran assumiu há 23 dias. Aberta
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 55/100
Todas as issues de nextcloud/helm
Issues semelhantes
-
github_actions
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
Hochfrequenz/aibap.mcp#578 ·
Mantenedores costumam responder em até 1 dia
-
triage:accepted
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
open-telemetry/otel-arrow#4343 ·
Mantenedores costumam responder em até 2 dias
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
mishraprafful/multihull#147 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
paperclipai/paperclip#15751 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
Mantenedores costumam responder em até 1 dia