Skip to content

fix(docker): add missing api-gw env vars for latest gateway image#1402

Merged
hkad98 merged 1 commit intomasterfrom
fix/docker-compose-missing-env-vars-master
Mar 11, 2026
Merged

fix(docker): add missing api-gw env vars for latest gateway image#1402
hkad98 merged 1 commit intomasterfrom
fix/docker-compose-missing-env-vars-master

Conversation

@tychtjan
Copy link
Contributor

@tychtjan tychtjan commented Mar 11, 2026

TL;DR: Without these changes, docker compose up fails — api-gw crashes on startup, blocking the entire bootstrap sequence.

Summary

  • Add GRPC_MDSINK_HOST/GRPC_MDSINK_PORT (gateway-md-sink:6574) to api-gw service — required by latest gateway-api-gw image for gRPC routing to md-sink
  • Add GEN_AI_HTTP_HOST/GEN_AI_HTTP_PORT (localhost:8989) to api-gw service — required env var, set to dummy value since gen-ai is not needed for SDK tests

Without these, api-gw crashes on startup with ApplicationConfigurationException: Required environment variable not found, which blocks metadata-organization-bootstrap and the entire bootstrap sequence.

  • Also regenerates VCR cassettes to match current API responses.

jira: STL-2167, GDAI-1452
risk: low

@tychtjan tychtjan force-pushed the fix/docker-compose-missing-env-vars-master branch from 1086e0c to 7bde891 Compare March 11, 2026 10:06
The latest gateway-api-gw image requires GRPC_MDSINK_HOST/PORT and
GEN_AI_HTTP_HOST/PORT environment variables. Without these, api-gw
crashes on startup, blocking the bootstrap sequence.

Also regenerates VCR cassettes to match current API responses.
@tychtjan tychtjan force-pushed the fix/docker-compose-missing-env-vars-master branch from 7bde891 to 09f2917 Compare March 11, 2026 10:06
@codecov
Copy link

codecov bot commented Mar 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.53%. Comparing base (c00ca3b) to head (09f2917).
⚠️ Report is 56 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1402       +/-   ##
===========================================
+ Coverage    0.00%   77.53%   +77.53%     
===========================================
  Files         158      225       +67     
  Lines       11048    14614     +3566     
===========================================
+ Hits            0    11331    +11331     
+ Misses      11048     3283     -7765     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@hkad98 hkad98 merged commit 46c0f4b into master Mar 11, 2026
12 checks passed
@hkad98 hkad98 deleted the fix/docker-compose-missing-env-vars-master branch March 11, 2026 11:36
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.

2 participants