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

Dev Build: add servers to docker compose, automatically start services with gradlew run #12137

Merged
merged 2 commits into from
Nov 28, 2023

Conversation

DanVanAtta
Copy link
Member

@DanVanAtta DanVanAtta commented Nov 28, 2023

After this update, ./gradlew run will automatically invoke composeUp to start all services. The locally running game client can then connect to the locally running lobby at "http://locahost" (port 80).

This will send lobby login request to the locally running nginx which will then route the request to the lobby. This usage with nginx mirrors the production setup, and also simplifies localy configuration. Instead of having to configure the URL of every local service, we can configure that once in the nginx config and send all requests to the same 'locahost:80'

Notes to Reviewer

Still need to update developer documentation. Planning to do that somewhat shortly - probably not as part of this PR, but soon 🤞

…s with gradlew run

After this update, `./gradlew run` will automatically invoke composeUp to start
all services. The locally running game client can then connect to the locally
running lobby at "http://locahost"  (port 80).

This will send lobby login request to the locally running nginx which will
then route the request to the lobby. This usage with nginx mirrors the
production setup, and also simplifies localy configuration. Instead of
having to configure the URL of every local service, we can configure that
once in the nginx config and send all requests to the same 'locahost:80'
@DanVanAtta DanVanAtta merged commit 243a68f into master Nov 28, 2023
1 check passed
@DanVanAtta DanVanAtta deleted the gradle-compose-up branch November 28, 2023 05:02
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.

None yet

1 participant