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

dashboard: support internal proxy #2508

Merged
merged 4 commits into from
Jun 5, 2020
Merged

Conversation

HunDunDM
Copy link
Member

@HunDunDM HunDunDM commented Jun 5, 2020

Signed-off-by: Zheng Xiangsheng hundundm@gmail.com

What problem does this PR solve?

realize #532 Each pd can handle dashboard front-end resource requests

What is changed and how it works?

  • config: add Dashboard.InternalProxy config
  • dashboard: support internal proxy

Check List

Tests

  • Unit test

Code changes

  • Has configuration change

Related changes

Release note

  • The built-in dashboard supports internal proxy.

Signed-off-by: Zheng Xiangsheng <hundundm@gmail.com>
Signed-off-by: Zheng Xiangsheng <hundundm@gmail.com>
Signed-off-by: Zheng Xiangsheng <hundundm@gmail.com>
Signed-off-by: Zheng Xiangsheng <hundundm@gmail.com>
@HunDunDM HunDunDM self-assigned this Jun 5, 2020
@HunDunDM HunDunDM requested review from rleungx and breezewish and removed request for rleungx June 5, 2020 00:23
@HunDunDM HunDunDM added this to the v4.0.1 milestone Jun 5, 2020
@HunDunDM HunDunDM added the needs-cherry-pick-release-4.0 The PR needs to cherry pick to release-4.0 branch. label Jun 5, 2020
@HunDunDM HunDunDM requested a review from rleungx June 5, 2020 00:31
@HunDunDM
Copy link
Member Author

HunDunDM commented Jun 5, 2020

/run-all-tests

@codecov-commenter
Copy link

codecov-commenter commented Jun 5, 2020

Codecov Report

Merging #2508 into master will increase coverage by 0.19%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2508      +/-   ##
==========================================
+ Coverage   77.10%   77.30%   +0.19%     
==========================================
  Files         204      204              
  Lines       21988    22002      +14     
==========================================
+ Hits        16954    17008      +54     
+ Misses       3749     3722      -27     
+ Partials     1285     1272      -13     
Impacted Files Coverage Δ
server/config/config.go 77.70% <ø> (ø)
pkg/dashboard/adapter/redirector.go 94.23% <100.00%> (+0.61%) ⬆️
pkg/dashboard/dashboard.go 90.69% <100.00%> (+2.46%) ⬆️
server/member/lease.go 92.18% <0.00%> (-3.13%) ⬇️
server/schedule/operator_controller.go 81.00% <0.00%> (+0.16%) ⬆️
server/grpc_service.go 60.34% <0.00%> (+0.19%) ⬆️
server/handler.go 51.61% <0.00%> (+0.43%) ⬆️
pkg/btree/btree.go 87.65% <0.00%> (+0.80%) ⬆️
server/server.go 78.31% <0.00%> (+1.24%) ⬆️
server/tso/tso.go 82.48% <0.00%> (+1.45%) ⬆️
... and 6 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 452a0f3...bf767bf. Read the comment docs.

@breezewish
Copy link
Member

/merge

1 similar comment
@HunDunDM
Copy link
Member Author

HunDunDM commented Jun 5, 2020

/merge

@sre-bot sre-bot added the status/can-merge Indicates a PR has been approved by a committer. label Jun 5, 2020
@sre-bot
Copy link
Contributor

sre-bot commented Jun 5, 2020

/run-all-tests

@sre-bot sre-bot merged commit 7094cb5 into tikv:master Jun 5, 2020
@sre-bot
Copy link
Contributor

sre-bot commented Jun 5, 2020

cherry pick to release-4.0 in PR #2511

sre-bot added a commit that referenced this pull request Jun 5, 2020
Signed-off-by: Zheng Xiangsheng <hundundm@gmail.com>
@HunDunDM HunDunDM deleted the prevent-redirect branch August 21, 2020 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-cherry-pick-release-4.0 The PR needs to cherry pick to release-4.0 branch. status/can-merge Indicates a PR has been approved by a committer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants