From 95b2262cc3b0663afb96f4b00dcfcc45f574f722 Mon Sep 17 00:00:00 2001 From: Hornwitser Date: Tue, 4 May 2021 14:10:13 +0200 Subject: [PATCH] Remove log rotation Remove log rotation as Factorio implemented its own log rotation an eternety ago and the cluster log already contains all the log entries. --- CHANGELOG.md | 1 + packages/slave/slave.js | 14 -------------- 2 files changed, 1 insertion(+), 14 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 302db369..eba715c2 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -76,6 +76,7 @@ UPS statistics is exported by the statistics exporter plugin. - Master server now defaults to hosting on https on port 8443. - Renamed client to slave. +- Removed rotation of factorio-current.log files. ### Breaking Changes diff --git a/packages/slave/slave.js b/packages/slave/slave.js index 2ee280b2..aff5d472 100755 --- a/packages/slave/slave.js +++ b/packages/slave/slave.js @@ -384,20 +384,6 @@ class Instance extends libLink.Link { ); } - this.logger.verbose("Rotating old logs..."); - // clean old log file to avoid crash - try { - let logPath = this.path("factorio-current.log"); - let stat = await fs.stat(logPath); - if (stat.isFile()) { - let logFilename = `factorio-${Math.floor(Date.parse(stat.mtime)/1000)}.log`; - await fs.rename(logPath, this.path(logFilename)); - this.logger.verbose(`Log rotated as ${logFilename}`); - } - } catch (err) { - this.logger.error(`Error rotating logs:\n${err.stack}`); - } - // eslint-disable-next-line no-use-before-define await symlinkMods(this, "sharedMods"); }