Skip to content

[CI][E2E] Stabilize SQL Server XA setup for schema evolution#11137

Draft
DanielLeens wants to merge 2 commits into
apache:devfrom
DanielLeens:dev-sqlserver-xa-init-20260620
Draft

[CI][E2E] Stabilize SQL Server XA setup for schema evolution#11137
DanielLeens wants to merge 2 commits into
apache:devfrom
DanielLeens:dev-sqlserver-xa-init-20260620

Conversation

@DanielLeens

Copy link
Copy Markdown
Contributor

What's changed

  • switch the SQL Server container setup to the supported MSSQL_SA_PASSWORD variable and configure the MSDTC RPC/DTC ports required by containerized XA transactions
  • wait for a real sqlcmd login before running XA setup so the test does not race SQL Server upgrade work
  • fail fast when sp_configure or sp_sqljdbc_xa_install fails, and wait until xp_sqljdbc_xa_init_ex becomes visible before the exactly-once path starts
  • remove the invalid external user enabled setup step and relax the ready-log matcher so startup does not depend on a trailing newline

Validation

  • ./mvnw -nsu -pl seatunnel-e2e/seatunnel-connector-v2-e2e/connector-jdbc-e2e/connector-jdbc-e2e-ddl spotless:apply
  • ./mvnw -nsu -pl seatunnel-e2e/seatunnel-connector-v2-e2e/connector-jdbc-e2e/connector-jdbc-e2e-ddl -q -DskipTests verify
  • ./mvnw -nsu -pl seatunnel-e2e/seatunnel-connector-v2-e2e/connector-jdbc-e2e/connector-jdbc-e2e-ddl -DfailIfNoTests=false -Dtest=SqlServerSchemaChangeIT test
    • on /Volumes/disk2/work/myworkspace/tmp/seatunnel-dev-sqlserver-xa-init-20260620, the host filesystem exposed src/test/resources/docker/setup.sql to the MySQL container with unreadable permissions and the run stopped before the SQL Server path (Permission denied)
    • on /Users/stone/work/myworkspace/seatunnel_third with normal file permissions, MySQL initialization succeeded and SQL Server 2022 still failed under the local Docker Desktop memory cap before reaching the ready log (Available Physical Memory 89.32 mb, Error 701, Failed allocate pages)
  • seatunnel-engine-ui was skipped because this change does not touch that module
  • no modules or deployment artifacts were reused in this run

References

@github-actions github-actions Bot added the e2e label Jun 19, 2026
@davidzollo davidzollo marked this pull request as draft June 19, 2026 22:03
@davidzollo davidzollo changed the title [Test][E2E] Stabilize SQL Server XA setup for schema evolution [CI][E2E] Stabilize SQL Server XA setup for schema evolution Jun 19, 2026
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