-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
Support EAP8 with SAML Adapter Galleon Feature Pack #22824
Conversation
<wildfly.build-tools.version>1.2.13.Final</wildfly.build-tools.version> | ||
<wildfly.core.version>19.0.1.Final</wildfly.core.version> | ||
<!-- Upstream WildFly Versions --> | ||
<upstream.wildfly.version>29.0.0.Final</upstream.wildfly.version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a minor note that WildFly 29.0.1.Final was released last week so this could be updated and the Core version below could also be updated to the corresponding 21.1.1.Final version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did it to match EAP8 and ease alignment process.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok makes sense, thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pskopek Thanks very much! This looks good to me, I've only added a minor comment about the latest WildFly and Core versions.
@jfdenise Just FYI, the SAML adapter fixes have been incorporated in this PR. |
@@ -30,13 +30,13 @@ | |||
<name>App Server - Wildfly</name> | |||
|
|||
<properties> | |||
<wildfly.version>23.0.2.Final</wildfly.version> | |||
<wildfly.server.version>23.0.2.Final</wildfly.server.version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be updated to a later version?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed with @pskopek, this is about the test suite and could/should IMHO be a follow up issue.
16824db
to
09ce401
Compare
09ce401
to
69f11b8
Compare
Unreported flaky test detectedIf the below flaky tests below are affected by the changes, please review and update the changes accordingly. Otherwise, a maintainer should report the flaky tests prior to merging the PR. org.keycloak.testsuite.admin.concurrency.ConcurrencyTest#testAllConcurrently
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unreported flaky test detected, please review
c441e10
to
93fc9b9
Compare
Signed-off-by: Peter Skopek <pskopek@redhat.com>
…buteDefinition#getAttributeMarshaller method Closes keycloak#22593 Signed-off-by: Peter Skopek <pskopek@redhat.com>
Signed-off-by: Peter Skopek <pskopek@redhat.com>
…dules Signed-off-by: Peter Skopek <pskopek@redhat.com>
93fc9b9
to
2dd9e2c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pskopek LGTM from Operator and Quarkus dist perspective. org.wildfly.common:wildfly-common
remained on 1.5.4.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - it's quite unrelated to storage.
Dependencies used in SAML Adapter Galleon Feature Pack need to be aligned with EAP8.
Closes #22823
Closes #22593
Closes #22623
Closes #22172