Update chronos-framework wrapper with current marathon-framework fixes

mesosphere / chronos-pkg

Apache License 2.0

5 stars 16 forks source link

Closed lloesche closed 9 years ago

lloesche commented 9 years ago

This PR adds the fixes we've applied to the marathon wrapper to the chronos wrapper which at one point shared the same codebase.

Namely:

being able to override JVM arguments by setting JAVA_OPTS in the environment
only calling ulimit -n if run as root and current limit is less than 8192
on systemd systems allow setting environment variables via /etc/sysconfig/chronos (RedHat recommended standard)

j1n6 commented 9 years ago

Nice, that's what I need for tuning log4j.

lingmann commented 9 years ago

Looks good to me, thanks @lloesche !