> On March 7, 2016, 8:28 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1.2.2/package/scripts/kafka.py, > > line 134 > > <https://reviews.apache.org/r/44445/diff/1/?file=1282832#file1282832line134> > > > > Why is the owner root?
That's just a read-only config file, as in most service scripts we give ownership over it to root . - Dmitro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44445/#review122332 ----------------------------------------------------------- On March 7, 2016, 1:31 p.m., Dmitro Lisnichenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44445/ > ----------------------------------------------------------- > > (Updated March 7, 2016, 1:31 p.m.) > > > Review request for Ambari and Vitalyi Brodetskyi. > > > Bugs: AMBARI-15315 > https://issues.apache.org/jira/browse/AMBARI-15315 > > > Repository: ambari > > > Description > ------- > > {code} > log4j:ERROR Could not read configuration file from URL > [file:/usr/hdp/current/kafka-broker/bin/../config/tools-log4j.properties]. > java.io.FileNotFoundException: > /usr/hdp/current/kafka-broker/bin/../config/tools-log4j.properties (No such > file or directory) > at java.io.FileInputStream.open(Native Method) > at java.io.FileInputStream.<init>(FileInputStream.java:146) > at java.io.FileInputStream.<init>(FileInputStream.java:101) > at > sun.net.www.protocol.file.FileURLConnection.connect(FileURLConnection.java:90) > at > sun.net.www.protocol.file.FileURLConnection.getInputStream(FileURLConnection.java:188) > at > org.apache.log4j.PropertyConfigurator.doConfigure(PropertyConfigurator.java:557) > at > org.apache.log4j.helpers.OptionConverter.selectAndConfigure(OptionConverter.java:526) > at org.apache.log4j.LogManager.<clinit>(LogManager.java:127) > at org.slf4j.impl.Log4jLoggerFactory.getLogger(Log4jLoggerFactory.java:66) > at org.slf4j.LoggerFactory.getLogger(LoggerFactory.java:277) > at org.slf4j.LoggerFactory.getLogger(LoggerFactory.java:288) > at org.apache.kafka.common.utils.Utils.<clinit>(Utils.java:54) > at kafka.utils.CoreUtils$.createObject(CoreUtils.scala:221) > at kafka.admin.AclCommand$.main(AclCommand.scala:61) > at kafka.admin.AclCommand.main(AclCommand.scala) > log4j:ERROR Ignoring configuration file > [file:/usr/hdp/current/kafka-broker/bin/../config/tools-log4j.properties]. > log4j:WARN No appenders could be found for logger (kafka.server.KafkaConfig). > log4j:WARN Please initialize the log4j system properly. > log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more > info. > Following is list of acls for resource: Topic:arpit > {code} > > {code} > [kafka@hcube2-1n01 kafka-broker]$ ls -latr /usr/hdp/current/kafka-broker/ > total 36 > -rw-r--r--. 1 root root 162 Feb 4 11:55 NOTICE > -rw-r--r--. 1 root root 11358 Feb 4 11:55 LICENSE > drwxr-xr-x. 3 root root 4096 Feb 4 18:13 bin > lrwxrwxrwx. 1 kafka hadoop 31 Feb 4 18:13 config -> > /usr/hdp/2.4.0.0-160/kafka/conf > drwxr-xr-x. 4 root root 32 Feb 4 18:13 doc > drwxr-xr-x. 3 root root 16 Feb 4 18:13 examples > lrwxrwxrwx. 1 root root 14 Feb 4 18:13 logs -> /var/log/kafka > drwxr-xr-x. 3 root root 4096 Feb 4 18:13 libs > lrwxrwxrwx. 1 root root 14 Feb 4 18:13 pids -> /var/run/kafka > drwxr-xr-x. 32 root root 4096 Feb 4 18:22 .. > lrwxrwxrwx. 1 root root 24 Feb 4 19:12 conf -> > /etc/kafka/2.4.0.0-160/0 > drwxr-xr-x. 6 root root 4096 Feb 4 19:12 . > [kafka@hcube2-1n01 kafka-broker]$ ls /usr/hdp/2.4.0.0-160/kafka/conf > kafka_client_jaas.conf kafka-env.sh kafka_jaas.conf log4j.properties > server.properties > {code} > > Seems like ambari did not setup the configs correctly. Not sure why we have > config and conf links. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1.2.2/package/scripts/kafka.py > b986607 > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1.2.2/package/templates/tools-log4j.properties.j2 > PRE-CREATION > > Diff: https://reviews.apache.org/r/44445/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Dmitro Lisnichenko > >