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 switch to reclaim pv when scale in tikv or pd #1037

Merged
merged 29 commits into from
Nov 6, 2019

Conversation

onlymellb
Copy link
Contributor

@onlymellb onlymellb commented Oct 22, 2019

What problem does this PR solve?

Resolve #908

What is changed and how does it work?

Add enablePVReclaim option in charts/tidbcluster/values.yaml,set this option to true to enable PV reclaim function.

Check List

Tests

  • Unit test
  • E2E test
  • Manual test

Code changes

  • Has Helm charts change
  • Has Go code change
  • Has documents change

Related changes

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

Does this PR introduce a user-facing change?:

add switch `enablePVReclaim` option to reclaim pv when scale in tikv or pd

@onlymellb onlymellb added the enhancement New feature or request label Oct 22, 2019
pkg/manager/member/pvc_cleaner.go Outdated Show resolved Hide resolved
pkg/manager/member/pvc_cleaner.go Outdated Show resolved Hide resolved
pkg/manager/member/pvc_cleaner.go Outdated Show resolved Hide resolved
@onlymellb onlymellb added the status/PTAL PR needs to be reviewed label Oct 29, 2019
@cofyc
Copy link
Contributor

cofyc commented Nov 1, 2019

/run-e2e-in-kind

@onlymellb
Copy link
Contributor Author

/run-e2e-in-kind

1 similar comment
@onlymellb
Copy link
Contributor Author

/run-e2e-in-kind

@weekface
Copy link
Contributor

weekface commented Nov 5, 2019

Can we add this feature to tkctl to reclaim pv manually?

Copy link
Contributor

@weekface weekface left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -10,6 +10,7 @@ tests/images/e2e/tidb-cluster/
tests/images/e2e/tidb-backup/
tests/images/e2e/tidb-operator/
tests/images/e2e/manifests/
manifests/crd-verify.yaml
Copy link
Contributor

Choose a reason for hiding this comment

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

@Yisaer better to update verify script to use a temporary file

@onlymellb onlymellb added this to the v1.1.0 milestone Nov 5, 2019
@onlymellb onlymellb added status/LGTM1 and removed status/PTAL PR needs to be reviewed labels Nov 5, 2019
@onlymellb
Copy link
Contributor Author

/run-e2e-in-kind

Copy link
Contributor

@cofyc cofyc left a comment

Choose a reason for hiding this comment

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

LGTM

@onlymellb onlymellb merged commit a8b94e0 into pingcap:master Nov 6, 2019
@onlymellb onlymellb deleted the add-switch-to-clean-pvc branch November 6, 2019 05:06
yahonda pushed a commit that referenced this pull request Dec 27, 2021
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

tikv-scaler need to auto delete old PVCs after scale in.
6 participants