Skip to content

fix CI failed because ttl-enable TiKV cluster cannot support TxnKV #281

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.

Sign up for GitHub

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

Jump to bottom
Merged
marsishandsome merged 1 commit into from
Sep 30, 2021
Merged

fix CI failed because ttl-enable TiKV cluster cannot support TxnKV #281

marsishandsome merged 1 commit into from
Sep 30, 2021

Conversation

@marsishandsome
Copy link
Collaborator

@marsishandsome marsishandsome commented Sep 24, 2021 •
edited
Loading

Signed-off-by: marsishandsome marsishandsome@gmail.com

What problem does this PR solve?

close #270

CI failed because ttl-enable TiKV cluster cannot support TxnKV

What is changed and how it works?

run two different tikv clusters

  • one for rawkv test
  • another for txnkv test

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Code changes

  • Has exported function/method change
  • Has exported variable/fields change
  • Has interface methods change
  • Has persistent data change

Side effects

  • Possible performance regression
  • Increased code complexity
  • Breaking backward compatibility

Related changes

  • Need to cherry-pick to the release branch
  • Need to update the documentation
  • Need to be included in the release note

@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 24, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 929a7ec to 6c9ac8c Compare September 24, 2021 07:48
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 24, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 6c9ac8c to 696bd2b Compare September 24, 2021 08:03
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 24, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 696bd2b to 9fba890 Compare September 24, 2021 08:37
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 24, 2021

/run-all-tests

@marsishandsome marsishandsome changed the title ignore ci test with TxnKV [WIP] ignore ci test with TxnKV Sep 24, 2021
@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 9fba890 to 859e6ac Compare September 27, 2021 03:28
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 27, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 859e6ac to 6a0cb84 Compare September 27, 2021 03:29
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 27, 2021

/run-all-tests

@marsishandsome marsishandsome changed the title [WIP] ignore ci test with TxnKV [WIP] optimize ci test Sep 27, 2021
@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 6a0cb84 to 588ed7a Compare September 27, 2021 03:40
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 27, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch 2 times, most recently from b6093bf to c659602 Compare September 27, 2021 05:49
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 27, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch 3 times, most recently from 56cd397 to c7e289e Compare September 27, 2021 06:26
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 27, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from c7e289e to 4117d3a Compare September 27, 2021 06:46
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 27, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 4117d3a to 3d5cd61 Compare September 27, 2021 06:47
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 27, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 3d5cd61 to 48519d8 Compare September 27, 2021 06:51
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 27, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch 2 times, most recently from f0886b6 to 63b3b7b Compare September 29, 2021 03:20
@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 58e852a to 6f9eb26 Compare September 29, 2021 06:07
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 29, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 6f9eb26 to 69f2b09 Compare September 29, 2021 06:20
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 29, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 69f2b09 to 7f15609 Compare September 29, 2021 06:25
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 29, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 7f15609 to 6427c02 Compare September 29, 2021 06:28
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 29, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch 2 times, most recently from 6928fb5 to 9b0f551 Compare September 29, 2021 06:33
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 29, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 9b0f551 to 29cbe17 Compare September 29, 2021 06:43
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 29, 2021

/run-all-tests

1 similar comment
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 29, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 29cbe17 to 80e29ff Compare September 29, 2021 06:51
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 29, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 80e29ff to 8126f07 Compare September 29, 2021 07:31
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 29, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 8126f07 to aeed7d5 Compare September 29, 2021 07:39
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 29, 2021

/run-all-tests

@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from aeed7d5 to 1ce1dda Compare September 29, 2021 07:51
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 29, 2021

/run-all-tests

@marsishandsome
ignore ci test with TxnKV
a1fa92e 
Signed-off-by: marsishandsome <marsishandsome@gmail.com>
@marsishandsome marsishandsome force-pushed the feature/fix-ci-with-txnkv branch from 1ce1dda to a1fa92e Compare September 29, 2021 07:58
@marsishandsome
Copy link
Collaborator Author

marsishandsome commented Sep 29, 2021

/run-all-tests

@marsishandsome marsishandsome changed the title [WIP] optimize ci test fix CI failed because ttl-enable TiKV cluster cannot support TxnKV Sep 29, 2021
@marsishandsome marsishandsome merged commit dcea3b3 into tikv:master Sep 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andylokandy andylokandy andylokandy approved these changes

Assignees

No one assigned

Labels

status/LGT1

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

CI failed because ttl-enable TiKV cluster cannot support TxnKV

3 participants

@marsishandsome @andylokandy @ti-srebot