----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51918/#review150300 -----------------------------------------------------------
LGTM, should we enable this by default? - Lior Zeno On Sept. 15, 2016, 5:01 p.m., Balázs Donát Bessenyei wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51918/ > ----------------------------------------------------------- > > (Updated Sept. 15, 2016, 5:01 p.m.) > > > Review request for Flume. > > > Bugs: FLUME-2993 > https://issues.apache.org/jira/browse/FLUME-2993 > > > Repository: flume-git > > > Description > ------- > > Flume does not currently support environment variable interpolation in the > properties file configuration. > > Enabling it would help with > * removing security credentials from config files > * help with copy-pastes in configuration files with multiple agents defined > > It is arguably a best practice to store (some) config in the environment > variables: https://12factor.net/config > > > Diffs > ----- > > c/flume-ng-doc/sphinx/FlumeUserGuide.rst 0fecee6 > c/flume-ng-node/pom.xml e33b566 > > c/flume-ng-node/src/main/java/org/apache/flume/node/PropertiesFileConfigurationProvider.java > b428c9e > > w/flume-ng-node/src/main/java/org/apache/flume/node/EnvVarResolverProperties.java > PRE-CREATION > > w/flume-ng-node/src/test/java/org/apache/flume/node/TestEnvVarResolverProperties.java > PRE-CREATION > w/flume-ng-node/src/test/resources/flume-conf-with-envvars.properties > PRE-CREATION > > Diff: https://reviews.apache.org/r/51918/diff/ > > > Testing > ------- > > mvn clean install runs successfully > > new unit tests were added > > > Thanks, > > Balázs Donát Bessenyei > >