Skip to content

Brev: Remove unused Docker-in-Docker to reduce image size.#156

Open
brycelelbach wants to merge 1 commit intomainfrom
fix/remove-docker-in-docker
Open

Brev: Remove unused Docker-in-Docker to reduce image size.#156
brycelelbach wants to merge 1 commit intomainfrom
fix/remove-docker-in-docker

Conversation

@brycelelbach
Copy link
Copy Markdown
Collaborator

@brycelelbach brycelelbach commented Mar 11, 2026

Summary

  • Remove Docker-in-Docker installation from the accelerated-python, cuda-cpp, and nvmath-python Brev dockerfiles.
  • Docker was unused inside these containers and removing it reduces image size.
$ brev/test-docker-image-sizes.bash compare --owner nvidia
 --tutorial accelerated-python --tag pull-request-156-latest --baseline main-latest
========================================
Image: ghcr.io/nvidia/accelerated-python-tutorial
========================================
  Current    (pull-request-156-latest)                12.53 GB
  Baseline   (main-latest)                            12.66 GB
  ────────────────────────────────────────────────────
  Change:  -130.09 MB (-1.00%)

@brycelelbach brycelelbach enabled auto-merge (rebase) March 11, 2026 18:56
@brycelelbach brycelelbach force-pushed the fix/remove-docker-in-docker branch 2 times, most recently from 97879f3 to d96ff17 Compare March 12, 2026 13:14
@brycelelbach brycelelbach force-pushed the fix/remove-docker-in-docker branch from d96ff17 to f52db3b Compare March 30, 2026 17:11
@github-actions
Copy link
Copy Markdown
Contributor

❌ Link Check Failed

Broken links were detected in this PR.

Please check the workflow run logs for details on which links are broken.

Common fixes:

  1. Typo in URL - Check for spelling mistakes in the link
  2. Outdated link - The page may have moved or been deleted
  3. Relative path issue - Ensure relative links use the correct path
  4. External site down - If the external site is temporarily down, you can add it to brev/.lycheeignore

To test links locally:

./brev/test-links.bash .

📚 Lychee documentation

@brycelelbach brycelelbach force-pushed the fix/remove-docker-in-docker branch from f52db3b to f79c649 Compare March 30, 2026 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant