Skip to content

chore: remove data load #43

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.

Sign up for GitHub

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
delcroip wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

chore: remove data load #43

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
4adff56
chore: remove data load
delcroip Sep 9, 2025
3a98ee4
chore: put back admin
delcroip Sep 9, 2025
8ca0113
chore: id 1 for Admin role
delcroip Sep 10, 2025
a1b964f
chore: remove tblLogins
delcroip Sep 23, 2025
0c471d9
empty should be default
delcroip Nov 12, 2025
ff78329
chore: empty should be default
delcroip Nov 12, 2025
0139860
fix move await demo in int script
delcroip Nov 12, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions .github/workflows/docker-dev-cd.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,9 @@ jobs:
password: ${{ secrets.GITHUB_TOKEN }}
- name: Build the pgsql EMPTY Docker image
run: |
docker build . --tag ghcr.io/openimis/openimis-pgsql:develop-base --target base
docker push ghcr.io/openimis/openimis-pgsql:develop-base
docker build . --tag ghcr.io/openimis/openimis-pgsql:develop --target base
docker push ghcr.io/openimis/openimis-pgsql:develop
- name: Build the pgsql DEMO Docker image
run: |
docker build . --tag ghcr.io/openimis/openimis-pgsql:develop
docker push ghcr.io/openimis/openimis-pgsql:develop
docker build . --tag ghcr.io/openimis/openimis-pgsql:develop-demo
docker push ghcr.io/openimis/openimis-pgsql:develop-demo
Comment on lines +20 to +25
Copy link

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

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

Similarly for the develop channel, the develop tag now publishes the base/empty image, which can break environments that track develop and expect the demo dataset. If this is intentional, it should be paired with an explicit migration path (e.g., keep pushing develop demo as well, or update any deployment manifests/README references that pull develop).

Copilot uses AI. Check for mistakes.
8 changes: 4 additions & 4 deletions .github/workflows/docker.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,9 @@ jobs:
password: ${{ secrets.GITHUB_TOKEN }}
- name: Build the pgsql EMPTY Docker image
run: |
docker build . --target base --tag ghcr.io/openimis/openimis-pgsql:${{ github.event.inputs.tag }}-base
docker push ghcr.io/openimis/openimis-pgsql:${{ github.event.inputs.tag }}-base
docker build . --target base --tag ghcr.io/openimis/openimis-pgsql:${{ github.event.inputs.tag }}
docker push ghcr.io/openimis/openimis-pgsql:${{ github.event.inputs.tag }}
- name: Build the pgsql DEMO Docker image
run: |
docker build . --tag ghcr.io/openimis/openimis-pgsql:${{ github.event.inputs.tag }}
docker push ghcr.io/openimis/openimis-pgsql:${{ github.event.inputs.tag }}
docker build . --tag ghcr.io/openimis/openimis-pgsql:${{ github.event.inputs.tag }}-demo
docker push ghcr.io/openimis/openimis-pgsql:${{ github.event.inputs.tag }}-demo
Comment on lines +22 to +27
Copy link

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

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

This changes the published tag contract: ${tag} now points to the base/empty image (via --target base) and ${tag}-demo points to the demo image. That’s a breaking change for any consumers currently pulling ${tag} expecting the demo image (as per the previous workflow). Consider publishing BOTH tags for one transition period (e.g., also push ${tag}-base and/or also push ${tag} for demo), or explicitly document/update downstream references in the repo in the same PR.

Copilot uses AI. Check for mistakes.
2 changes: 1 addition & 1 deletion Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,4 +11,4 @@ RUN chmod u+x install_postgres_json_schema_extension.sh
RUN ./install_postgres_json_schema_extension.sh

FROM base AS demo
COPY ["database scripts/demo_db.sql", "/docker-entrypoint-initdb.d/"]
COPY ["scripts/demo_db.sql", "/docker-entrypoint-initdb.d/"]
2 changes: 1 addition & 1 deletion concatenate_files.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,5 @@

mkdir output
cd database\ scripts
cat 00_dump.sql 0[2345]_*.sql demo_db.sql > ../output/fullDemoDatabase.sql
cat 00_dump.sql 0[2345]_*.sql ../scripts/demo_db.sql > ../output/fullDemoDatabase.sql
cat 00_dump.sql 0[2345]_*.sql > ../output/EmptyDatabase.sql
347 changes: 9 additions & 338 deletions database scripts/00_dump.sql
View file
Open in desktop

Large diffs are not rendered by default.

5,037 changes: 0 additions & 5,037 deletions database scripts/demo_db.sql
View file
Open in desktop

This file was deleted.

1 change: 0 additions & 1 deletion migtool/tables.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,6 @@ tblHealthStatus
tblIdentificationTypes
tblIMISDefaults
tblIMISDefaultsPhone
tblLogins
tblOfficerVillages
tblPayerType
tblPayment
Expand Down
446 changes: 0 additions & 446 deletions scripts/demo_db.sql
View file
Open in desktop

Large diffs are not rendered by default.

Loading