Skip to content
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

Quick-start script fails if run a second time (with different data) without bringing containers down first #900

Closed
maxalbert opened this issue Jun 5, 2019 · 2 comments · Fixed by #1130
Labels
bug Something isn't working quick-start-script

Comments

@maxalbert
Copy link
Contributor

To reproduce:

The first run of the quick-start script succeeds.

$ wget https://raw.githubusercontent.com/Flowminder/FlowKit/master/quick_start.sh

$ bash quick_start.sh
Starting containers (this may take a few minutes)
Pulling flowdb                   ... done
Pulling flowdb_testdata          ... done
Pulling flowdb_synthetic_data    ... done
Pulling worked_examples          ... done
Pulling flowapi                  ... done
Pulling flowauth                 ... done
Pulling flowmachine_query_locker ... done
Pulling flowmachine              ... done
Pulling flowetl                  ... done
Pulling flowetl_db               ... done
[...]
All containers ready!
[...]

However, the second run fails if we try to bring up the system with larger_data but without running bash quick_start.sh stop first.

$ bash quick_start.sh larger_data
Starting containers (this may take a few minutes)
Existing containers are running and will be replaced. Are you sure? [y/N]? y

[...]
flowmachine_query_locker is up-to-date
Creating flowdb_synthetic_data ... 
flowauth is up-to-date
flowapi is up-to-date
Creating flowdb_synthetic_data ... error

ERROR: for flowdb_synthetic_data  Cannot start service flowdb_synthetic_data: driver failed programming external connectivity on endpoint flowdb_synthetic_data (174f969ed534104329099edb017829020356c81f62511bdfd6346185bf59300a): Bind for 0.0.0.0:9000 failed: port is already allocated

ERROR: for flowdb_synthetic_data  Cannot start service flowdb_synthetic_data: driver failed programming external connectivity on endpoint flowdb_synthetic_data (174f969ed534104329099edb017829020356c81f62511bdfd6346185bf59300a): Bind for 0.0.0.0:9000 failed: port is already allocated
ERROR: Encountered errors while bringing up the project.
@maxalbert maxalbert added bug Something isn't working quick-start-script labels Jun 5, 2019
@greenape
Copy link
Member

greenape commented Jun 5, 2019

Guess we need an explicit down in case you aren’t bringing up the same image.

@greenape
Copy link
Member

Post #1057, could we resolve this by giving all the containers the same name? Would also reduce the duplication, because compose merges the environment section if overridden in a subsequent docker file.

greenape added a commit that referenced this issue Aug 6, 2019
…ut putting them in different compose files
greenape added a commit that referenced this issue Aug 7, 2019
…ut putting them in different compose files
greenape added a commit that referenced this issue Aug 7, 2019
…ut putting them in different compose files
greenape added a commit that referenced this issue Aug 12, 2019
…ut putting them in different compose files
greenape added a commit that referenced this issue Aug 12, 2019
…ut putting them in different compose files
greenape added a commit that referenced this issue Aug 15, 2019
…ut putting them in different compose files
greenape added a commit that referenced this issue Aug 19, 2019
…ut putting them in different compose files
@mergify mergify bot closed this as completed in #1130 Aug 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working quick-start-script
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants