-
Notifications
You must be signed in to change notification settings - Fork 283
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
engine(dm, externalresouce): enable s3 for dm #7336
engine(dm, externalresouce): enable s3 for dm #7336
Conversation
[REVIEW NOTIFICATION] This pull request has been approved by:
To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by submitting an approval review. |
can you also deploy minio in helm? https://github.com/pingcap/tiflow/blob/master/deployments/engine/helm/tiflow/values.yaml |
We do not need to deploy minio directly in helm, since a public minio service is available on test_infra. Here we provide a configuration file to deploy minio in k8s, which can be deployed manually in githuc action. |
905b195
to
934c12c
Compare
/run-all-tests |
chao tests verified in https://github.com/CharlesCheung96/tiflow/actions/runs/3247512899 |
0253f82
to
b4b607b
Compare
/run-all-tests |
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.
rest lgtm
Can some TODO in chaos test also be fixed? @GMHDBJD |
3694c50
to
befb66a
Compare
/run-verify |
/merge |
This pull request has been accepted and is ready to merge. Commit hash: efcac30
|
99f5e80
to
22a04e9
Compare
/run-all-tests |
/merge |
This pull request has been accepted and is ready to merge. Commit hash: 22a04e9
|
/run-engine-integration-test |
What problem does this PR solve?
Issue Number: close #7114, ref #7304
What is changed and how it works?
Check List
Tests
Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note