Skip to content

Conversation

yuanoOo
Copy link
Contributor

@yuanoOo yuanoOo commented Aug 28, 2025

What is the purpose of the change

This PR resolves the issue where OceanBase CDC integration tests were disabled in GitHub CI due to the unavailability of OceanBase Binlog Service docker image.

Brief change log

  • Add comprehensive e2e integration tests in `flink-cdc-source-e2e-tests`
  • Update test base classes to use Docker containers instead of external OceanBase instances
  • Add proper DDL test resources for OceanBase testing

Verifying this change

  • ✅ All existing OceanBase integration tests now run successfully in CI
  • ✅ Added new e2e tests to verify connector functionality end-to-end
  • ✅ Tests use containerized OceanBase instances, making them reproducible and CI-friendly

… CI with Docker image

- Remove @disabled annotations from OceanBase test classes
- Replace environment variable dependencies with obbinlog-ce:4.2.5-test container
- Add comprehensive e2e integration tests for oceanbase-cdc connector
@lvyanquan
Copy link
Contributor

You may need to double check for the failed case of OceanBaseE2eITCase.testOBBinlogCDC:196 in https://github.com/apache/flink-cdc/actions/runs/17577358020/job/50196816609?pr=4111#step:7:9145.

@yuanoOo
Copy link
Contributor Author

yuanoOo commented Sep 16, 2025

You may need to double check for the failed case of OceanBaseE2eITCase.testOBBinlogCDC:196 in https://github.com/apache/flink-cdc/actions/runs/17577358020/job/50196816609?pr=4111#step:7:9145.

OK, I tried to fix it.

import java.util.stream.Stream;

/** Test supporting different column charsets for OceanBase. */
public class OceanBaseCharsetITCase extends OceanBaseSourceTestBase {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm wondering why these test classes need to be added, since we already have the corresponding tests in the OceanBase module. If there are any unsupported changes, they should also be reflected in the tests of the OceanBase source.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants