You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@aurora.apache.org by "Kevin Sweeney (JIRA)" <ji...@apache.org> on 2015/05/27 23:47:34 UTC
[jira] [Issue Comment Deleted] (AURORA-1338) Remove
ChainedPathDetector from thermos
[ https://issues.apache.org/jira/browse/AURORA-1338?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kevin Sweeney updated AURORA-1338:
----------------------------------
Comment: was deleted
(was: [~benley] I feel like that use case is better satisfied by the Vagrant environment - you can already spin up a small mesos cluster to test with and launch tasks with the full Aurora config syntax available, see how all the mesos isolator modules interact with your task, etc. Using a standalone thermos you are restricted to a subset of those features.
There's also a lot of feature duplication between mesos - for example the CLI acts as an admin control interface to the executor, but in mesos this is not really needed - you can use an admin endpoint on the slave to kill tasks.
Similarly there's no reason to fsync checkpoints to disk if the observer is hosted in the executor itself (and with the GC executor replaced by mesos task reconciliation).
PID namespaces obviate the logic that requires thermos to be really sure it's killing a process it owns - as an executor in mesos it owns every process.
Disk isolation and resource accounting are also provided by Mesos now (though per-process accounting is still useful).
Thermos had these features long before Mesos did, but IMO now that the core supports them we should take advantage of them and reduce the amount of code we maintain.)
> Remove ChainedPathDetector from thermos
> ---------------------------------------
>
> Key: AURORA-1338
> URL: https://issues.apache.org/jira/browse/AURORA-1338
> Project: Aurora
> Issue Type: Story
> Components: Executor
> Reporter: Maxim Khutornenko
>
> Once the default checkpoint path moves into sandbox (AURORA-1337), remove the ChainedPathDetector (1) and make sure FixedPathDetector (2) is not used in Aurora.
> (1) - https://github.com/apache/aurora/blob/8fc322940d58034d5603159e9dbd500c8cb9a07f/src/main/python/apache/thermos/monitoring/detector.py#L49
> (2) - https://github.com/apache/aurora/blob/8fc322940d58034d5603159e9dbd500c8cb9a07f/src/main/python/apache/thermos/monitoring/detector.py#L39
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)