Skip to content

fix(test): correct dynamic config path in docker-compose#137

Open
abhishekj720 wants to merge 1 commit into
mainfrom
fix/docker-compose-dynamic-config-path
Open

fix(test): correct dynamic config path in docker-compose#137
abhishekj720 wants to merge 1 commit into
mainfrom
fix/docker-compose-dynamic-config-path

Conversation

@abhishekj720

@abhishekj720 abhishekj720 commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Summary

The ubercadence/server image places dynamic config files under /etc/cadence/dynamicconfig/, but the docker-compose DYNAMIC_CONFIG_FILE_PATH was set to config/dynamicconfig/development.yaml. That path does not exist in the image so the config file was silently ignored meaning worker.enableScheduler: true never loaded and all schedule API calls timed out with DEADLINE_EXCEEDED.

Fix: set the path to dynamicconfig/development.yaml, which resolves correctly to /etc/cadence/dynamicconfig/development.yaml inside the container.

Also removes the now-redundant CADENCE_OVERRIDE_DYNAMIC_CONFIG=system.enableSchedules:true line, the bundled development.yaml already sets worker.enableScheduler: true.

How it was found

Running samples/schedule_sample.py demo against the Docker stack: create_schedule succeeded but describe_schedule always returned DEADLINE_EXCEEDED. Confirmed by running find inside the container the file was at dynamicconfig/development.yaml, not config/dynamicconfig/development.yaml. After the fix, the full demo ran end-to-end successfully.

Test plan

  • docker compose up -duv run python samples/schedule_sample.py demo completes all steps without error

The ubercadence/server image places dynamic config under
/etc/cadence/dynamicconfig/, so DYNAMIC_CONFIG_FILE_PATH must be
dynamicconfig/development.yaml — not config/dynamicconfig/development.yaml.

The wrong path caused the config file (including worker.enableScheduler=true)
to silently not load, making all schedule API calls time out with
DEADLINE_EXCEEDED. Verified by running the full schedule_sample demo
locally after the fix.

Signed-off-by: abhishek.jha <abhishek.jha@uber.com>
@gitar-bot

gitar-bot Bot commented Jun 14, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Updates the docker-compose dynamic config path to align with the server image structure, resolving scheduler timeout issues. Redundant environment overrides were removed as the updated configuration file now correctly enables the scheduler.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@codecov

codecov Bot commented Jun 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant