orchestrator: NFS-cache "cache path is not set" WARN logged on every build/launch instead of once

Abierto Apto para principiantes
  • ¿Algo no está bien?
#3,786 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 2 días

@AdaAibaby ya está trabajando en esto.

Desde el 2/11/2026.

  • #3787 de @AdaAibaby — abierto

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
72/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
go
Área
backend

Línea de trabajo

Empieza con los dos sitios de llamada mencionados en el issue: la comprobación de useNFSCache alrededor de packages/orchestrator/pkg/template/build/builder.go:304 y la que está alrededor de packages/orchestrator/pkg/sandbox/template/cache.go:1218. El helper propuesto debe ir en packages/shared/pkg/storage, junto a storage.WrapInNFSCache. Se considera terminado cuando la advertencia aparece una sola vez por proceso, su texto indica que la caché NFS sigue deshabilitada hasta que se configure el directorio de caché, y una prueba unitaria cubre el comportamiento de una sola vez.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Summary

When an NFS-cache feature flag is enabled but no cache path is configured, the orchestrator logs WARN "NFSCache feature flag is enabled but cache path is not set" on the hot path — once per template build, per sandbox launch, and per snapshot resume — and NFS caching is silently disabled. On a dev-default deployment this floods the logs (hundreds of identical lines observed) while telling the operator nothing new after the first line.

Why it happens by default

Three flags default to env.IsDevelopment() (i.e. on in dev):

  • use-nfs-for-building-templates (UseNFSCacheForBuildingTemplatesFlag)
  • use-nfs-for-snapshots (SnapshotFeatureFlag)
  • use-nfs-for-templates (TemplateFeatureFlag)

If the shared chunk cache dir is not set (common in a plain dev bring-up), each of the two call sites warns on every operation:

  • packages/orchestrator/pkg/template/build/builder.go:304 (useNFSCache, per build)
  • packages/orchestrator/pkg/sandbox/template/cache.go:1218 (useNFSCache, per launch/resume)

Both emit the identical message, and both return "cache disabled". So the warning is pure repetition — the path comes from immutable config and won't change within a process.

Proposal

Emit the warning once per process instead of on every operation, and make the message state the consequence (NFS caching disabled until the cache dir is configured). A shared WarnNFSCachePathUnsetOnce(ctx) in packages/shared/pkg/storage (already imported by both call sites, since both call storage.WrapInNFSCache) guarded by sync.Once keeps it to a single line and removes the duplicated string. Behavior (cache disabled) is unchanged.

Happy to send the PR (small, with a unit test).

Lenguaje dominante
Go
Estrellas
1.8k
Forks
503
Sin PR fusionados en 30 d

Preparar el entorno

Abrir en Codespaces

Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de e2b-dev/runtime

Todos los issues de e2b-dev/runtime

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.