[PATCH compass-ci] container: delete invalid start depends fluentd
fluentd renamed master-fluentd, but scheduler and extract-stats have no dependency with master-fluentd now, so remove it. Signed-off-by: Liu Yinsi <liuyinsi@163.com> --- container/extract-stats/start-depends | 1 - container/scheduler/start-depends | 1 - 2 files changed, 2 deletions(-) diff --git a/container/extract-stats/start-depends b/container/extract-stats/start-depends index 4a016e6..efbe5a2 100755 --- a/container/extract-stats/start-depends +++ b/container/extract-stats/start-depends @@ -1,2 +1 @@ taskqueue -fluentd diff --git a/container/scheduler/start-depends b/container/scheduler/start-depends index 3e28dc1..4a17b46 100755 --- a/container/scheduler/start-depends +++ b/container/scheduler/start-depends @@ -1,4 +1,3 @@ redis -fluentd es taskqueue -- 2.23.0
On Tue, Nov 03, 2020 at 11:41:25AM +0800, Liu Yinsi wrote:
fluentd renamed master-fluentd, but scheduler and extract-stats have no dependency with master-fluentd now, so remove it.
fluentd just renamed master-fluentd, and why have no dependency with master-fluentd? Thanks, Zhangyu
Signed-off-by: Liu Yinsi <liuyinsi@163.com> --- container/extract-stats/start-depends | 1 - container/scheduler/start-depends | 1 - 2 files changed, 2 deletions(-)
diff --git a/container/extract-stats/start-depends b/container/extract-stats/start-depends index 4a016e6..efbe5a2 100755 --- a/container/extract-stats/start-depends +++ b/container/extract-stats/start-depends @@ -1,2 +1 @@ taskqueue -fluentd diff --git a/container/scheduler/start-depends b/container/scheduler/start-depends index 3e28dc1..4a17b46 100755 --- a/container/scheduler/start-depends +++ b/container/scheduler/start-depends @@ -1,4 +1,3 @@ redis -fluentd es taskqueue -- 2.23.0
On Tue, Nov 03, 2020 at 12:02:56PM +0800, Zhang Yu wrote:
On Tue, Nov 03, 2020 at 11:41:25AM +0800, Liu Yinsi wrote:
fluentd renamed master-fluentd, but scheduler and extract-stats have no dependency with master-fluentd now, so remove it.
fluentd just renamed master-fluentd, and why have no dependency with master-fluentd?
log-driver is fluentd has been config in /etc/docker/daemon.json. works for all containers, so have no dependency, except some no need log-driver fluentd, it can add '--log-driver json-file' in contaiener start script. Thanks, Yinsi
Thanks, Zhangyu
Signed-off-by: Liu Yinsi <liuyinsi@163.com> --- container/extract-stats/start-depends | 1 - container/scheduler/start-depends | 1 - 2 files changed, 2 deletions(-)
diff --git a/container/extract-stats/start-depends b/container/extract-stats/start-depends index 4a016e6..efbe5a2 100755 --- a/container/extract-stats/start-depends +++ b/container/extract-stats/start-depends @@ -1,2 +1 @@ taskqueue -fluentd diff --git a/container/scheduler/start-depends b/container/scheduler/start-depends index 3e28dc1..4a17b46 100755 --- a/container/scheduler/start-depends +++ b/container/scheduler/start-depends @@ -1,4 +1,3 @@ redis -fluentd es taskqueue -- 2.23.0
participants (2)
-
Liu Yinsi -
Zhang Yu