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

Lobby config: direct 2.5 users to 'prod.triplea-game.org' #11194

Merged
merged 1 commit into from
Nov 6, 2022

Conversation

DanVanAtta
Copy link
Member

@DanVanAtta DanVanAtta commented Nov 6, 2022

Only the 2.5 game engine and below reads 'servers.yml'. This update directs those game clients to 'prod.triplea-game.org'. We have NGINX running on that URL which will in turn redirect the game client to an appropriate lobby server (NGINX going forward will control which game client versions connect to which lobby).

Change Summary & Additional Notes

Merging this would have immediate effect for any 2.5 users logging into the lobby.

Release Note

Only the 2.5 game engine and below reads 'servers.yml'. This update
directs those game clients to 'prod.triplea-game.org'. We have NGINX
running on that URL which will in turn redirect the game client
to an appropriate lobby server (NGINX going forward will control
which game client versions connect to which lobby).
@DanVanAtta
Copy link
Member Author

There are some additional 2.6 bots coming online (I turned them off in the 2.5 lobby).
We can probably merge this somewhat sooner when:

  • confirmed that we have at least 12 2.6 bots online (maybe we want more?)
  • announced on forums

After merge, we can essentially turn off the 2.5 lobby entirely (keeping in mind we will be disconnecting any users and preventing them from rejoining bot-games. Perhaps best down when the lobby is at 'low-tide' in terms of activity).

@codecov
Copy link

codecov bot commented Nov 6, 2022

Codecov Report

Merging #11194 (69c1529) into master (76e809d) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #11194   +/-   ##
=========================================
  Coverage     27.22%   27.22%           
  Complexity     8250     8250           
=========================================
  Files          1217     1217           
  Lines         78077    78077           
  Branches      10615    10615           
=========================================
+ Hits          21253    21255    +2     
+ Misses        54856    54854    -2     
  Partials       1968     1968           
Impacted Files Coverage Δ
...lea/ai/flowfield/influence/InfluenceTerritory.java 96.96% <0.00%> (-3.04%) ⬇️
...rc/main/java/games/strategy/net/nio/NioWriter.java 81.25% <0.00%> (-2.09%) ⬇️
.../main/java/games/strategy/net/ClientMessenger.java 64.58% <0.00%> (+2.08%) ⬆️
.../src/main/java/games/strategy/net/nio/Decoder.java 79.31% <0.00%> (+5.17%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@DanVanAtta DanVanAtta merged commit 970f02c into master Nov 6, 2022
@DanVanAtta DanVanAtta deleted the lobby-config/migrate-2-5-users branch November 6, 2022 07:55
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