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

The log is confusing #9036

Closed
rleungx opened this issue Feb 6, 2025 · 1 comment
Closed

The log is confusing #9036

rleungx opened this issue Feb 6, 2025 · 1 comment
Labels
affects-6.5 This bug affects the 6.5.x(LTS) versions. affects-7.1 This bug affects the 7.1.x(LTS) versions. severity/minor type/bug The issue is confirmed as a bug.

Comments

@rleungx
Copy link
Member

rleungx commented Feb 6, 2025 •
edited
Loading

Bug Report

[2025/01/20 04:08:24.588 +00:00] [INFO] [base_client.go:305] ["[pd] cannot update member from this address"] [address=https://mussel-prod-ml-dr-pd-0.mussel-prod-ml-dr-pd-peer.tidb-mussel-prod-ml-dr.svc.us
-east-1a.mussel-prod-ml-dr.tidb.musta.ch:2379] [error="[PD:client:ErrClientGetLeader]get leader from leader address don't exist error"]

get leader from leader address don't exist error is confusing. We need to improve it.
@rleungx rleungx added the type/enhancement The issue or PR belongs to an enhancement. label Feb 6, 2025
@rleungx
Copy link
Member Author

rleungx commented Feb 7, 2025

Only release-6.5 and release-7.1 have this issue. Improved by #6321.

@rleungx rleungx mentioned this issue Feb 10, 2025
Merged
@rleungx rleungx added type/bug The issue is confirmed as a bug. severity/minor affects-6.5 This bug affects the 6.5.x(LTS) versions. affects-7.1 This bug affects the 7.1.x(LTS) versions. and removed type/enhancement The issue or PR belongs to an enhancement. labels Feb 10, 2025
ti-chi-bot bot pushed a commit that referenced this issue Feb 11, 2025
@rleungx
client: fix confusing error (#9050)
aed98ba 
close #9036

Signed-off-by: Ryan Leung <[email protected]>
ti-chi-bot pushed a commit to ti-chi-bot/pd that referenced this issue Feb 11, 2025
@rleungx @ti-chi-bot
This is an automated cherry-pick of tikv#9050
55f17a7 
close tikv#9036

Signed-off-by: ti-chi-bot <[email protected]>
@ti-chi-bot ti-chi-bot mentioned this issue Feb 11, 2025
Merged
@rleungx rleungx closed this as completed Feb 11, 2025
ti-chi-bot bot pushed a commit that referenced this issue Feb 11, 2025
@ti-chi-bot @rleungx
client: fix confusing error (#9050) (#9061)
e19478d 
close #9036

Signed-off-by: ti-chi-bot <[email protected]>
Signed-off-by: Ryan Leung <[email protected]>

Co-authored-by: Ryan Leung <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
affects-6.5 This bug affects the 6.5.x(LTS) versions. affects-7.1 This bug affects the 7.1.x(LTS) versions. severity/minor type/bug The issue is confirmed as a bug.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rleungx