[PR #525] [CLOSED] External per-instance configuration files #3121

New issue

Closed

opened 2026-02-27 03:53:22 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 03:53:22 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/GameServerManagers/LinuxGSM/pull/525
Author: @jaredballou
Created: 11/3/2015
Status: ❌ Closed

Base: master ← Head: master

📝 Commits (10+)

790c90b Pulled upstream changes, added functions and cfg/servers in each game's directory to .gitignore
b6d63b1 Merged default/common/instance configs and GitHub fetch updates
6d90017 Updating Git fetcher
63595d5 Added JSON parser for pulling Git metadata and doing updates
8f23e8a Created addons tool as a proof of concept and test for Git fetching of addons for gameservers
b80d3fc Updated check_root to allow root execution for Docker. Updated insserver script with more over-complicated features nobody wants or needs.
903362b Added GitHub downloader/updater POC. It downloads a manifest of the hashes for all files for use in comparisons.
25130ff Updated insserver script to have headers and default values set from script in a more standardized way.
b44f930 Major rework to put subdirectories under lgsm directory, as a test. New games support function that has parameters set from hierarchially loaded gamedata files to stgart to move towards single-instance manager for all games
963eef3 Bugfixes, new game parser updates

📊 Changes

393 files changed (+16198 additions, -1019 deletions)

View changed files

📝 .gitignore (+5 -1)
📝 GameServerQuery/gsquery.py (+0 -0)
➖ Insurgency/insserver (+0 -139)
📝 README.md (+54 -1)
📝 cfg/ARKSurvivalEvolved/lgsm-default.ini (+0 -0)
📝 cfg/Arma3/lgsm-default.network.cfg (+0 -0)
📝 cfg/Arma3/lgsm-default.server.cfg (+0 -0)
📝 cfg/BlackMesa/lgsm-default.cfg (+0 -0)
📝 cfg/BladeSymphony/lgsm-default.cfg (+0 -0)
📝 cfg/BrainBread2/lgsm-default.cfg (+0 -0)
📝 cfg/CounterStrike/lgsm-default.cfg (+0 -0)
📝 cfg/CounterStrikeConditionZero/lgsm-default.cfg (+0 -0)
📝 cfg/CounterStrikeGlobalOffensive/lgsm-default.cfg (+0 -0)
📝 cfg/CounterStrikeSource/lgsm-default.cfg (+0 -0)
📝 cfg/DayOfDefeat/lgsm-default.cfg (+0 -0)
📝 cfg/DayOfDefeatSource/lgsm-default.cfg (+0 -0)
📝 cfg/DeathmatchClassic/lgsm-default.cfg (+0 -0)
📝 cfg/DoubleActionBoogaloo/lgsm-default.cfg (+0 -0)
📝 cfg/FistfulOfFrags/lgsm-default.cfg (+0 -0)
📝 cfg/GarrysMod/lgsm-default.cfg (+0 -0)

...and 80 more files

📄 Description

I created a version of insserver to support putting configs that allow overrides of script settings for each instance in a way that makes sense (at least to me). This allows users to simply update the script from GitHub or via download, and not have to back up or re-configure their settings. It also supports multiple instances a little better. I had all this in version 200914, I ported it over to the latest version and tested it with my servers. As far as I can see, it works, but this is my first time doing a PR so be aware that I am pretty new at this.

Added external config support, load files from cfg/servers directory, in this order:
- _default.cfg (which is managed by Git and should not be edited by end users)
- _common.cfg (which is the user's common settings for all servers)
- SERVICENAME.cfg for the individual instances.
TODO: If this PR is accepted, modify the default config creation to use a fetch from GitHub as the functions files are.
Cleaned up the function script calling, it no longer has to change directory when function files are downloaded, which annoyed me.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/GameServerManagers/LinuxGSM/pull/525 **Author:** [@jaredballou](https://github.com/jaredballou) **Created:** 11/3/2015 **Status:** ❌ Closed **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (10+) - [`790c90b`](https://github.com/GameServerManagers/LinuxGSM/commit/790c90be3917f42686442b136d05487bd181058a) Pulled upstream changes, added functions and cfg/servers in each game's directory to .gitignore - [`b6d63b1`](https://github.com/GameServerManagers/LinuxGSM/commit/b6d63b1a4c97c0936d49f9c92cee84af4e7737c8) Merged default/common/instance configs and GitHub fetch updates - [`6d90017`](https://github.com/GameServerManagers/LinuxGSM/commit/6d90017c85e46c6409208569ea0afa2fca5ab6a1) Updating Git fetcher - [`63595d5`](https://github.com/GameServerManagers/LinuxGSM/commit/63595d5e9b401bbea9b5a07750bf7f7fcd958c9e) Added JSON parser for pulling Git metadata and doing updates - [`8f23e8a`](https://github.com/GameServerManagers/LinuxGSM/commit/8f23e8a23e9422340ec605696cbe15e7444dffbc) Created addons tool as a proof of concept and test for Git fetching of addons for gameservers - [`b80d3fc`](https://github.com/GameServerManagers/LinuxGSM/commit/b80d3fc0cb9f6f464f1bd7bd36852bdd159b7bf3) Updated check_root to allow root execution for Docker. Updated insserver script with more over-complicated features nobody wants or needs. - [`903362b`](https://github.com/GameServerManagers/LinuxGSM/commit/903362b6a06c32c7cf9443dd1ab16884db17ba53) Added GitHub downloader/updater POC. It downloads a manifest of the hashes for all files for use in comparisons. - [`25130ff`](https://github.com/GameServerManagers/LinuxGSM/commit/25130ff22a066eaf7a5c6df492398db7d14b3a79) Updated insserver script to have headers and default values set from script in a more standardized way. - [`b44f930`](https://github.com/GameServerManagers/LinuxGSM/commit/b44f93085ace116f587db6e5c65e716b96853073) Major rework to put subdirectories under lgsm directory, as a test. New games support function that has parameters set from hierarchially loaded gamedata files to stgart to move towards single-instance manager for all games - [`963eef3`](https://github.com/GameServerManagers/LinuxGSM/commit/963eef36e321cb7f67d0557f2be251eeb158ebc9) Bugfixes, new game parser updates ### 📊 Changes **393 files changed** (+16198 additions, -1019 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+5 -1) 📝 `GameServerQuery/gsquery.py` (+0 -0) ➖ `Insurgency/insserver` (+0 -139) 📝 `README.md` (+54 -1) 📝 `cfg/ARKSurvivalEvolved/lgsm-default.ini` (+0 -0) 📝 `cfg/Arma3/lgsm-default.network.cfg` (+0 -0) 📝 `cfg/Arma3/lgsm-default.server.cfg` (+0 -0) 📝 `cfg/BlackMesa/lgsm-default.cfg` (+0 -0) 📝 `cfg/BladeSymphony/lgsm-default.cfg` (+0 -0) 📝 `cfg/BrainBread2/lgsm-default.cfg` (+0 -0) 📝 `cfg/CounterStrike/lgsm-default.cfg` (+0 -0) 📝 `cfg/CounterStrikeConditionZero/lgsm-default.cfg` (+0 -0) 📝 `cfg/CounterStrikeGlobalOffensive/lgsm-default.cfg` (+0 -0) 📝 `cfg/CounterStrikeSource/lgsm-default.cfg` (+0 -0) 📝 `cfg/DayOfDefeat/lgsm-default.cfg` (+0 -0) 📝 `cfg/DayOfDefeatSource/lgsm-default.cfg` (+0 -0) 📝 `cfg/DeathmatchClassic/lgsm-default.cfg` (+0 -0) 📝 `cfg/DoubleActionBoogaloo/lgsm-default.cfg` (+0 -0) 📝 `cfg/FistfulOfFrags/lgsm-default.cfg` (+0 -0) 📝 `cfg/GarrysMod/lgsm-default.cfg` (+0 -0) _...and 80 more files_ </details> ### 📄 Description I created a version of insserver to support putting configs that allow overrides of script settings for each instance in a way that makes sense (at least to me). This allows users to simply update the script from GitHub or via download, and not have to back up or re-configure their settings. It also supports multiple instances a little better. I had all this in version 200914, I ported it over to the latest version and tested it with my servers. As far as I can see, it works, but this is my first time doing a PR so be aware that I am pretty new at this. - Added external config support, load files from cfg/servers directory, in this order: - _default.cfg (which is managed by Git and should not be edited by end users) - _common.cfg (which is the user's common settings for all servers) - SERVICENAME.cfg for the individual instances. - TODO: If this PR is accepted, modify the default config creation to use a fetch from GitHub as the functions files are. - Cleaned up the function script calling, it no longer has to change directory when function files are downloaded, which annoyed me. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

Rows
Columns