Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

logrotate #3541

Merged
merged 21 commits into from
Dec 27, 2021
Merged

logrotate #3541

merged 21 commits into from
Dec 27, 2021

Conversation

jiayuehua
Copy link
Contributor

@jiayuehua jiayuehua commented Dec 23, 2021

What type of PR is this?

  • bug
  • [*] feature
  • enhancement

What does this PR do?

logrotate rotate log

Which issue(s)/PR(s) this PR relates to?

close #3066

Special notes for your reviewer, ex. impact of this fix, etc:

do not merge util thirdparty 3.0 is merged.

Additional context/ Design document:

Checklist:

  • Documentation affected (Please add the label if documentation needs to be modified.)
  • Incompatibility (If it breaks the compatibility, please describe it and add the corresponding label.)
  • If it's needed to cherry-pick (If cherry-pick to some branches is required, please label the destination version(s).)
  • Performance impacted: Consumes more CPU/Memory

Release notes:

Please confirm whether to be reflected in release notes and how to describe:

                                                            `

if (!FileUtils::makeDir(FLAGS_log_dir)) {
return Status::Error("Failed to create log directory `%s'", FLAGS_log_dir.c_str());
}
if (!FileUtils::exist(FLAGS_log_dir) and !FileUtils::makeDir(FLAGS_log_dir)) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and => &&

@@ -0,0 +1,16 @@
/* Copyright (c) 2018 vesoft inc. All rights reserved.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2021

@Sophie-Xie Sophie-Xie added the ready-for-testing PR: ready for the CI test label Dec 27, 2021
@Sophie-Xie Sophie-Xie removed the request for review from panda-sheep December 27, 2021 09:40
@yixinglu yixinglu requested a review from critical27 December 27, 2021 09:41
@@ -20,6 +20,8 @@
--stderr_log_file=metad-stderr.log
# Copy log messages at or above this level to stderr in addition to logfiles. The numbers of severity levels INFO, WARNING, ERROR, and FATAL are 0, 1, 2, and 3, respectively.
--stderrthreshold=2
# wether logging files' name contain time stamp, If Using logrotate to rotate logging files, than should set it to true.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo whether

Copy link
Contributor

@critical27 critical27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally LGTM, a typo.

@yixinglu yixinglu merged commit f42afac into vesoft-inc:master Dec 27, 2021
@jiayuehua jiayuehua deleted the logrotate branch December 28, 2021 02:13
@Sophie-Xie Sophie-Xie added the doc affected PR: improvements or additions to documentation label Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc affected PR: improvements or additions to documentation ready for review ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Log rotation mechanism
4 participants