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

Daily run of sql-db-modules in native doesn't finish in 6 hours #1171

Closed
rsvoboda opened this issue Apr 13, 2023 · 5 comments
Closed

Daily run of sql-db-modules in native doesn't finish in 6 hours #1171

rsvoboda opened this issue Apr 13, 2023 · 5 comments
Assignees
Labels
priority/urgent Issue that needs to be resolved by next RHBQ release.

Comments

@rsvoboda
Copy link
Member

Daily run of sql-db-modules in native doesn't finish in 6 hours

Example: https://github.com/quarkus-qe/quarkus-test-suite/actions/runs/4684604273/jobs/8301175364

My initial bet goes to compatibility mode tests

@rsvoboda rsvoboda added the priority/urgent Issue that needs to be resolved by next RHBQ release. label Apr 13, 2023
@rsvoboda
Copy link
Member Author

Another topic for investigation is time used to build the native image

GraalVM seems to be faster than Mandrel

@rsvoboda rsvoboda self-assigned this Apr 13, 2023
@rsvoboda
Copy link
Member Author

GraalVM is faster than Mandrel (at least when using container build): quarkusio/quarkus#32618

@rsvoboda
Copy link
Member Author

rsvoboda commented Apr 13, 2023

From https://github.com/quarkus-qe/quarkus-test-suite/actions/runs/4672855488/jobs/8275701492:

[INFO] Quarkus QE TS: Parent .............................. SUCCESS [  9.801 s]
[INFO] Quarkus QE TS: Environment info .................... SUCCESS [  4.057 s]
[INFO] Quarkus QE TS: Hibernate exploratory ............... SUCCESS [06:13 min]
[INFO] Quarkus QE TS: SQL Database: Application ........... SUCCESS [  01:19 h]
[INFO] Quarkus QE TS: SQL Database: Application Compatibility SUCCESS [  01:17 h]
[INFO] Quarkus QE TS: SQL Database: Multiple Persistence Units SUCCESS [06:23 min]
[INFO] Quarkus QE TS: SQL Database: Panache + Flyway ...... SUCCESS [15:27 min]
[INFO] Quarkus QE TS: SQL Database: RESTEasy Reactive + Rest Data Panache SUCCESS [52:57 min]
[INFO] Quarkus QE TS: SQL Database: Vert.x SQL ............ SUCCESS [10:52 min]
[INFO] Quarkus QE TS: SQL Database: Hibernate Reactive .... SUCCESS [17:39 min]
[INFO] Quarkus QE TS: SQL Database: Reactive Vanilla ...... SUCCESS [03:56 min]
[INFO] Quarkus QE TS: Hibernate FullText Search ........... SUCCESS [22:39 min]
[INFO] Quarkus QE TS: SQL Database: Narayana-transactions . SUCCESS [50:35 min]

@rsvoboda
Copy link
Member Author

Proposed changes to the compatibility module. Other modules would need to reduce the coverage on supported databases and that's a step I don't want to take atm.

@michalvavrik
Copy link
Member

We haven't experienced this for a long time. There is definitely room for improvements, but that is for all the modules, not just the database ones.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority/urgent Issue that needs to be resolved by next RHBQ release.
Projects
None yet
Development

No branches or pull requests

2 participants