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

Removed RELEASE workflow input on GitHub Actions. #449

Merged
merged 10 commits into from
Apr 9, 2024
Merged

Conversation

samoht9277
Copy link
Contributor

This PR aims to facilitate the deploying sequence on each individual server.

By removing the abiity to choose what release to deploy, there is no possible confusion while deploying a certain release, since this PR no longer supports the ability to choose the deploy.

Arena    Server        -> Only arena
Loadtest Server        -> Only arena
Loadtest Client        -> Loadest and/or bots

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By removing this, we no longer have a way of confirming if the release we are deploying is the same as the already deployed (if any).

Since we no longer have the option to choose which release we want, these lines are no longer in need.

However, the loadtest client servers still have the choice to select between loadtest and bots manager. In this case, the script would generate two different services that run concurrently.

Nico-Sanchez
Nico-Sanchez previously approved these changes Apr 8, 2024
lotuuu
lotuuu previously approved these changes Apr 9, 2024
Copy link
Contributor

@lotuuu lotuuu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good. As we talked, deploy is failing because of server issues and not because of the code modifications

lotuuu and others added 4 commits April 9, 2024 12:47
* Rename game_backend release to main_server

* Make arena_load_test release only arena_load_test app

* Remove comment

* Rename main_server to central_backend

* Restore unintended changes

---------

Co-authored-by: Nicolas Sanchez <[email protected]>
Co-authored-by: Tomás Casagrande <[email protected]>
@lotuuu lotuuu merged commit dec34cc into main Apr 9, 2024
7 checks passed
@lotuuu lotuuu deleted the change-release-mode branch April 9, 2024 18:59
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.

3 participants