Skip to content

Commit

Permalink
Fix bug w/ diff between ngen and cal images.
Browse files Browse the repository at this point in the history
Fixing issue where main ngen worker and calibration images would not
necessarily be built using the same ngen branch or commit due to the
Docker compose build config not applying all ngen .env-sourced ARGS to
the calibration image build.
  • Loading branch information
robertbartel committed Sep 18, 2023
1 parent aa69298 commit f1050f1
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions docker/main/docker-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -103,6 +103,8 @@ services:
target: ngen_cal_worker
args:
REPO_URL: ${NGEN_REPO_URL?No NGen repo url configured}
BRANCH: ${NGEN_BRANCH?No NGen branch configured}
COMMIT: ${NGEN_COMMIT}
NGEN_CAL_BRANCH: ${NGEN_CAL_BRANCH:-master}
NGEN_CAL_COMMIT: ${NGEN_CAL_COMMIT}
TROUTE_REPO_URL: ${TROUTE_REPO_URL:-https://github.com/NOAA-OWP/t-route}
Expand Down

0 comments on commit f1050f1

Please sign in to comment.