Skip to content

docs: fix docker container names in local setup guides#577

Merged
lfnovo merged 1 commit into
mainfrom
fix/docker-container-names-docs
Feb 14, 2026
Merged

docs: fix docker container names in local setup guides#577
lfnovo merged 1 commit into
mainfrom
fix/docker-container-names-docs

Conversation

@lfnovo
Copy link
Copy Markdown
Owner

@lfnovo lfnovo commented Feb 14, 2026

Summary

  • Fix incorrect container names in quick-start-local.md (6 occurrences) and docker-compose.md (1 occurrence)
  • Docker Compose v2 derives container names from the directory name using dashes, so the correct name is open-notebook-local-ollama-1, not open_notebook-ollama-1

Test plan

  • Verified all docker exec commands reference the correct container name matching the open-notebook-local directory name

Fixes #575

Docker Compose v2 derives container names from the directory name using
dashes. Since the docs instruct users to create an `open-notebook-local`
folder, the correct container name is `open-notebook-local-ollama-1`,
not `open_notebook-ollama-1`.

Fixes #575
@lfnovo lfnovo merged commit 9811c58 into main Feb 14, 2026
6 checks passed
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

@lfnovo lfnovo mentioned this pull request Feb 15, 2026
demobdev pushed a commit to demobdev/open-notebook that referenced this pull request Feb 25, 2026
Docker Compose v2 derives container names from the directory name using
dashes. Since the docs instruct users to create an `open-notebook-local`
folder, the correct container name is `open-notebook-local-ollama-1`,
not `open_notebook-ollama-1`.

Fixes lfnovo#575
@lfnovo lfnovo deleted the fix/docker-container-names-docs branch April 6, 2026 11:26
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.

[Install]: quick-start-local.md commands examples with wrong container_name

1 participant