oscerd commented on PR #26969: URL: https://github.com/apache/camel/pull/26969#issuecomment-5867258194
Thanks for tracking this down, @apupier. The cause turned out to be a merge race rather than CAMEL-24742 itself: #26670 added `OpaSecurityPolicyWasmTest`, which loads `classpath:authz.wasm`, one minute before #26677 deleted that file. Each was green against its own base, but together on `main` every test in that class fails at context startup. Those 8 tests are the only camel-opa failures in Jenkins build #2223. #26987 fixes it forward: the three tests that evaluate a policy move to a new `OpaSecurityPolicyWasmIT`, which compiles `authz.rego` with `OpaWasmBundleBuilder` the way `OpaWasmIT` does, and the five startup-failure tests stay unit tests. It only touches camel-opa test code. If that works for you, could we close this revert once #26987 is green? The revert would put the compiled binaries back in `src/test/resources`, and CAMEL-24742 removed them so they can't drift from the Rego or end up in the source release. If you'd rather merge the revert to unblock Jenkins right away, that's fine too, and I'll rework the change on top of it. _Claude Code on behalf of @oscerd_ -- 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]
