Skip to content

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 Sanne and gsmet 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).

@github-actions
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
@quarkus-bot

This comment has been minimized.

@gastaldi
Copy link
Contributor Author

@gsmet I found a nicer solution

@quarkus-bot
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
@Sanne
Copy link
Member

Sanne commented Nov 27, 2024

Just out of curiousity, what did you do to get the TCP_KEEPIDLE error?

@gastaldi
Copy link
Contributor Author

@Sanne I bumped the devservice image to 12.1.0.0 too, do you think it may be related?

@Sanne
Copy link
Member

Sanne commented Nov 28, 2024

I don't know about the devservice.. I couldn't try this out, just curious how you got the TCP_KEEPIDLE : is that in native only or affecting JVM mode as well?

@gastaldi
Copy link
Contributor Author

@gsmet
Copy link
Member

gsmet commented Nov 28, 2024

I wonder if they somehow get this value by reflection cause it's new in Java 21+ and thus we would need to add it.

@Sanne
Copy link
Member

Sanne commented Nov 28, 2024

ah if it's just native then I might have a hunch related to class initialization. I'll try to look into it but no promises

@Sanne
Copy link
Member

Sanne commented Nov 28, 2024

I've found this GralVM issue which looks related and has a possible solution, but it didn't work for me in this case :

Weird, I've asked the Mandrel team for suggestions.

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.

3 participants