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

config: disable label check by default #1568

Merged
merged 3 commits into from Jun 10, 2019

Conversation

nolouch
Copy link
Contributor

@nolouch nolouch commented Jun 10, 2019

Signed-off-by: nolouch nolouch@gmail.com

What problem does this PR solve?

Now, tidb-operator not supports the user-defined label, it does not startup success. Until it is supported, we will open it by default.
disable label check by default

Check List

Tests

  • Unit test

Signed-off-by: nolouch <nolouch@gmail.com>
Signed-off-by: nolouch <nolouch@gmail.com>
@nolouch nolouch requested review from disksing and rleungx June 10, 2019 06:50
@siddontang
Copy link
Contributor

why default to false now?

@nolouch
Copy link
Contributor Author

nolouch commented Jun 10, 2019

Now, tidb-operator not supports the user-defined label, it does not startup success. Until it is supported, we will open it by default @siddontang

@rleungx rleungx merged commit 8051f7b into tikv:master Jun 10, 2019
nolouch added a commit to nolouch/pd that referenced this pull request Jun 17, 2019
Signed-off-by: nolouch <nolouch@gmail.com>
disksing pushed a commit that referenced this pull request Jun 17, 2019
* config: warn undefined config item (#1577)

* config: warn undefined config item

Signed-off-by: nolouch <nolouch@gmail.com>

* *: add option to check label strictly (#1553)

* *: add option to check tikv label strictly

Signed-off-by: nolouch <nolouch@gmail.com>

* config: disable label check by default (#1568)

Signed-off-by: nolouch <nolouch@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants