Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

allow for garbage collection algo preference #40

Merged
merged 8 commits into from
Nov 2, 2022

Conversation

ryphon
Copy link
Contributor

@ryphon ryphon commented Sep 26, 2022

https://theindiestone.com/forums/index.php?/topic/47417-dedicated-server-on-linux-vps-jre-crashes/

Only way to keep my server stable with 10+ people is to set to G1GC, but there's other java jre gc options

Afaik there's only a few java GC's that'd matter. ZGC is usually best, but G1GC can sometimes be more efficient with it's RAM usage. Probably related to something dumb in either my vm configs or otherwise, but seems like an easy change to allow it to be configurable more generally.

Maybe just adds tech debt to potential future work to deal with #38 and #23 but idk, this is an easy fix for my specific problems.

Signed-off-by: Dylan Shepard dylan@shepard.dev

@Renegade-Master
Copy link
Owner

Hey!

Thanks for the contribution!
If you are adding new variables, would you mind adding them to the docker-compose.yaml and README.md, as well as the run_server.sh?

That way it is documented, and available via all deployment methods.

@ryphon
Copy link
Contributor Author

ryphon commented Oct 17, 2022

Updated with both fixed docs in run_server, as well as docker-compose and README updates @Renegade-Master

@Renegade-Master
Copy link
Owner

Renegade-Master commented Oct 17, 2022

Thank you for adding the extra bits.
Looks like the developers have changed the successful shutdown message though.

Would you mind changing the line in .github/workflows/docker-build.yml#L265 according to the following diff?

-          check_for_config "znet: Java_zombie_core_znet_SteamUtils_n_1Shutdown"
+          check_for_config "ZNet: SZombienet -> SSteamSDK: LogOff"

Should be good to go after that.

@ryphon
Copy link
Contributor Author

ryphon commented Oct 25, 2022

Actions updated, sorry for the delay on this one!

@Renegade-Master
Copy link
Owner

Thanks for updating that! I would do it myself, but I have too many other things to be doing at the moment 😅


And it looks like the new versions of Project Zomboid have changed some configuration things.

image

That Port was definitely set, so something must have changed about the ini file structure.

image

I will try to take a look at this tomorrow. It should just be a case of figuring out what the new name for the main Port is.

@Renegade-Master
Copy link
Owner

@ryphon
The required changes should be implemented in #43. When that is merged, we can rebase this PR, and hopefully get this merged in too.

https://theindiestone.com/forums/index.php?/topic/47417-dedicated-server-on-linux-vps-jre-crashes/

Only way to keep my server stable with 10+ people is to set to G1GC, but there's other java jre gc options

Afaik there's only a few java GC's that'd matter. ZGC is usually best, but G1GC can sometimes be more efficient with it's RAM usage. Probably related to something dumb in either my vm configs or otherwise, but seems like an easy change to allow it to be configurable more generally.

Maybe just adds tech debt to potential future work to deal with Renegade-Master#38 and Renegade-Master#23 but idk, this is an easy fix for my specific problems.

Signed-off-by: Dylan Shepard <dylan@shepard.dev>
Signed-off-by: Renegade-Master <renegade.master.dev@protonmail.com>
- .github/workflows/docker-build.yml
  - Add environment configuration for new GC Config option.
  - Add Test for new configuration option.

- README.md
  - Formatting pass.

- docker-compose.yaml
  - Formatting pass.
@ryphon
Copy link
Contributor Author

ryphon commented Nov 2, 2022

Hey I've been at Kubecon all week, anything you need from me here? Happy to rebase whatever.

@Renegade-Master
Copy link
Owner

I didn't realise there were two Kubecons per year. I went to the European one 6 months ago 😅

I just need to get this test working for the changed GC. It does the change, my syntax is just wrong for the validation. I couldn't get it last night.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 2, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Owner

@Renegade-Master Renegade-Master left a comment

Choose a reason for hiding this comment

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

Looks good to me anyway!

@Renegade-Master Renegade-Master merged commit bdb86eb into Renegade-Master:main Nov 2, 2022
@ryphon ryphon deleted the patch-1 branch November 2, 2022 23:21
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants