Skip to content
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.

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

Adds "HASURA_API_URL" to docker-compose #1623

Merged
merged 1 commit into from
Jan 2, 2025

Conversation

Mythicaeda
Copy link
Contributor

Description

Explicitly adds the HASURA_API_URL to our docker-composes. The value used is the default value, so there is no behavior change.

This change should mitigate the usability issue brought up in #1619, where it wasn't clear that GQL_API_URL and HASURA_API_URL were separate envvars that both need to be configured.

Verification

No verification was done, as there is no behavior change, just explicitly setting an envvar to its default value.

Documentation

Improved the wording of this envvar in the Gateway docs.

@Mythicaeda Mythicaeda added the deployment Anything related to dev ops and deployment label Jan 2, 2025
@Mythicaeda Mythicaeda requested a review from dandelany January 2, 2025 20:54
@Mythicaeda Mythicaeda self-assigned this Jan 2, 2025
@Mythicaeda Mythicaeda requested a review from a team as a code owner January 2, 2025 20:54
@Mythicaeda Mythicaeda requested a review from mattdailis January 2, 2025 20:54
@Mythicaeda Mythicaeda merged commit bd52e5e into develop Jan 2, 2025
14 checks passed
@Mythicaeda Mythicaeda deleted the refactor/explicitly-add-gateway-envvar branch January 2, 2025 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployment Anything related to dev ops and deployment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

docker-compose.yml is not consistent with aerie-gateway
2 participants