> On June 12, 2016, 3:14 p.m., Stephan Erb wrote: > > specs/rpm/SOURCES/aurora-scheduler.sysconfig, lines 33-36 > > <https://reviews.apache.org/r/48606/diff/1/?file=1416269#file1416269line33> > > > > Should we adopt the same configuration schema also for Debian and > > Ubuntu? > > > > Using a Bash array is easier and more flexible than the list of > > environment variables that is currently used by the DEB packages. > > > > What do you think?
I'm a fan. That usually makes things easier to deal with in config management systems, too. Just make sure that it's safe to assume that the shell processing it is bash on the debian/ubuntu side. - Benjamin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48606/#review137203 ----------------------------------------------------------- On June 12, 2016, 3:08 p.m., Stephan Erb wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48606/ > ----------------------------------------------------------- > > (Updated June 12, 2016, 3:08 p.m.) > > > Review request for Aurora, Benjamin Staffin, Jake Farrell, and Kasisnu Singh. > > > Bugs: AURORA-1394 > https://issues.apache.org/jira/browse/AURORA-1394 > > > Repository: aurora-packaging > > > Description > ------- > > * Use identical scheduler default arguments > * Log to stderr rather than disk in all packages > * Use vagrant as the example role. It is available for all distributions > * Use the same clusters.json for all distributions > * Rename the clutername from main to example > > > Diffs > ----- > > specs/debian/aurora-executor.thermos.init > 9bc8675952bb21474617e99374fa20c452085fa6 > specs/debian/aurora-scheduler.default > 71d1625b5c2fbe17f68731c120e1c354c96c75d5 > specs/debian/clusters.json 29c2e0b61c6a2e945152ea3a39cee883696de2c1 > specs/rpm/SOURCES/aurora-scheduler.sysconfig > d812341cc44e1ae086132deb117e89ad92f3228b > specs/rpm/SOURCES/clusters.json 133013ba2ede037eb5c5e8e9255e6391cddf0d40 > test/deb/debian-jessie/README.md b84ffd9b92910c7cd1bc014f12a14a39857b4af9 > test/deb/ubuntu-trusty/README.md bb793410f1bb2bbb1231eb3ca0ad7f7d8ba14fc0 > test/rpm/centos-7/README.md bb34fb6b5dad0971226b4a9adb89d69e85c633a4 > > Diff: https://reviews.apache.org/r/48606/diff/ > > > Testing > ------- > > > Thanks, > > Stephan Erb > >
