Skip to content

Commit 9d97fe7

Browse files
authored
remove dependence on and references to db docker definition in delphi/operations repo (#1000)
* remove references to database docker definition in delphi/operations * some documentation updates
1 parent c2185ba commit 9d97fe7

File tree

7 files changed

+9
-16
lines changed

7 files changed

+9
-16
lines changed

.github/workflows/ci.yaml

-1
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,6 @@ jobs:
5151

5252
- name: Build docker images
5353
run: |
54-
docker build -t delphi_database -f repos/delphi/operations/dev/docker/database/Dockerfile .
5554
docker build -t delphi_python -f repos/delphi/operations/dev/docker/python/Dockerfile .
5655
docker build -t delphi_database_epidata -f ./repos/delphi/delphi-epidata/dev/docker/database/epidata/Dockerfile .
5756
docker build -t delphi_web_python -f repos/delphi/delphi-epidata/dev/docker/python/Dockerfile .

dev/docker/database/epidata/Dockerfile

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ ENV MYSQL_ROOT_PASSWORD pass
1313
# create the `epidata` database
1414
ENV MYSQL_DATABASE epidata
1515

16-
# create the `epi` user account with a development-only password
16+
# create the `user` account with a development-only password
1717
ENV MYSQL_USER user
1818
ENV MYSQL_PASSWORD pass
1919

dev/docker/database/epidata/README.md

+4-4
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
# `delphi_database_epidata`
22

3-
This image extends Delphi's database by:
3+
This image extends a Percona database by:
44

5-
- adding the `epi` user account
6-
- adding the `epidata` database
7-
- creating empty tables in `epidata`
5+
- adding the `user` account
6+
- adding the `epidata` & other appropriate databases
7+
- creating empty tables in those databases
88

99
To start a container from this image, run:
1010

dev/docker/python/Dockerfile

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
# start with the `delphi_python` image
1+
# start with the `delphi_python` image, which is built from repos/delphi/operations/dev/docker/python/Dockerfile
22
FROM delphi_python
33

44
RUN pip install --no-cache-dir -r repos/delphi/delphi-epidata/requirements.txt -r repos/delphi/delphi-epidata/requirements.dev.txt

dev/local/Makefile

+1-5
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ web:
7878
@# Build the web_epidata image
7979
@cd repos/delphi/delphi-epidata;\
8080
docker build -t delphi_web_epidata -f ./devops/Dockerfile .;\
81-
cd ../../../
81+
cd -
8282

8383
@# Run the web server
8484
@docker run --rm -p 127.0.0.1:10080:80 \
@@ -97,10 +97,6 @@ db:
9797
@# Setup virtual network if it doesn't exist
9898
@docker network ls | grep delphi-net || docker network create --driver bridge delphi-net
9999

100-
@# Only build prereqs if we need them
101-
@docker images delphi_database | grep delphi || \
102-
docker build -t delphi_database -f repos/delphi/operations/dev/docker/database/Dockerfile .
103-
104100
@# Build the database_epidata image
105101
@docker build -t delphi_database_epidata \
106102
-f repos/delphi/delphi-epidata/dev/docker/database/epidata/Dockerfile .

docs/epidata_development.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -119,8 +119,8 @@ above. The base images are built first, followed by the derived
119119
the Epidata API to the `delphi_web` image.
120120
- The
121121
[`delphi_database_epidata` image](https://github.com/cmu-delphi/delphi-epidata/blob/main/dev/docker/database/epidata/README.md)
122-
adds the `epi` user account, `epidata` database, and relevant tables
123-
(initially empty) to the `delphi_database` image.
122+
adds user accounts, `epidata` & other appropriate databases, and relevant tables
123+
(initially empty) to a Percona database image.
124124

125125
From the root of your workspace, all of the images can be built as follows:
126126

docs/new_endpoint_tutorial.md

-2
Original file line numberDiff line numberDiff line change
@@ -277,8 +277,6 @@ docker build -t delphi_web \
277277
-f repos/delphi/operations/dev/docker/web/Dockerfile .
278278
docker build -t delphi_web_epidata \
279279
-f repos/delphi/delphi-epidata/dev/docker/web/epidata/Dockerfile .
280-
docker build -t delphi_database \
281-
-f repos/delphi/operations/dev/docker/database/Dockerfile .
282280
docker build -t delphi_database_epidata \
283281
-f repos/delphi/delphi-epidata/dev/docker/database/epidata/Dockerfile .
284282

0 commit comments

Comments
 (0)