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.

Sign up for GitHub

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

Jump to bottom

Tracking issue for resource manager #5851

Closed
7 tasks done
CabinfeverB opened this issue Jan 11, 2023 · 0 comments
Closed
7 tasks done

Tracking issue for resource manager #5851

CabinfeverB opened this issue Jan 11, 2023 · 0 comments
Labels
type/development The issue belongs to a development tasks

Comments

@CabinfeverB
Copy link
Member

CabinfeverB commented Jan 11, 2023 •
edited
Loading

Development Task

Ref pingcap/tidb#38825
@CabinfeverB CabinfeverB added the type/development The issue belongs to a development tasks label Jan 11, 2023
This was referenced Jan 11, 2023
Merged
Merged
@rleungx rleungx mentioned this issue Jan 12, 2023
Merged
ti-chi-bot added a commit that referenced this issue Jan 13, 2023
@rleungx @ti-chi-bot
*: add Makefile for msc (#5856)
6be27aa 
ref #5851

Signed-off-by: Ryan Leung <rleungx@gmail.com>

Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
@JmPotato JmPotato mentioned this issue Jan 13, 2023
Merged
This was referenced Jan 15, 2023
Merged
Merged
ti-chi-bot added a commit that referenced this issue Jan 16, 2023
@JmPotato @ti-chi-bot
resource_manager: introduce the resource unit model and config (#5862)
c6ebcec 
ref #5811, ref #5851

Introduce the resource unit (RU) model and related config.

Signed-off-by: JmPotato <ghzpotato@gmail.com>

Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
ti-chi-bot added a commit that referenced this issue Jan 17, 2023
@CabinfeverB @ti-chi-bot
resouce_manager: fix the bug of loading resource group and using copy (…
fda03e5 
…#5867)

ref #5851

Signed-off-by: Cabinfever_B <cabinfeveroier@gmail.com>

Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
ti-chi-bot added a commit that referenced this issue Jan 17, 2023
@CabinfeverB @ti-chi-bot
resource_manager: impl limiter for token controlloer (#5868)
b2a0532 
ref #5851

impl limiter for token controlloer

Signed-off-by: Cabinfever_B <cabinfeveroier@gmail.com>

Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
ti-chi-bot added a commit that referenced this issue Jan 19, 2023
@CabinfeverB @ti-chi-bot
resouce_manager: impl resource controller for tokens client (#5811)
402c2bf 
ref #5851

Signed-off-by: Cabinfever_B <cabinfeveroier@gmail.com>

Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
@CabinfeverB CabinfeverB mentioned this issue Jan 28, 2023
Merged
ti-chi-bot pushed a commit that referenced this issue Jan 29, 2023
@CabinfeverB
resource_manager: remove maxRequestTokens in limiter (#5880)
c55e632 
ref #5851

Signed-off-by: Cabinfever_B <cabinfeveroier@gmail.com>
This was referenced Jan 29, 2023
Closed
Merged
Merged
ti-chi-bot added a commit that referenced this issue Feb 1, 2023
@CabinfeverB @ti-chi-bot
resource_manager, storage: create new storage endpoint for resource m…
73c188f 
…anager (#5887)

ref #5851, ref #5886, ref pingcap/tiflow#8110

Signed-off-by: Cabinfever_B <cabinfeveroier@gmail.com>

Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
ti-chi-bot added a commit that referenced this issue Feb 1, 2023
@CabinfeverB @ti-chi-bot
resource_manager: unify RRU and WRU into RU for token limit (#5888)
6ca9a33 
ref #5851

Signed-off-by: Cabinfever_B <cabinfeveroier@gmail.com>

Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
@JmPotato JmPotato mentioned this issue Feb 2, 2023
Merged
ti-chi-bot added a commit that referenced this issue Feb 6, 2023
@JmPotato @ti-chi-bot
resource_manager, server: implement the resource manager failover (#5900
3844971 
)

ref #5851

- Refine the resource manager initialization to support failover via PD election.
- Implement the client failover.
- Fix a race bug.

Signed-off-by: JmPotato <ghzpotato@gmail.com>

Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
@JmPotato JmPotato mentioned this issue Feb 6, 2023
Merged
ti-chi-bot added a commit that referenced this issue Feb 6, 2023
@JmPotato @ti-chi-bot
resource_manager: adjust the RU model config (#5912)
489f786 
ref #5851

- Adjust the RU model config.
- Unify the SQL and KV CPU time RU cost.

Signed-off-by: JmPotato <ghzpotato@gmail.com>

Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
disksing pushed a commit to oh-my-tidb/pd that referenced this issue Feb 6, 2023
@JmPotato @ti-chi-bot @disksing
resource_manager: adjust the RU model config (tikv#5912)
9de7fe9 
ref tikv#5851

- Adjust the RU model config.
- Unify the SQL and KV CPU time RU cost.

Signed-off-by: JmPotato <ghzpotato@gmail.com>

Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
This was referenced Feb 6, 2023
Merged
Merged
@HuSharp HuSharp mentioned this issue Feb 7, 2023
Merged
ti-chi-bot added a commit that referenced this issue Feb 8, 2023
@CabinfeverB @ti-chi-bot
resource_manager: replace MaxToken by BurstLimit (#5931)
5811b9f 
ref #5851

Signed-off-by: Cabinfever_B <cabinfeveroier@gmail.com>

Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
@JmPotato JmPotato mentioned this issue Feb 8, 2023
Merged
ti-chi-bot added a commit that referenced this issue Feb 9, 2023
@CabinfeverB @nolouch @ti-chi-bot
resource_manager: save tokens in persistent storage (#5906)
162778e 
ref #5851

Signed-off-by: Cabinfever_B <cabinfeveroier@gmail.com>

Co-authored-by: ShuNing <nolouch@gmail.com>
Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
@glorv glorv mentioned this issue Mar 7, 2023
Merged
ti-chi-bot pushed a commit that referenced this issue Mar 7, 2023
@glorv
resource_manager: change cpu time unit from ms to ns (#6102)
0c3ac6e 
ref #5851

Signed-off-by: glorv <glorvs@163.com>
This was referenced Mar 8, 2023
Closed
Closed
@HuSharp HuSharp mentioned this issue Mar 10, 2023
Merged
ti-chi-bot pushed a commit that referenced this issue Mar 10, 2023
@HuSharp
resource_manager_client: fix client nil possible to fix panic (#6137)
f850aab 
ref #5851, close #6131

need to check GetOrCreateGRPCConn error to avoid panic

Signed-off-by: husharp <jinhao.hu@pingcap.com>
ti-chi-bot added a commit that referenced this issue Mar 13, 2023
@HuSharp @ti-chi-bot
resource_manager_client: remove stale resource group (#6098)
35e2e11 
ref #5851

remove stale resource group which have not been requested

Signed-off-by: husharp <jinhao.hu@pingcap.com>

Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
@JmPotato JmPotato mentioned this issue Mar 13, 2023
Merged
ti-chi-bot pushed a commit that referenced this issue Mar 13, 2023
@JmPotato
controller: return the consumption info on RU request/response (#6125)
3e3ae55 
ref #5851, ref pingcap/tidb#41972

- Return the consumption info on RU request/response.
- Remove some unnecessary `goto`s.

Signed-off-by: JmPotato <ghzpotato@gmail.com>
ti-chi-bot added a commit that referenced this issue Mar 16, 2023
@CabinfeverB @ti-chi-bot
resource_manager: add degraded mode (#6063)
7a0ce10 
ref #5851

Signed-off-by: Cabinfever_B <cabinfeveroier@gmail.com>

Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
@HuSharp HuSharp mentioned this issue Apr 12, 2023
Merged
ti-chi-bot added a commit that referenced this issue Apr 13, 2023
@HuSharp @ti-chi-bot
resource_manager/client: Abstract the funcs of all group iterative ex…
ab0dbdc 
…ec. (#6303)

ref #5851

Abstract the funcs of all group iterative exec.

Signed-off-by: husharp <jinhao.hu@pingcap.com>

Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
@glorv glorv mentioned this issue Jun 29, 2023
Merged
ti-chi-bot bot added a commit that referenced this issue Jul 3, 2023
@glorv @ti-chi-bot
resourcemanager: do not check existence when add resource group (#6717)
5f7236f 
ref #5851, ref pingcap/tidb#45050

Signed-off-by: glorv <glorvs@163.com>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
@ti-chi-bot ti-chi-bot mentioned this issue Jul 3, 2023
Merged
@HuSharp HuSharp mentioned this issue Jul 6, 2023
Merged
ti-chi-bot bot added a commit that referenced this issue Jul 24, 2023
@HuSharp @ti-chi-bot
resource_manager: ignore ru metrics for background request (#6766)
de985b8 
ref #5851

Signed-off-by: husharp <jinhao.hu@pingcap.com>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
ti-chi-bot bot added a commit that referenced this issue Oct 12, 2023
@ti-chi-bot @glorv @ti-chi-bot
resourcemanager: do not check existence when add resource group (#6717)…
b0477c9 
… (#6739)

ref #5851, ref pingcap/tidb#45050

Signed-off-by: glorv <glorvs@163.com>

Co-authored-by: glorv <glorvs@163.com>
Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
@glorv glorv mentioned this issue Nov 30, 2023
Merged
ti-chi-bot bot pushed a commit that referenced this issue Dec 4, 2023
@glorv
client: return total wait duration in resource interceptor OnRequestW…
259435d 
…ait call (#7488)

ref #5851

Signed-off-by: glorv <glorvs@163.com>
ti-chi-bot pushed a commit to ti-chi-bot/pd that referenced this issue Feb 2, 2024
@glorv @ti-chi-bot
This is an automated cherry-pick of tikv#7488
2d59247 
ref tikv#5851

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
@ti-chi-bot ti-chi-bot mentioned this issue Feb 2, 2024
Merged
ti-chi-bot pushed a commit to ti-chi-bot/pd that referenced this issue Feb 2, 2024
@glorv @ti-chi-bot
This is an automated cherry-pick of tikv#7488
7dd9b8e 
ref tikv#5851

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
@ti-chi-bot ti-chi-bot mentioned this issue Feb 2, 2024
Closed
@nolouch nolouch added the affects-7.5 This bug affects the 7.5.x(LTS) versions. label Feb 2, 2024
ti-chi-bot bot pushed a commit that referenced this issue Feb 2, 2024
@ti-chi-bot @glorv
client: return total wait duration in resource interceptor OnRequestW…
6978558 
…ait call (#7488) (#7794)

ref #5851

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
Signed-off-by: glorv <glorvs@163.com>

Co-authored-by: glorv <glorvs@163.com>
@nolouch nolouch removed the affects-7.5 This bug affects the 7.5.x(LTS) versions. label Feb 2, 2024
@HuSharp HuSharp mentioned this issue Feb 22, 2024
Merged
ti-chi-bot bot pushed a commit that referenced this issue Feb 26, 2024
@HuSharp
meta_storage: support delete interface (#7096)
e199866 
ref #5851

This PR adds the basic delete function
Here is the proto definition: pingcap/kvproto#1226

Signed-off-by: husharp <jinhao.hu@pingcap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/development The issue belongs to a development tasks
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nolouch @CabinfeverB