Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

rely on docker to rotate logs for watchdog/job_exporter #1239

Merged
merged 1 commit into from
Sep 3, 2018
Merged

rely on docker to rotate logs for watchdog/job_exporter #1239

merged 1 commit into from
Sep 3, 2018

Conversation

xudifsd
Copy link
Member

@xudifsd xudifsd commented Aug 31, 2018

fix #1217

Also changed health check for job_exporter from healthy_check.py to no_older_than.py.

@xudifsd xudifsd requested a review from YanjieGao August 31, 2018 07:16
@coveralls
Copy link

Coverage Status

Coverage decreased (-11.9%) to 61.596% when pulling 88094a9 on dixu/do-not-rotate-ourself into 1e363c0 on master.

@xudifsd xudifsd changed the title replay on docker to rotate logs for watchdog/job_exporter rely on docker to rotate logs for watchdog/job_exporter Aug 31, 2018
@xudifsd xudifsd merged commit 5f6dd86 into master Sep 3, 2018
@xudifsd xudifsd deleted the dixu/do-not-rotate-ourself branch September 3, 2018 07:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@YanjieGao YanjieGao YanjieGao approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[exporter] exporter's log rotation refactor
3 participants
@xudifsd @coveralls @YanjieGao