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

[WFCORE-5169] upgraded to Apache Directory Server AM27 and Kerby #5848

Merged
merged 1 commit into from
Feb 14, 2024

Conversation

istudens
Copy link
Contributor

@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label Jan 26, 2024
@istudens
Copy link
Contributor Author

@darranl @fjuma can you review this please? thanks

@yersan yersan requested a review from fjuma January 29, 2024 09:21
@fjuma
Copy link
Contributor

fjuma commented Jan 31, 2024

Thank you very much for working on this @istudens!

Just wanted to ask if you've done a quick sanity check with IBM JDK just to make sure there are no new test failures with these changes (i.e., just to make sure the failures from WFCORE-5432 won't occur anymore with these new changes).

@istudens
Copy link
Contributor Author

istudens commented Feb 5, 2024

@fjuma Nope, I missed IBM JDK. I will test it.

@istudens
Copy link
Contributor Author

istudens commented Feb 6, 2024

@fjuma It looks good to me even with IBM JDK 11 (both /elytron and /testsuite/elytron tests).

@fjuma
Copy link
Contributor

fjuma commented Feb 6, 2024

Thanks for confirming that @istudens!

@fjuma fjuma requested a review from Skyllarr February 6, 2024 14:43
@yersan
Copy link
Collaborator

yersan commented Feb 9, 2024

CC @Skyllarr in case you missed the request for review

@Skyllarr
Copy link
Contributor

Oops I missed this, reviewing now

@Skyllarr
Copy link
Contributor

@istudens Thanks!

@yersan yersan added ready-for-merge This PR is ready to be merged and fulfills all requirements and removed ready-for-merge This PR is ready to be merged and fulfills all requirements labels Feb 13, 2024
@yersan
Copy link
Collaborator

yersan commented Feb 13, 2024

/retest

Let's give it another try after merging the Elytron upgrade

@yersan yersan added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Feb 14, 2024
@yersan yersan merged commit 6f1d674 into wildfly:main Feb 14, 2024
12 checks passed
@yersan
Copy link
Collaborator

yersan commented Feb 14, 2024

Thanks @fjuma @Skyllarr and @istudens

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-ok Dependencies have been checked, and there are no significant changes ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants