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

replication.strictly-match-label=true & engine=tiflash have Incompatibility problem #8480

Closed
Chad20N13 opened this issue Aug 1, 2024 · 1 comment · Fixed by #8486
Closed
Labels
affects-5.4 This bug affects the 5.4.x(LTS) versions. affects-6.1 This bug affects the 6.1.x(LTS) versions. affects-6.5 This bug affects the 6.5.x(LTS) versions. affects-7.1 This bug affects the 7.1.x(LTS) versions. affects-7.5 This bug affects the 7.5.x(LTS) versions. affects-8.1 This bug affects the 8.1.x(LTS) versions. severity/major type/bug The issue is confirmed as a bug.

Comments

@Chad20N13
Copy link

Bug Report

What did you do?

  1. After expanding the tiflash, the tiflash node will automatically have a layer of label information: engine: "tiflash"
    2, When the pd is configured: replication. strictly-match-label: true
    3, This causes tiflash unable to start because replication. location-labels in the default pd config does not exist: engine layer configuration.

What did you expect to see?

1、tiflash start successfully
2、replication. location-labels can automatically recognize and add a new "engine" layer;

What did you see instead?

1、tiflash start failed;
2、replication. location-labels can not automatically recognize and keep old layer;

What version of PD are you using (pd-server -V)?

Release Version: v7.5.0
Edition: Community
Git Commit Hash: ef6ba85
Git Branch: heads/refs/tags/v7.5.0
UTC Build Time: 2023-11-16 10:31:04

@Chad20N13 Chad20N13 added the type/bug The issue is confirmed as a bug. label Aug 1, 2024
@rleungx
Copy link
Member

rleungx commented Aug 1, 2024

Thanks, indeed there is a problem, I can reproduce it in my local environment:

With the following config.toml:

[replication]
strictly-match-label = true

Then run tiup playground nightly --pd.config config.toml --tiflash 2

$ tiup playground nightly --pd.config conf/config.toml --tiflash 2
Start pd instance:v8.3.0-alpha-nightly
Start tikv instance:v8.3.0-alpha-nightly
Start tidb instance:v8.3.0-alpha-nightly
Waiting for tidb instances ready
127.0.0.1:33191 ... Done
component grafana version v8.3.0-alpha-nightly is already installed
Start tiflash instance:v8.3.0-alpha-nightly
Start tiflash instance:v8.3.0-alpha-nightly
Waiting for tiflash instances ready
127.0.0.1:3930 ... ⠦
127.0.0.1:3931 ... ⠦
tiflash quit: exit status 1
[2024/08/01 20:05:09.931 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=192]
[2024/08/01 20:05:09.931 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=192]
[2024/08/01 20:05:09.931 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=192]
[2024/08/01 20:05:09.931 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=192]
[2024/08/01 20:05:09.931 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=192]
[2024/08/01 20:05:09.931 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=192]
[2024/08/01 20:05:09.931 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=192]
[2024/08/01 20:05:09.931 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=192]
[2024/08/01 20:05:09.931 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=192]
Waiting for tiflash instances ready
127.0.0.1:3930 ... ⠏
127.0.0.1:3931 ... ⠏
tiflash quit: exit status 1
[2024/08/01 20:05:10.124 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=171]
[2024/08/01 20:05:10.124 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=171]
[2024/08/01 20:05:10.124 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=171]
[2024/08/01 20:05:10.124 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=171]
[2024/08/01 20:05:10.124 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=171]
[2024/08/01 20:05:10.124 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=171]
[2024/08/01 20:05:10.124 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=171]
[2024/08/01 20:05:10.124 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=171]
[2024/08/01 20:05:10.124 +08:00] [DEBUG] [SegmentReader.cpp:45] [Stopped] [thread_id=171]
Waiting for tiflash instances ready
127.0.0.1:3930 ... Error
127.0.0.1:3931 ... Error

Here is the log:

[2024/08/01 20:05:09.899 +08:00] [FATAL] [run.rs:1372] ["failed to start node: Other(\"[components/pd_client/src/util.rs:952]: key matching the label was not found in the PD, store label key: engine \")"] [thread_id=1]

@rleungx rleungx added affects-6.1 This bug affects the 6.1.x(LTS) versions. affects-6.5 This bug affects the 6.5.x(LTS) versions. affects-7.1 This bug affects the 7.1.x(LTS) versions. affects-7.5 This bug affects the 7.5.x(LTS) versions. affects-8.1 This bug affects the 8.1.x(LTS) versions. severity/major labels Aug 5, 2024
@rleungx rleungx added affects-5.4 This bug affects the 5.4.x(LTS) versions. and removed may-affects-5.4 labels Aug 5, 2024
ti-chi-bot bot added a commit that referenced this issue Aug 6, 2024
close #8480

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

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
@ti-chi-bot ti-chi-bot bot closed this as completed in #8486 Aug 6, 2024
ti-chi-bot pushed a commit to ti-chi-bot/pd that referenced this issue Aug 27, 2024
ti-chi-bot pushed a commit to ti-chi-bot/pd that referenced this issue Aug 27, 2024
ti-chi-bot pushed a commit to ti-chi-bot/pd that referenced this issue Aug 27, 2024
ti-chi-bot pushed a commit to ti-chi-bot/pd that referenced this issue Aug 27, 2024
ti-chi-bot pushed a commit to ti-chi-bot/pd that referenced this issue Aug 27, 2024
ti-chi-bot bot pushed a commit that referenced this issue Aug 27, 2024
ti-chi-bot bot added a commit that referenced this issue Aug 27, 2024
close #8480

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

Co-authored-by: Ryan Leung <[email protected]>
Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
ti-chi-bot bot pushed a commit that referenced this issue Aug 28, 2024
close #8480

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

Co-authored-by: Ryan Leung <[email protected]>
ti-chi-bot bot pushed a commit that referenced this issue Aug 28, 2024
close #8480

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

Co-authored-by: Ryan Leung <[email protected]>
ti-chi-bot bot pushed a commit that referenced this issue Aug 28, 2024
close #8480

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

Co-authored-by: Ryan Leung <[email protected]>
ti-chi-bot bot added a commit that referenced this issue Sep 11, 2024
close #8480

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

Co-authored-by: Ryan Leung <[email protected]>
Co-authored-by: Purelind <[email protected]>
Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-5.4 This bug affects the 5.4.x(LTS) versions. affects-6.1 This bug affects the 6.1.x(LTS) versions. affects-6.5 This bug affects the 6.5.x(LTS) versions. affects-7.1 This bug affects the 7.1.x(LTS) versions. affects-7.5 This bug affects the 7.5.x(LTS) versions. affects-8.1 This bug affects the 8.1.x(LTS) versions. severity/major type/bug The issue is confirmed as a bug.
Projects
Development

Successfully merging a pull request may close this issue.

2 participants