Skip to content

Fix tiflash not start with newest nightly PD#902

Merged
AstroProfundis merged 3 commits intopingcap:masterfrom
lucklove:fix-tispark
Nov 16, 2020
Merged

Fix tiflash not start with newest nightly PD#902
AstroProfundis merged 3 commits intopingcap:masterfrom
lucklove:fix-tispark

Conversation

@lucklove
Copy link
Copy Markdown
Member

What problem does this PR solve?

#888

What is changed and how it works?

Change "enable-placement-rules" to "false" before modify "max-replicas"

Check List

Tests

  • Manual test (add detailed scripts or steps below)

Related changes

  • Need to cherry-pick to the release branch

Release notes:

NONE

@lucklove lucklove added this to the v1.3.0 milestone Nov 12, 2020
@codecov-io
Copy link
Copy Markdown

codecov-io commented Nov 12, 2020

Codecov Report

Merging #902 (e728eac) into master (cc41ad5) will increase coverage by 0.01%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #902      +/-   ##
==========================================
+ Coverage   53.24%   53.25%   +0.01%     
==========================================
  Files         259      259              
  Lines       18785    18786       +1     
==========================================
+ Hits        10002    10005       +3     
+ Misses       7253     7251       -2     
  Partials     1530     1530              
Flag Coverage Δ
cluster 45.52% <ø> (+0.02%) ⬆️
dm 25.21% <ø> (ø)
integrate 48.01% <0.00%> (+0.01%) ⬆️
playground 22.26% <0.00%> (-0.01%) ⬇️
tiup 10.75% <ø> (ø)
unittest 21.39% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
components/playground/instance/tiflash.go 0.00% <0.00%> (ø)
pkg/cluster/api/pdapi.go 62.53% <0.00%> (+0.92%) ⬆️

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 cc41ad5...e728eac. Read the comment docs.

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Nov 13, 2020
@AstroProfundis AstroProfundis merged commit 402079d into pingcap:master Nov 16, 2020
@lucklove lucklove modified the milestones: v1.3.0, v1.2.4 Nov 19, 2020
lucklove added a commit to lucklove/tiup that referenced this pull request Apr 5, 2021
This reverts commit 402079d.

We should reverts pingcap#902 because it introduced an issue:
pingcap#1279

BTW, PD itself fixes pingcap#888 in tikv/pd#3163
So we don't need pingcap#902 anymore with recently released PD.

However, after revert pingcap#902, *** old PDs (before #3163) may not work
with tiup-playground ***
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/LGT1 Indicates that a PR has LGTM 1.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants