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

Add support for max-index-length to TidbCluster CRD #3076

Merged
merged 3 commits into from
Aug 7, 2020

Conversation

kolbe
Copy link
Contributor

@kolbe kolbe commented Aug 5, 2020

What problem does this PR solve?

Fixes #3075

What is changed and how does it work?

Add max-index-length to TidbCluster and TiDBGroup in manifests/crd.yaml, and add MaxIndexLength to tidb_config.go.

Check List

Tests

  • Manual test (add detailed scripts or steps below)

I was able to successfully add max-index-length to the config in TidbCluster and have it applied to nodes in the cluster.

Code changes

  • Has Go code change

Side effects

n/a

Related changes

  • Need to cherry-pick to the release branch
  • Need to update the documentation

Does this PR introduce a user-facing change?:

Add support for the `max-index-length` (https://docs.pingcap.com/tidb/stable/tidb-configuration-file#max-index-length) TiDB config option to the TidbCluster CRD

@codecov-commenter
Copy link

codecov-commenter commented Aug 5, 2020

Codecov Report

Merging #3076 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3076   +/-   ##
=======================================
  Coverage   42.35%   42.35%           
=======================================
  Files         156      156           
  Lines       16767    16767           
=======================================
  Hits         7102     7102           
  Misses       9092     9092           
  Partials      573      573           
Flag Coverage Δ
#unittest 42.35% <ø> (?)

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

@DanielZhangQD
Copy link
Contributor

@kolbe Please run hack/update-all.sh if you change something in the types.go.

Copy link
Contributor

@DanielZhangQD DanielZhangQD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cofyc cofyc merged commit 22a911a into pingcap:master Aug 7, 2020
ti-srebot pushed a commit to ti-srebot/tidb-operator that referenced this pull request Aug 7, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot
Copy link
Contributor

cherry pick to release-1.1 in PR #3087

ti-srebot added a commit that referenced this pull request Aug 7, 2020
* cherry pick #3076 to release-1.1

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>

* update crd

Co-authored-by: Kolbe Kegel <kolbe@pingcap.com>
Co-authored-by: DanielZhangQD <zhanghailong810@aliyun.com>
Co-authored-by: Lonng <heng@lonng.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

max-index-length missing from crd and operator
5 participants