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

server: add --log-dir to specify logstore dir #873

Merged
merged 3 commits into from
Sep 17, 2021

Conversation

9547
Copy link
Contributor

@9547 9547 commented Mar 10, 2021

The default logstore path is under ioutil.TempFile. In some cases, we need to store the searched logs in a specific place.

@ti-chi-bot
Copy link
Member

ti-chi-bot commented Mar 10, 2021

[REVIEW NOTIFICATION]

This pull request has been approved by:

  • breeswish

To complete the pull request process, please ask the reviewers in the list to review by filling /cc @reviewer in the comment.
After your PR has acquired the required number of LGTMs, you can assign this pull request to the committer in the list by filling /assign @committer in the comment to help you merge this pull request.

The full list of commands accepted by this bot can be found here.

Reviewer can indicate their review by submitting an approval review.
Reviewer can cancel approval by submitting a request changes review.

Copy link
Member

@breezewish breezewish left a comment

Choose a reason for hiding this comment

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

Thanks a lot for the contribution! I have a few ideas about the name.

pkg/config/config.go Outdated Show resolved Hide resolved
pkg/config/config.go Outdated Show resolved Hide resolved
9547 added 2 commits March 23, 2021 22:15
@9547 9547 force-pushed the feature/specify-log-dir branch from 907d392 to c1dbbed Compare March 23, 2021 14:19
@9547
Copy link
Contributor Author

9547 commented Mar 23, 2021

@breeswish PTAL.

@breezewish breezewish force-pushed the master branch 2 times, most recently from b4a5d28 to ab56290 Compare July 9, 2021 10:30
Copy link
Member

@breezewish breezewish left a comment

Choose a reason for hiding this comment

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

Sorry for the late review! This PR looks good!

@breezewish
Copy link
Member

/merge

@ti-chi-bot
Copy link
Member

This pull request has been accepted and is ready to merge.

Commit hash: c1dbbed

@ti-chi-bot ti-chi-bot merged commit 49e2b75 into pingcap:master Sep 17, 2021
baurine pushed a commit to baurine/tidb-dashboard that referenced this pull request Sep 26, 2021
baurine added a commit that referenced this pull request Sep 26, 2021
* Support non root user login (#985)

Co-authored-by: Wenxuan <hi@breeswish.org>
Co-authored-by: crazycs <chen.two.cs@gmail.com>

* tweak(debugapi): pprof endpoint debug param (#1005)

* Tweak: component invalid address check (#994)

* tweak(client): add clients addr check & fix debugapi SSRF vulnerability

* chore: update comments

* fix: lints

* fix: proxy url at tidb/client

* feat(client): member cache

* chore: rename

* chore: remove unnecessary tidb proxy addr parse

* server: add --log-dir to specify logstore dir (#873)

* build(deps): bump tmpl from 1.0.4 to 1.0.5 in /ui (#1007)

* build(deps): bump tmpl from 1.0.4 to 1.0.5 in /ui/tests (#1008)

* support authorize non root user to sso login (#1003)

* fix: debug api pprof timeout & refine api model (#964)

* fix(stmt): min value of max stmt count size & blank detail page (#982)

* Refine non root login (#1009)

* make non-root-login default value false

* add tooltip to ask for upgrade tidb to support login with non root

* Update ui/dashboardApp/layout/translations/zh.yaml

Co-authored-by: Wenxuan <hi@breeswish.org>

* Update ui/dashboardApp/layout/translations/en.yaml

Co-authored-by: Wenxuan <hi@breeswish.org>

Co-authored-by: Wenxuan <hi@breeswish.org>

* add doc link for insufficient privileges error (#1012)

* update release-version

Co-authored-by: Wenxuan <hi@breeswish.org>
Co-authored-by: crazycs <chen.two.cs@gmail.com>
Co-authored-by: Suhaha <jklopsdfw@gmail.com>
Co-authored-by: 9547 <nivefive9547@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants