You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@fluo.apache.org by GitBox <gi...@apache.org> on 2020/05/06 16:59:56 UTC
[GitHub] [fluo-muchos] arvindshmicrosoft opened a new issue #344: Muchos needs to be updated for new log4j2 config files
arvindshmicrosoft opened a new issue #344:
URL: https://github.com/apache/fluo-muchos/issues/344
Follow-up issue to track an observation raised during review of #343 - when accumulo-2.1.0-SNAPSHOT is used, variables like accumulo.log.dir are not being expanded properly and as a result the Accumulo log directory names are literally set as ${accumulo.log.dir} and so on. This is possibly related to the Accumulo changes to switch to [log4j2](https://github.com/apache/accumulo/pull/1528) so we need to handle those in Muchos for 2.1.0 and above.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [fluo-muchos] arvindshmicrosoft commented on issue #344: Muchos needs to be updated for new log4j2 config files
Posted by GitBox <gi...@apache.org>.
arvindshmicrosoft commented on issue #344:
URL: https://github.com/apache/fluo-muchos/issues/344#issuecomment-630572132
Oh you can totally take it over @keith-turner - I have not been able to spend much time on it. I'm un-assigning myself.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [fluo-muchos] ctubbsii commented on issue #344: Muchos needs to be updated for new log4j2 config files
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on issue #344:
URL: https://github.com/apache/fluo-muchos/issues/344#issuecomment-625156547
Could the root of this problem be that Muchos is attempting to override `accumulo-env.sh` with its own `ansible/roles/accumulo/templates/accumulo-env.sh`, which sets up its own `ansible/roles/accumulo/templates/log4j-service.properties`?
It's possible log4j2 is trying to do some backwards-compatible log4j1 config file parsing, since log4j2 jars are on the classpath for 2.1.0-SNAPSHOT, but not succeeding for some reason.
A good fix might update Muchos' version of `accumulo-env.sh` to be based on the one in the 2.1.0-SNAPSHOT, and similarly, use different log4j2 config files, closer to those in the 2.1.0-SNAPSHOT source.
Separate, but related, issue: Since Muchos is trying to override log configuration explicitly anyway, I think it would probably be better if Muchos' log4j settings just used the system logger, and instead of trying to output to `${accumulo.log.dir}`. Ideally, Muchos would only support systemd, and the config file would just log stuff to the console, which is automatically captured by journald, and it wouldn't use any file-based log settings at all. The file-based logging solution is really for out-of-the-box Accumulo installs... a curated deployment using something like Muchos can use smarter approaches to log collection, by leveraging the deployed environment's tooling for log collection maintained by the system admin (journald, syslog, etc.).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [fluo-muchos] keith-turner commented on issue #344: Muchos needs to be updated for new log4j2 config files
Posted by GitBox <gi...@apache.org>.
keith-turner commented on issue #344:
URL: https://github.com/apache/fluo-muchos/issues/344#issuecomment-630503588
@arvindshmicrosoft I am interested in fixing this, but I see you self assigned it. Do you still want to fix this or do you have any outstanding work on this?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [fluo-muchos] keith-turner closed issue #344: Muchos needs to be updated for new log4j2 config files
Posted by GitBox <gi...@apache.org>.
keith-turner closed issue #344:
URL: https://github.com/apache/fluo-muchos/issues/344
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org