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-5781] Remove log4j 1.x support. This removes the module, the … #5140

Merged
merged 1 commit into from Jun 30, 2022

Conversation

jamezp
Copy link
Member

@jamezp jamezp commented Jun 24, 2022

…ability to use configuration files in a deployment and the ability to use an Appender as a custom-handler.

https://issues.redhat.com/browse/WFCORE-5781

wildfly/wildfly#15698 may need to be merged first.

…ability to use configuration files in a deployment and the ability to use an Appender as a custom-handler.

https://issues.redhat.com/browse/WFCORE-5781
Signed-off-by: James R. Perkins <jperkins@redhat.com>
@wildfly-ci
Copy link

Core -> Full Integration Build 11699 outcome was FAILURE using a merge of bca19c8
Summary: Exit code 1 (Step: Build & test full (Maven)) (new) Build time: 00:02:05

@wildfly-ci
Copy link

Core -> Full Integration Build 11538 outcome was FAILURE using a merge of bca19c8
Summary: Exit code 1 (Step: Build & test full (Maven)) (new) Build time: 00:02:24

@wildfly-ci
Copy link

Core -> WildFly Preview Integration Build 11703 outcome was FAILURE using a merge of bca19c8
Summary: Exit code 1 (Step: Build & test full (Maven)) (new) Build time: 00:02:31

@github-actions github-actions bot added the deps-changed Dependencies have been checked, and there are changes highlighted in a comment label Jun 24, 2022
@github-actions
Copy link

Dependency Tree Analyzer Output:

Removed Dependencies:

  • org.jboss.logmanager:log4j-jboss-logmanager:jar:1.3.0.Final:compile

CC @wildfly/prod

@yersan
Copy link
Collaborator

yersan commented Jun 30, 2022

/retest

Copy link
Collaborator

@yersan yersan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is ready to go, there was an issue reporting job status to the GitHub UI, but all the Jobs have been tested and passed, see:

https://ci.wildfly.org/project.html?projectId=WildFlyCore_PullRequest&branch_WildFlyCore_PullRequest=pull%2F5140

And the integration jobs linked on the WildFly Full counterpart

@yersan yersan added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Jun 30, 2022
@yersan yersan merged commit e1bf234 into wildfly:main Jun 30, 2022
@jamezp jamezp deleted the WFCORE-5781 branch July 5, 2022 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-changed Dependencies have been checked, and there are changes highlighted in a comment ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
3 participants