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

cluster: update dashboard metadata on reload #1003

Merged
merged 7 commits into from
Dec 28, 2020

Conversation

AstroProfundis
Copy link
Contributor

What problem does this PR solve?

The prometheus URL in PD is not updated on reload, thus dashboard is not using the correct prometheus for querying.

Currently, run reload with any node (even a non-exist node) could trigger the update and workaround this.

Check List

Tests

  • Manual test (add detailed scripts or steps below)

Related changes

  • Need to cherry-pick to the release branch

Release notes:

cluster: update dashboard metadata on reload

@AstroProfundis AstroProfundis added type/bug-fix Categorizes PR as a bug-fix category/usability Categorizes issue or PR as a usability enhancement. category/monitoring Categorizes issue or PR related to monitoring components. labels Dec 21, 2020
@AstroProfundis AstroProfundis self-assigned this Dec 21, 2020
@ti-chi-bot ti-chi-bot requested review from lonng and nrc December 21, 2020 09:56
@ti-chi-bot ti-chi-bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Dec 21, 2020
@codecov-io
Copy link

codecov-io commented Dec 21, 2020

Codecov Report

Merging #1003 (58e632d) into master (ac3f756) will decrease coverage by 3.71%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1003      +/-   ##
==========================================
- Coverage   55.62%   51.91%   -3.72%     
==========================================
  Files         280      280              
  Lines       19734    19741       +7     
==========================================
- Hits        10977    10248     -729     
- Misses       7048     7876     +828     
+ Partials     1709     1617      -92     
Flag Coverage Δ
cluster 38.45% <100.00%> (-5.04%) ⬇️
dm 23.96% <22.22%> (-0.08%) ⬇️
integrate 46.15% <100.00%> (-3.73%) ⬇️
playground 20.31% <ø> (ø)
tiup 16.48% <ø> (ø)
unittest 22.27% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pkg/cluster/manager/reload.go 77.55% <100.00%> (+3.74%) ⬆️
components/cluster/command/check.go 6.27% <0.00%> (-73.27%) ⬇️
pkg/cluster/task/limits.go 0.00% <0.00%> (-68.75%) ⬇️
pkg/cluster/task/sysctl.go 0.00% <0.00%> (-66.67%) ⬇️
components/cluster/command/audit.go 27.27% <0.00%> (-54.55%) ⬇️
pkg/cluster/operation/check.go 0.00% <0.00%> (-53.07%) ⬇️
pkg/cluster/manager/patch.go 0.00% <0.00%> (-51.81%) ⬇️
pkg/cluster/task/rmdir.go 0.00% <0.00%> (-50.00%) ⬇️
pkg/cluster/manager/cleanup.go 0.00% <0.00%> (-49.19%) ⬇️
pkg/cluster/operation/operation.go 34.78% <0.00%> (-43.48%) ⬇️
... and 32 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ac3f756...58e632d. Read the comment docs.

@lucklove
Copy link
Member

/lgtm

@ti-chi-bot ti-chi-bot added the status/LGT1 Indicates that a PR has LGTM 1. label Dec 22, 2020
@lucklove
Copy link
Member

/merge

@ti-chi-bot ti-chi-bot added the status/can-merge Indicates a PR has been approved by a committer. label Dec 22, 2020
@ti-chi-bot
Copy link
Member

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

Commit hash: 7e81676

@ti-chi-bot ti-chi-bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed status/can-merge Indicates a PR has been approved by a committer. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Dec 22, 2020
@lucklove
Copy link
Member

/merge

@ti-chi-bot ti-chi-bot added the status/can-merge Indicates a PR has been approved by a committer. label Dec 28, 2020
@ti-chi-bot
Copy link
Member

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

Commit hash: 36ac1d4

@ti-chi-bot
Copy link
Member

@AstroProfundis: Your PR has out-of-dated and I have automatically updated it for you.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the ti-community-infra/ti-community-prow repository.

@ti-chi-bot ti-chi-bot merged commit b693df4 into pingcap:master Dec 28, 2020
@lucklove lucklove added this to the v1.3.1 milestone Dec 31, 2020
lucklove added a commit that referenced this pull request Dec 31, 2020
* cluster: update dashboard metadata on reload

* only reload for tidb cluster

Co-authored-by: SIGSEGV <[email protected]>
Co-authored-by: Ti Prow Robot <[email protected]>
@AstroProfundis AstroProfundis deleted the reload-dashboard branch January 19, 2021 03:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category/monitoring Categorizes issue or PR related to monitoring components. category/usability Categorizes issue or PR as a usability enhancement. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. status/can-merge Indicates a PR has been approved by a committer. status/LGT1 Indicates that a PR has LGTM 1. type/bug-fix Categorizes PR as a bug-fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants