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

Do not merge -- Combined PR #5324

Closed
wants to merge 3 commits into from
Closed

Do not merge -- Combined PR #5324

wants to merge 3 commits into from

Conversation

yersan
Copy link
Collaborator

@yersan yersan commented Dec 13, 2022

Combines:

#5313
#5323

@yersan yersan added the hold Do not merge this PR label Dec 13, 2022
@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label Dec 13, 2022
Signed-off-by: Flavia Rainone <frainone@redhat.com>
@fl4via
Copy link
Contributor

fl4via commented Dec 14, 2022

@yersan the regression in the deny uncovered methods testcase is fixed by Undertow 2.3.2.Final (that includes only undertow-io/undertow#1423 )
the #5313 PR is updated to upgrade to 2.3.2.Final instead of 2.3.1.Final

@yersan
Copy link
Collaborator Author

yersan commented Dec 14, 2022

Got it @fl4via, I'm going to update this one and kick it off again. Notice we are also getting failures on Windows - JDK 11 very often on the same tests. Those failures are unrelated to this PR. Lately, the CI has become to be less stable; we are still working around this.

@wildfly wildfly deleted a comment from wildfly-ci Dec 14, 2022
@wildfly wildfly deleted a comment from wildfly-ci Dec 14, 2022
@wildfly wildfly deleted a comment from wildfly-ci Dec 14, 2022
@wildfly wildfly deleted a comment from wildfly-ci Dec 15, 2022
@yersan yersan closed this Dec 16, 2022
@yersan yersan deleted the combined-pr branch December 16, 2022 11:17
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 hold Do not merge this PR
Projects
None yet
2 participants