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

Mods do not get removed from game config when GAME_MODS_IDS_LIST is empty #31

Closed
hafometh88 opened this issue Nov 28, 2023 · 4 comments
Closed
Labels
bug Something isn't working

Comments

@hafometh88
Copy link
Contributor

hafometh88 commented Nov 28, 2023

Describe the bug
I am not sure if it this a bug or a feature request, but mod IDs do not get removed from the server config when I leave GAME_MODS_IDS_LIST empty

To Reproduce
Steps to reproduce the behavior:

  1. Add mod IDs to GAME_MODS_IDS_LIST and start the server with mods
  2. Remove IDs from GAME_MODS_IDS_LIST and update the stack
  3. Game still starts with mods, because the mod IDs are still present in the server config

Expected behavior
I expect the server to start with no mods when GAME_MODS_IDS_LIST is empty

@sdsznsk
Copy link
Contributor

sdsznsk commented Dec 1, 2023

https://github.com/acemod/docker-reforger#configs

Configs

By default the configs are generated from the ENV variables in the dockerfile. After the first run the file can be expanded with additional options manually, but the fields will always be overwritten by the ENV variables.

Alternatively, change the ARMA_CONFIG variable to a file present in the Configs volume. It will be used without modification.

This one is by design according to the readme (at least if I interpret it correctly).
Sure, I could change that but I will leave the decision to the folks if they actually want it.

@hafometh88
Copy link
Contributor Author

I think the ENV vars should be the main source of truth. If people want to primarily modify the server config itself, there is the feature to change ARMA_CONFIG to a different config file.

@BrettMayson BrettMayson added the bug Something isn't working label Dec 7, 2023
@hafometh88
Copy link
Contributor Author

hafometh88 commented Dec 11, 2023

I will leave the decision to the folks if they actually want it.

@sdsznsk So what do the folks say? :)

@hafometh88
Copy link
Contributor Author

docker-reforger/launch.py

Lines 121 to 123 in 32a30c9

if env_defined("GAME_MODS_IDS_LIST") or env_defined("GAME_MODS_JSON_FILE_PATH"):
# Since we want to keep ENVs as a single source of truth
# we will regenerate the mod list in case any manual changes were made

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants