bitwalker / distillery

Simplify deployments in Elixir with OTP releases!
MIT License
2.97k stars 398 forks source link

Windows: Remove erlsrv `-debugType new` option from install to stop unbounded logs #622

Closed ellbee closed 5 years ago

ellbee commented 5 years ago

Summary of changes

We have had extremely large log files generated on deploys to Windows servers. It seems like it is caused by the -debugType new option.

From the erlsrv documentation:

The new and reuse options might seem convenient in a production system, but consider that the logs grow indefinitely during the system lifetime and cannot be truncated, except if the service is restarted.

In short, the DebugType is intended for debugging only. Logs during production are better produced with the standard Erlang logging facilities.

This pull request removes the -debugType flag entirely which has solved the problem for us. I am not sure what to do about the test section of the checklist, let me know if there is anything you require me to do here.

Checklist

Licensing/Copyright

By submitting this PR, you agree to the following statement, please read before submission!

I certify that I own, and have sufficient rights to contribute, all source code and related material intended to be compiled or integrated with the source code for Distillery (the "Contribution"). My Contribution is licensed under the MIT License.

NOTE: If you submit a PR and remove the statement above, your PR will be rejected. For your PR to be considered, it must contain your agreement to license under the MIT license.

bitwalker commented 5 years ago

Thanks!