Skip to content

Add support for DC_INSTRUCTIONS_DIR in Docker convenience script#6185

Merged
kmoscoe merged 102 commits intodatacommonsorg:masterfrom
kmoscoe:script
Apr 15, 2026
Merged

Add support for DC_INSTRUCTIONS_DIR in Docker convenience script#6185
kmoscoe merged 102 commits intodatacommonsorg:masterfrom
kmoscoe:script

Conversation

@kmoscoe
Copy link
Copy Markdown
Contributor

@kmoscoe kmoscoe commented Apr 7, 2026

This PR updates the run_cdc_dev_docker.sh script to enable both local and GCS paths for custom instructions

  • adds a new section in run_service() for running with custom instructions
  • refactors the run_service() function to construct Docker commands according to variable and flag settings (thanks to @clincoln8 )
  • refactors the run_data function in the same way
  • adds a new function to check if a service container is already running on port 8080 and kills it if so
  • fixes server paths to use FLASK_ENV (I'm not sure how the previous "CUSTOM_DIR" even worked!)
  • removes some details from the help

I've tested it running all types of run flags (release, latest, custom build) as well as different settings in the env.list file (local and remote gs paths).

kmoscoe and others added 30 commits April 6, 2025 11:18
This reverts commit 6c875ed.
@kmoscoe kmoscoe closed this Apr 8, 2026
@kmoscoe kmoscoe reopened this Apr 8, 2026
@kmoscoe kmoscoe changed the title Add support for DC_INSTRUCTIONS_DIR in website env file and Docker convenience script Add support for DC_INSTRUCTIONS_DIR in Docker convenience script Apr 8, 2026
@kmoscoe kmoscoe requested review from clincoln8 and keyurva and removed request for keyurva April 9, 2026 23:24
Copy link
Copy Markdown
Contributor

@clincoln8 clincoln8 left a comment

Choose a reason for hiding this comment

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

Thanks Kara! Just want to make sure that deletion is intentional.

Comment thread run_cdc_dev_docker.sh
Comment thread run_cdc_dev_docker.sh Outdated
Comment thread run_cdc_dev_docker.sh Outdated
Comment thread run_cdc_dev_docker.sh Outdated
Comment thread run_cdc_dev_docker.sh
@kmoscoe kmoscoe requested a review from clincoln8 April 15, 2026 18:29
Copy link
Copy Markdown
Contributor

@clincoln8 clincoln8 left a comment

Choose a reason for hiding this comment

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

Nice!

Comment thread run_cdc_dev_docker.sh Outdated
@kmoscoe kmoscoe merged commit 5112c97 into datacommonsorg:master Apr 15, 2026
13 checks passed
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.

3 participants