-
Notifications
You must be signed in to change notification settings - Fork 501
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
http-server: add deleteTidbCluster api #5371
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Welcome @z2665! It looks like this is your first PR to pingcap/tidb-operator 🎉 |
917e645
to
cc04ee7
Compare
34387f4
to
bf432c8
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #5371 +/- ##
=======================================
Coverage 61.59% 61.60%
=======================================
Files 228 228
Lines 28866 28866
=======================================
+ Hits 17780 17782 +2
+ Misses 9342 9341 -1
+ Partials 1744 1743 -1
|
What problem does this PR solve?
add delete TidbCluster api for http-server
What is changed and how does it work?
Code changes
Tests
use CreateCluster http api to create a tidb cluster
see example/create-cluster.json
check tidb running success
call DeleteCluster http api to delete the tidb cluster
check namespace,secret,tidbmonitor and tidbInitializers all be deleted
Side effects
Related changes
Release Notes
Please refer to Release Notes Language Style Guide before writing the release note.