DanielLeens opened a new pull request, #11233:
URL: https://github.com/apache/seatunnel/pull/11233

   ### Purpose
   Fix an unrelated SQLServer CDC E2E CI failure where the test container 
downloads the SQLServer JDBC driver with `wget` from Maven Central during 
runtime. That path can fail with transient TLS/network errors such as `Unable 
to establish SSL connection`, even though Maven has already resolved the same 
driver for the module.
   
   ### Changes
   - Reuse the Maven-resolved `mssql-jdbc-9.4.1.jre8.jar` from the local 
repository.
   - Copy the driver into `/tmp/seatunnel/plugins/SqlServer-CDC/lib` via 
Testcontainers instead of downloading it inside the container.
   - Keep the driver version unchanged and do not modify `pom.xml`.
   
   ### Validation
   - `./mvnw -pl 
seatunnel-e2e/seatunnel-connector-v2-e2e/connector-cdc-sqlserver-e2e 
spotless:apply -nsu -Dmaven.gitcommitid.skip=true`
   - 
`JAVA_HOME=/Library/Java/JavaVirtualMachines/jdk1.8.0_171.jdk/Contents/Home 
./mvnw -B -pl :connector-cdc-sqlserver-e2e -DskipTests -DskipITs=true 
-Dlicense.skipAddThirdParty=true -Dskip.ui=true -Dskip.spotless=true 
-Dmaven.gitcommitid.skip=true -nsu test-compile`
   
   Full Testcontainers E2E was not run locally because Docker daemon is not 
available on this machine (`Cannot connect to the Docker daemon`). 
`seatunnel-engine-ui` was not built because this change only touches a 
connector E2E Java test.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to