Skip to content
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.

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

Bump DB2 Driver to 12.1.0.0 #44752

Closed
wants to merge 1 commit into from
Closed

Conversation

gastaldi
Copy link
Contributor

@gastaldi gastaldi commented Nov 26, 2024

It also uses the jakarta.transaction packages, so no need to transform it

@gastaldi gastaldi requested review from gsmet and Sanne November 26, 2024 17:44
@quarkus-bot quarkus-bot bot added area/dependencies Pull requests that update a dependency file area/jdbc Issues related to the JDBC extensions labels Nov 26, 2024
@gsmet
Copy link
Member

gsmet commented Nov 26, 2024

You were able to compile to native? The Dependabot update was complaining?

@gastaldi gastaldi marked this pull request as draft November 26, 2024 18:00
@gastaldi
Copy link
Contributor Author

You were able to compile to native? The Dependabot update was complaining?

I was waiting for CI to do that because there is no support for DB2 DevService in arm64:

[ERROR] DOCKER> I/O Error [Unable to pull 'icr.io/db2_community/db2:11.5.9.0' from registry 'icr.io' : {"message":"no image found in manifest list for architecture "arm64", variant "v8", OS "linux""} (Internal Server Error: 500)]

@gsmet
Copy link
Member

gsmet commented Nov 26, 2024

You will probably end up with the error in #44665 (see my comment).

Copy link

github-actions bot commented Nov 26, 2024

🎊 PR Preview e77760f has been successfully built and deployed to https://quarkus-pr-main-44752-preview.surge.sh/version/main/guides/

  • Images of blog posts older than 3 months are not available.
  • Newsletters older than 3 months are not available.

@gastaldi gastaldi marked this pull request as ready for review November 26, 2024 19:02
@gastaldi
Copy link
Contributor Author

@gsmet it passes now

@gastaldi gastaldi force-pushed the bump_db2 branch 2 times, most recently from 34ecb90 to bcadf36 Compare November 26, 2024 19:02
It also uses the jakarta.transaction packages, so no need to transform it

This comment has been minimized.

@gastaldi
Copy link
Contributor Author

@gsmet I found a nicer solution

Copy link

quarkus-bot bot commented Nov 26, 2024

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 6fb905a.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

Warning

There are other workflow runs running, you probably need to wait for their status before merging.

@gastaldi
Copy link
Contributor Author

Now it fails with:

Caused by: java.lang.UnsupportedOperationException: 'TCP_KEEPIDLE' not supported
	at java.base@21.0.5/sun.nio.ch.NioSocketImpl.setOption(NioSocketImpl.java:959)
	at java.base@21.0.5/java.net.DelegatingSocketImpl.setOption(DelegatingSocketImpl.java:144)
	at java.base@21.0.5/java.net.Socket.setOption(Socket.java:2009)
	at com.ibm.db2.jcc.t4.w.a(w.java:76)
	at com.ibm.db2.jcc.t4.az.a(az.java:441)
	... 21 more

It's impossible to fix it without the sources, so I'll leave it to someone who knows the JDBC driver better to perform this update.

Closing

@gastaldi gastaldi closed this Nov 26, 2024
@quarkus-bot quarkus-bot bot added the triage/invalid This doesn't seem right label Nov 26, 2024
@gastaldi gastaldi deleted the bump_db2 branch November 26, 2024 23:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dependencies Pull requests that update a dependency file area/jdbc Issues related to the JDBC extensions triage/invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants