obkv-table-client-rs icon indicating copy to clipboard operation
obkv-table-client-rs copied to clipboard

ci: start OB server and run all tests

Open tisonkun opened this issue 2 years ago • 4 comments

Try to run all tests in CI.

tisonkun avatar Oct 21 '23 15:10 tisonkun

cc @jiacai2050

tisonkun avatar Oct 21 '23 15:10 tisonkun

@IHEII Please have a review.

jiacai2050 avatar Oct 23 '23 07:10 jiacai2050

Thanks to @tisonkun. We are now considering adding CI to all our projects. Actually, the Rust client may be a project that adds CI later for four reasons:

  1. We could not implement an OceanBase Server in GitHub Docker now. (We are trying to provide a slim OB Server in docker, still in progress ref)
  2. ConfigUrl used to connect to the server is unstable now. (In progress)
  3. The Slim OB Server in Docker mentioned above is 4.x, which is incompatible with our client now(only works in 3.x, which means we will use more time to fix it)
  4. Tests should be adapted to CI

IHEII avatar Oct 23 '23 07:10 IHEII

Hi @tisonkun, I think it is time to restart ci now. Because we have implemented docker for the observer and we have already supported 4.x observer now. I tested observer docker in Java client a month ago https://github.com/oceanbase/obkv-table-client-java/pull/68. And CI works well on Java client right now (https://github.com/oceanbase/obkv-table-client-java/pull/75/checks). Could you help to implement a CI check on the rust client? All we need to do is to run the docker(like testcontainers in Java, and parameters are in here) and try to run tests in tests/. For convenience, we may select some basic tests for CI.

IHEII avatar Jan 12 '24 09:01 IHEII

Closed as I don't have time and energy to nudge this PR.

tisonkun avatar Jun 21 '24 08:06 tisonkun