nikhiln64 commented on PR #6809: URL: https://github.com/apache/incubator-kie-drools/pull/6809#issuecomment-5024700917
@mariofusco I investigated the CI failures before asking you to consider merging - status so far: **What fails:** `KJarLoadingTest` (kie-ci, with+without domain) with `Cannot find KieModule: ...999-SNAPSHOT` at container init - a module *resolution* failure, before any class loading through the project classloaders happens. **What I could and couldn't prove locally:** in my local environment the test fails **identically with and without this change** (checked out the parent commit's wiring sources, reinstalled, reran) - so I can't reproduce a causal link locally; but my environment can't resolve the test KJar in either state, so I don't treat that as conclusive for CI. **One variable I could eliminate:** this branch was based on a Jul 14 main while the recently passing PRs run on current main (which since picked up e.g. the quarkus bump in #6808). I've rebased onto today's main and pushed, which should also re-trigger CI. If CI still shows the KJar failures on the fresh base, I'll treat that as real evidence of an interaction with this change and dig into the kie-ci module discovery path before this goes anywhere near a merge - I would not want it merged with an unexplained failure either. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
