-
Notifications
You must be signed in to change notification settings - Fork 526
Add TiCI support to TidbCluster with TiFlash integration and example #6699
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
base: release-1.x
Are you sure you want to change the base?
Conversation
|
Skipping CI for Draft Pull Request. |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## release-1.x #6699 +/- ##
==============================================
Coverage ? 56.42%
==============================================
Files ? 266
Lines ? 39210
Branches ? 0
==============================================
Hits ? 22124
Misses ? 15002
Partials ? 2084
🚀 New features to boost your workflow:
|
|
please submit pr to release-1.x |
7646038 to
b143dc5
Compare
b143dc5 to
6513fa7
Compare
JaySon-Huang
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
|
@JaySon-Huang: adding LGTM is restricted to approvers and reviewers in OWNERS files. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: JaySon-Huang The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
pkg/manager/member/tiflash_util.go
Outdated
| common.SetIfNil("flash.tidb_status_addr", tidbStatusAddr) | ||
| if !mountCMInTiflashContainer { | ||
| common.SetIfNil("flash.service_addr", fmt.Sprintf("%s:%d", listenHost, v1alpha1.DefaultTiFlashFlashPort)) | ||
| common.SetIfNil("flash.service_addr", fmt.Sprintf("%s-POD_NUM.%s.%s.svc%s:%d", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This may cause existing TiFlash pods to be recreated after upgrading the TiDB Operator.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for pointing this out. I changed flash.service_addr in getTiFlashConfigV2 to be conditional: for non-TiCI clusters it keeps the previous value (listenHost:3930, e.g. 0.0.0.0:3930) to avoid unnecessary TiFlash pod recreation on operator upgrade; when TiCI is enabled (spec.tici.s3 configured) it uses the new headless service address.
examples/tici/tici-tc-gcs-wi.yaml
Outdated
| region: us-east-1 | ||
| accessKey: "" | ||
| secretKey: "" | ||
| bucket: tici_test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"_" is invalid in gcs bucket name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed. I changed the GCS bucket example from tici_test to tici-test and updated the corresponding sinkURI to match. I also aligned the related docs/examples to use the same valid bucket name consistently.
|
@3pointer: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Summary
This PR adds TiCI (meta/worker) as a first‑class component in TidbCluster, wires it into the controller sync loop, integrates TiFlash with TiCI config, and provides an end‑to‑end example (including MinIO + FTS verification) for local testing.
For more details about how to run tici via tidb-operator: please refer to
examples/tici/readme.mdKey Changes
##Testing
Notes / Operational Considerations
Follow‑ups