You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/03/02 03:17:39 UTC

[GitHub] [accumulo] ctubbsii opened a new pull request #2537: Remove log4j2 config file from accumuo-test jar

ctubbsii opened a new pull request #2537:
URL: https://github.com/apache/accumulo/pull/2537


   This change removes the log4j2-test.properties file from the
   accumulo-test.jar, but keeps it on the class path for the unit and
   integration tests in the test module. This ensures our tests log useful
   information during the build testing, but avoids leaking that log4j
   configuration into the class path in other situations where the
   accumulo-test.jar might be used for other kinds of testing.
   
   This fixes #2527
   
   Also move the shellit.shellit file to complete #2497 and remove the last
   item from the root of our jars.
   
   Also tweak the system property configuration of maven-failsafe-plugin in
   the root pom.xml to ensure the configuration there is applied even when
   the test module sets its own additional system properties for the
   plugin.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] EdColeman commented on pull request #2537: Remove log4j2 config file from accumuo-test jar

Posted by GitBox <gi...@apache.org>.
EdColeman commented on pull request #2537:
URL: https://github.com/apache/accumulo/pull/2537#issuecomment-1060939313


   Without the log4j-test.properties file I am not seeing logs if I run tests in an IDE.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] ctubbsii merged pull request #2537: Remove log4j2 config file from accumuo-test jar

Posted by GitBox <gi...@apache.org>.
ctubbsii merged pull request #2537:
URL: https://github.com/apache/accumulo/pull/2537


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] ctubbsii commented on pull request #2537: Remove log4j2 config file from accumuo-test jar

Posted by GitBox <gi...@apache.org>.
ctubbsii commented on pull request #2537:
URL: https://github.com/apache/accumulo/pull/2537#issuecomment-1061817465


   > Without the log4j-test.properties file I am not seeing logs if I run tests in an IDE.
   
   Thanks @EdColeman . Fixed differently in #2548 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org