NVIDIA / holodeck / 21830415155

46%

main: 48%

Build Type

push

github

Committed by ArangoGutierrez

Commit Message

fix: critical bugs — storageSizeGB race, log.Fatalf in goroutine, file permissions

- Use local variable for storageSizeGB instead of taking address of
  package-level var, preventing data race and pointer aliasing with AWS SDK
- Replace log.Fatalf in provisioner goroutine with proper error propagation
  via channel and sync.WaitGroup, preventing process crash and goroutine leaks
- Simplify status.go update() to MkdirAll + WriteFile + Chmod(0600),
  removing redundant os.Create(0666) that left cache files world-readable

Re-implemented against current upstream/main (a19ea845).

Signed-off-by: Carlos Eduardo Arango Gutierrez <eduardoa@nvidia.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Carlos Eduardo Arango Gutierrez <eduardoa@nvidia.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Carlos Eduardo Arango Gutierrez <eduardoa@nvidia.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Carlos Eduardo Arango Gutierrez <eduardoa@nvidia.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Carlos Eduardo Arango Gutierrez <eduardoa@nvidia.com>
Co-authored-by: Cursor <cursoragent@cursor.com>

Run Details

5 of 27 new or added lines in 3 files covered. (18.52%)

5 existing lines in 2 files now uncovered.

2313 of 4989 relevant lines covered (46.36%)

0.52 hits per line