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

Disable mp syncreports and other minor server cleanups #15859

Merged
merged 4 commits into from Dec 15, 2018

Conversation

Projects
None yet
3 participants
@pchote
Copy link
Member

pchote commented Nov 25, 2018

  • Fixes #15790.
  • SyncReport generation is now profiled using its own line on the graph
  • Removed the IgnoreVersionMismatch setting, which appears to have been broken years ago (the setting is not propagated to the GlobalSettings) and stopped making sense when the server started doing things with the mod rules server-side.
  • Added newer server settings to the dedicated launch scripts.

pchote added some commits Nov 25, 2018

Disable MP sync report generation by default.
A new Server.EnableSyncReports option is provided
so that server operators can restore them remotely
in the event of a future desync bug.

@pchote pchote added this to the Next Release milestone Nov 25, 2018

@pchote

This comment has been minimized.

Copy link
Member

pchote commented Nov 25, 2018

Ping @IceReaper, who was recently asking about this.

@reaperrr
Copy link
Contributor

reaperrr left a comment

Can't say much about the script changes (though they look fine to me), the rest 👍

@chrisforbes chrisforbes self-requested a review Dec 15, 2018

@reaperrr reaperrr merged commit 0435920 into OpenRA:bleed Dec 15, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment