starred/LinuxGSM

Fork 0

mirror of https://github.com/GameServerManagers/LinuxGSM.git synced 2026-04-25 06:05:57 +03:00

[GH-ISSUE #3248] [BUG] On CentOS Linux 8, LinuxGSM does should install hostname package dependency. #2255

New issue

Closed

opened 2026-02-27 03:01:46 +03:00 by kerem · 9 comments

kerem commented

2026-02-27 03:01:46 +03:00

Owner

Originally created by @yeenbean on GitHub (Feb 14, 2021).
Original GitHub issue: https://github.com/GameServerManagers/LinuxGSM/issues/3248

User Story

As a [not sure what to put here], I want hostname package included as a dependency so that $HOSTNAME environment variable is filled properly, allowing Discord alerts to function when this package isn't found on the system.

Basic info

Distro: CentOS Linux 8 running in LXC container
Game: Issue seen in Minecraft and Killing Floor 2 servers.
Command: ./mcserver test-alert and ./kf2server test-alert
LinuxGSM version: [v20.1.3]

Further Information

After initial configuration of LGSM and getting my SkyBlock server up and running, I configured a Discord webhook to receive server alerts. When testing alerts using ./mcserver ta I get the following error:

[ FAIL ] Sending Alert mcserver: Sending Discord alert: {"embeds": ["0"]}

After doing some research, I learn that this error is returned from Discord when improper JSON information is found in an embed. I did some digging in developer mode to see if I could resolve the issue. tl;dr: I discover the HOSTNAME environment variable was not being set by my system. So I yum install hostname, log out, and log back in to get the environment variable set correctly, which then gets Discord alerts working smoothly.

To Reproduce

Steps to reproduce the behaviour:

Create LXC container using CentOS Linux 8 image
Install wget and sudo packages
Create user account for LGSM and add to sudoers file
Log out of root, login to new user account, and install LGSM
LGSM installs deps automatically
Configure server
Start server
Test alerts; Discord alert fails

Expected behaviour

All steps above, but with Discord alerts working.

Originally created by @yeenbean on GitHub (Feb 14, 2021). Original GitHub issue: https://github.com/GameServerManagers/LinuxGSM/issues/3248 ## User Story As a [not sure what to put here], I want `hostname` package included as a dependency so that `$HOSTNAME` environment variable is filled properly, allowing Discord alerts to function when this package isn't found on the system. ## Basic info * **Distro:** CentOS Linux 8 running in LXC container * **Game:** Issue seen in Minecraft and Killing Floor 2 servers. * **Command:** `./mcserver test-alert` and `./kf2server test-alert` * **LinuxGSM version:** [v20.1.3] ## Further Information After initial configuration of LGSM and getting my SkyBlock server up and running, I configured a Discord webhook to receive server alerts. When testing alerts using `./mcserver ta` I get the following error: ``` [ FAIL ] Sending Alert mcserver: Sending Discord alert: {"embeds": ["0"]} ``` After doing some research, I learn that this error is returned from Discord when improper JSON information is found in an embed. I did some digging in developer mode to see if I could resolve the issue. tl;dr: I discover the `HOSTNAME` environment variable was not being set by my system. So I `yum install hostname`, log out, and log back in to get the environment variable set correctly, which then gets Discord alerts working smoothly. ## To Reproduce Steps to reproduce the behaviour: 1. Create LXC container using CentOS Linux 8 image 2. Install `wget` and `sudo` packages 3. Create user account for LGSM and add to sudoers file 4. Log out of root, login to new user account, and install LGSM 5. LGSM installs deps automatically 6. Configure server 7. Start server 8. Test alerts; Discord alert fails ## Expected behaviour All steps above, but with Discord alerts working.

kerem

2026-02-27 03:01:46 +03:00

closed this issue
added the
type: bug

info: alerts

info: dependency
labels

kerem commented

2026-02-27 03:01:47 +03:00

Author

Owner

@dgibbs64 commented on GitHub (Feb 14, 2021):

I am assuming that the LXC image is a cut down version of the full centos 8?

@dgibbs64 commented on GitHub (Feb 14, 2021): I am assuming that the LXC image is a cut down version of the full centos 8?

kerem commented

2026-02-27 03:01:47 +03:00

Author

Owner

@yeenbean commented on GitHub (Feb 14, 2021):

Yes I believe so.

@yeenbean commented on GitHub (Feb 14, 2021): Yes I believe so.

kerem commented

2026-02-27 03:01:47 +03:00

Author

Owner

@dgibbs64 commented on GitHub (Feb 15, 2021):

This is a tough one as hostname is a very basic feature of Linux systems so I am surprised it isn't installed by default

@dgibbs64 commented on GitHub (Feb 15, 2021): This is a tough one as hostname is a very basic feature of Linux systems so I am surprised it isn't installed by default

kerem commented

2026-02-27 03:01:47 +03:00

Author

Owner

@h3o66 commented on GitHub (Feb 17, 2021):

Is there a easy way to detect the container ?
We could maybe add a detection for LXC and add then dynamically the dependency

Edit: maybe just we could check if systemd-detect-virt outputs lxc and then add the dependency for this. Maybe you can test what this does output in the container

@h3o66 commented on GitHub (Feb 17, 2021): Is there a easy way to detect the container ? We could maybe add a detection for LXC and add then dynamically the dependency Edit: maybe just we could check if `systemd-detect-virt` outputs lxc and then add the dependency for this. Maybe you can test what this does output in the container

kerem commented

2026-02-27 03:01:47 +03:00

Author

Owner

@yeenbean commented on GitHub (Feb 17, 2021):

Output of systemd-detect-virt:

[mcserver@mc-paper ~]$ systemd-detect-virt
lxc

This was only tested in the CentOS container I currently have up, but I have the resources to test in the other supported OSes for this project if needed. Will do this later in the evening. It seems like a viable option though.

@yeenbean commented on GitHub (Feb 17, 2021): Output of `systemd-detect-virt`: ``` [mcserver@mc-paper ~]$ systemd-detect-virt lxc ``` This was only tested in the CentOS container I currently have up, but I have the resources to test in the other supported OSes for this project if needed. Will do this later in the evening. It seems like a viable option though.

kerem commented

2026-02-27 03:01:47 +03:00

Author

Owner

@h3o66 commented on GitHub (Feb 17, 2021):

I guess with this a hotfix coule be done, as this would be easy to implement.

@h3o66 commented on GitHub (Feb 17, 2021): I guess with this a hotfix coule be done, as this would be easy to implement.

kerem commented

2026-02-27 03:01:47 +03:00

Author

Owner

@h3o66 commented on GitHub (Mar 7, 2021):

afaik there seem to be 2 values for lxc containers for systemd-detect-virt in this case: lxc and lxc-libvirt

All types can be looked up with systemd-detect-virt --list

@h3o66 commented on GitHub (Mar 7, 2021): afaik there seem to be 2 values for lxc containers for `systemd-detect-virt` in this case: `lxc` and `lxc-libvirt` All types can be looked up with `systemd-detect-virt --list`

kerem commented

2026-02-27 03:01:47 +03:00

Author

Owner

@h3o66 commented on GitHub (Mar 7, 2021):

@yeenbean if you have time, you could test the connected PR.
Be aware that the PR branch is on the development state and contains more changes.

@h3o66 commented on GitHub (Mar 7, 2021): @yeenbean if you have time, you could test the connected PR. Be aware that the PR branch is on the development state and contains more changes.

kerem commented

2026-02-27 03:01:47 +03:00

Author

Owner

@github-actions[bot] commented on GitHub (Mar 15, 2022):

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions[bot] commented on GitHub (Mar 15, 2022): This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

kerem referenced this issue

2026-02-27 03:54:56 +03:00

[PR #2254] [MERGED] mtaserver url broken since 190401 update #3552