Skip to content

re-add docker network create dbs #9648

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bpow
Copy link
Contributor

@bpow bpow commented Jun 23, 2025

The code to create the dbs network has a storied past:

The docker-compose.yml says this network should be created automatically (although there's a misspelling there, fixed here as a bonus), and without this network, trying to follow the dev environment setup if this network had not already been created results in a cryptic error.

Of course, the real fix would be to address this in the cookiecutter to prevent this from
being overwritten the next time that might be applied, so I'll make a pull request there, too.

The code to create the dbs network has a storied past:
- added hypothesis#7673
- removed abbe35c
- added again c2a4fe7
- removed again eb998e1 when cookiecutter was applied to h but
  didn't include this code.

The docker-compose.yml says this network should be created automatically
(although there's a misspelling there, fixed here as a bonus), and
without this network, trying to follow the dev environment setup if this
network had not already been created results in a cryptic error.
@bpow bpow force-pushed the create-network-dbs branch from 262cf7b to 9b4b123 Compare June 23, 2025 22:21
@bpow
Copy link
Contributor Author

bpow commented Jun 23, 2025

Force-pushed to fixup with yet another misspelling (I hadn't noticed the "docker crate network" misspelling was both in the compose file and in an include provided to cookiecutter.

Not sure how y'all might want to handle the fix in cookiecutter-- is it OK to include this @docker network create dbs there when other projects within h may use it? Because of this, I'll leave making the fix there as "an exercise to the reader".

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