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

GetRegionLabelRulesByIDs incorrectly returned all the region label rules #8491

Closed
JmPotato opened this issue Aug 5, 2024 · 0 comments · Fixed by #8490
Closed

GetRegionLabelRulesByIDs incorrectly returned all the region label rules #8491

JmPotato opened this issue Aug 5, 2024 · 0 comments · Fixed by #8490
Labels
affects-8.1 severity/critical type/bug The issue is confirmed as a bug.

Comments

@JmPotato
Copy link
Member

JmPotato commented Aug 5, 2024

Bug Report

With an incorrect URI, GetRegionLabelRulesByIDs will return all region label rules without using the passed ruleIDs .
@JmPotato JmPotato added type/bug The issue is confirmed as a bug. severity/critical labels Aug 5, 2024
@JmPotato JmPotato mentioned this issue Aug 5, 2024
Merged
@ti-chi-bot ti-chi-bot bot closed this as completed in #8490 Aug 5, 2024
@ti-chi-bot ti-chi-bot bot closed this as completed in 838ee79 Aug 5, 2024
ti-chi-bot pushed a commit to ti-chi-bot/pd that referenced this issue Aug 5, 2024
@JmPotato @ti-chi-bot
This is an automated cherry-pick of tikv#8490
456fa93 
close tikv#8491, ref pingcap/tidb#55188

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
@ti-chi-bot ti-chi-bot mentioned this issue Aug 5, 2024
Merged
ti-chi-bot bot pushed a commit that referenced this issue Aug 5, 2024
@ti-chi-bot @JmPotato
client/http: fix the URI of GetRegionLabelRulesByIDs (#8490) (#8493)
3d11c4d 
close #8491, ref pingcap/tidb#55188

Fix the URI of `GetRegionLabelRulesByIDs` to correct the call.

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
Signed-off-by: JmPotato <ghzpotato@gmail.com>

Co-authored-by: JmPotato <ghzpotato@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
affects-8.1 severity/critical type/bug The issue is confirmed as a bug.
Projects
Questions and Bug Reports
Status: Closed
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

client/http: fix the URI of GetRegionLabelRulesByIDs JmPotato/pd
1 participant
@JmPotato