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

server: tolerate backslash in the label name #1595

Merged
merged 2 commits into from
Jun 24, 2019

Conversation

shafreeck
Copy link
Contributor

Signed-off-by: Shafreeck Sea shafreeck@gmail.com

Closes #1588

What problem does this PR solve?

tolerate backslash in the label name.

What is changed and how it works?

relax the rules to check the validation of a label.

Check List

Tests

  • Unit test

Signed-off-by: Shafreeck Sea <shafreeck@gmail.com>

Closes tikv#1588
@shafreeck shafreeck self-assigned this Jun 24, 2019
@shafreeck shafreeck added component/schedule Scheduling logic. type/enhancement The issue or PR belongs to an enhancement. labels Jun 24, 2019
server/util.go Outdated Show resolved Hide resolved
Signed-off-by: Shafreeck Sea <shafreeck@gmail.com>
@nolouch nolouch merged commit d91268e into tikv:master Jun 24, 2019
@nolouch nolouch added needs-cherry-pick-release-3.0 The PR needs to cherry pick to release-3.0 branch. needs-cherry-pick-release-3.1 The PR needs to cherry pick to release-3.1 branch. labels Jan 7, 2020
@nolouch
Copy link
Contributor

nolouch commented Jan 7, 2020

/run-cherry-picker

@sre-bot
Copy link
Contributor

sre-bot commented Jan 7, 2020

cherry pick to release-3.0 in PR #2083

@sre-bot
Copy link
Contributor

sre-bot commented Jan 7, 2020

cherry pick to release-3.1 in PR #2084

sre-bot added a commit that referenced this pull request Jan 7, 2020
Signed-off-by: Shafreeck Sea <shafreeck@gmail.com>
sre-bot added a commit that referenced this pull request Jan 7, 2020
Signed-off-by: Shafreeck Sea <shafreeck@gmail.com>
@shafreeck shafreeck changed the title feature(label): tolerate backslash in the label name server: tolerate backslash in the label name Jan 7, 2020
@shafreeck shafreeck deleted the feature/label-validation branch January 7, 2020 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/schedule Scheduling logic. needs-cherry-pick-release-3.0 The PR needs to cherry pick to release-3.0 branch. needs-cherry-pick-release-3.1 The PR needs to cherry pick to release-3.1 branch. type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

location-labels does not support Backslash symbols
4 participants