[
https://issues.apache.org/jira/browse/MESOS-6122?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15468272#comment-15468272
]
Jie Yu commented on MESOS-6122:
-------------------------------
commit 123b9e57584f9b1c11c6a2f9668d8ccb5d5c37d1
Author: Jie Yu <[email protected]>
Date: Tue Sep 6 12:16:57 2016 -0700
Performed systemd::exist() check if systemd support is enabled.
The flags should come first before the systemd::exists() check runs.
Currently the systemd.exists() always runs and there's no way to
disable that check from running in mesos-slave.
Review: https://reviews.apache.org/r/51627/
> Mesos slave throws systemd errors even when passed a flag to disable systemd
> ----------------------------------------------------------------------------
>
> Key: MESOS-6122
> URL: https://issues.apache.org/jira/browse/MESOS-6122
> Project: Mesos
> Issue Type: Bug
> Components: slave
> Affects Versions: 1.0.1
> Reporter: Gennady Feldman
> Assignee: Jie Yu
> Fix For: 1.1.0, 1.0.2
>
>
> Seems like the code in slave/main.cpp is logically in the wrong order:
> #ifdef __linux__
> // Initialize systemd if it exists.
> if (systemd::exists() && flags.systemd_enable_support) {
> Lines 339-341:
> https://github.com/apache/mesos/blob/master/src/slave/main.cpp#L341
> The flags should come first before the systemd::exists() check runs.Currently
> the systemd.exists() always runs and there's no way to disable that check
> from running in mesos-slave.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
