Closed rajsuvariya closed 1 week ago
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
17 out of 18 committers have signed the CLA.
:white_check_mark: smineyev81
:white_check_mark: ideascf
:white_check_mark: wangz1x
:white_check_mark: HuSharp
:white_check_mark: csuzhangxc
:white_check_mark: michaelmdeng
:white_check_mark: fgksgf
:white_check_mark: xhebox
:white_check_mark: BornChanger
:white_check_mark: YuJuncen
:white_check_mark: WangLe1321
:white_check_mark: donbowman
:white_check_mark: lhy1024
:white_check_mark: wxiaomou
:white_check_mark: rajsuvariya
:white_check_mark: wuhuizuo
:white_check_mark: djshow832
:x: MarsBighead
You have signed the CLA already but the status is still pending? Let us recheck it.
Welcome @rajsuvariya! It looks like this is your first PR to pingcap/tidb-operator š
@anish-db: adding LGTM is restricted to approvers and reviewers in OWNERS files.
/run-pull-e2e-kind
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: csuzhangxc
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Timeline:
2024-06-26 07:58:05.375752717 +0000 UTC m=+792811.861241547
: :ballot_box_with_check: agreed by csuzhangxc.2024-06-26 08:10:31.120393519 +0000 UTC m=+793557.605882351
: :heavy_multiplication_x::repeat: reset by ti-chi-bot[bot].New changes are detected. LGTM label has been removed.
/retest
@rajsuvariya: Cannot trigger testing until a trusted user reviews the PR and leaves an /ok-to-test
message.
/run-pull-e2e-kind
/run-pull-e2e-kind
I will raise another PR, due to CLA agreement
Raised another PR #5670
What problem does this PR solve?
https://github.com/pingcap/tidb-operator/blob/9ef26f88bf75338cef427b44aa6303237ddcdbd6/pkg/features/features.go#L55C9-L57C44 PVC replace feature is currently enabled / disabled at tidb-operator level, which might not be comforting situation as a single tidb-operator can be handling multiple tidbclusters, so this PR enables tidbcluster level flag to enable / disable this feature. This is similar to another similar feature
enablePVReclaim
flagWhat is changed and how does it work?
Added a new flag in CRD
enablePVCReplace
with a false as default value. If this flag is enabled then only tidb-operator would perform the PVC replacement process.Code changes
Tests
Side effects
Related changes
Release Notes
Please refer to Release Notes Language Style Guide before writing the release note.