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

integration-tests/kafka-oauth-keycloak failing in native #36653

Closed
gsmet opened this issue Oct 24, 2023 · 7 comments · Fixed by #36657
Closed

integration-tests/kafka-oauth-keycloak failing in native #36653

gsmet opened this issue Oct 24, 2023 · 7 comments · Fixed by #36657

Comments

@gsmet
Copy link
Member

gsmet commented Oct 24, 2023

I saw it failing in several builds with the following error:

Error: Unsupported features in 2 methods
Detailed message:
Error: Discovered unresolved method during parsing: io.strimzi.kafka.oauth.jsonpath.JsonPathQuery.apply(com.fasterxml.jackson.databind.JsonNode). This error is reported at image build time because class io.strimzi.kafka.oauth.common.PrincipalExtractor is registered for linking at image build time by command line and command line.
Error encountered while parsing io.strimzi.kafka.oauth.common.PrincipalExtractor.getPrincipal(PrincipalExtractor.java:81) 
Parsing context:
   at io.strimzi.kafka.oauth.common.TokenIntrospection.introspectAccessToken(TokenIntrospection.java:48)
   at io.strimzi.kafka.oauth.common.OAuthAuthenticator.post(OAuthAuthenticator.java:483)
   at io.strimzi.kafka.oauth.common.OAuthAuthenticator.loginWithClientSecret(OAuthAuthenticator.java:167)
   at io.strimzi.kafka.oauth.client.JaasClientOauthLoginCallbackHandler.handleCallback(JaasClientOauthLoginCallbackHandler.java:284)
   at root method.(Unknown Source)

Error: Discovered unresolved method during parsing: io.strimzi.kafka.oauth.jsonpath.JsonPathQuery.parse(java.lang.String). This error is reported at image build time because class io.strimzi.kafka.oauth.common.PrincipalExtractor is registered for linking at image build time by command line and command line.
Error encountered while parsing io.strimzi.kafka.oauth.client.JaasClientOauthLoginCallbackHandler.configure(JaasClientOauthLoginCallbackHandler.java:89) 
@quarkus-bot
Copy link

quarkus-bot bot commented Oct 24, 2023

/cc @alesj (kafka), @cescoffier (kafka), @ozangunalp (kafka), @pedroigor (keycloak), @sberyozkin (keycloak)

@ozangunalp
Copy link
Contributor

ozangunalp commented Oct 24, 2023

I'll probably have to rewrite this IT in order to fix this. Will look at it today or tomorrow.

Edit: I'll try a quick fix first

@zakkak
Copy link
Contributor

zakkak commented Oct 24, 2023

FTR the failures started after merging #36462

@gsmet
Copy link
Member Author

gsmet commented Oct 24, 2023

Yeah, I think we need to revert this change. We had a few failures with Keycloak not starting and I though it was that when I scanned the CI feedback.

I'm a bit puzzled by the issue though as I still see the method in the updated library (and all classes come from the same library so I don't see how they could be out of sync) so I'm not sure what's going on.

@gsmet
Copy link
Member Author

gsmet commented Oct 24, 2023

I created #36657 to revert the change.

@ozangunalp
Copy link
Contributor

kafka-oauth-common pulls the jayway json-path but kafka-oauth-client excludes it.
I think with the version 0.14.0 the usage of json-path is in the nominal path.

I agree that we need to revert this.

@ozangunalp
Copy link
Contributor

Created an issue in upstream kafka-oauth-client. But I still need to rewrite this IT.

@quarkus-bot quarkus-bot bot added this to the 3.6 - main milestone Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants