----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63955/#review191538 -----------------------------------------------------------
Ship it! Ship It! - Robert Nettleton On Nov. 20, 2017, 5:06 p.m., Myroslav Papirkovskyy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63955/ > ----------------------------------------------------------- > > (Updated Nov. 20, 2017, 5:06 p.m.) > > > Review request for Ambari, Andrew Onischuk, Jonathan Hurley, Nate Cole, > Robert Nettleton, and Sid Wagle. > > > Bugs: AMBARI-22480 > https://issues.apache.org/jira/browse/AMBARI-22480 > > > Repository: ambari > > > Description > ------- > > Blueprint validation should fail if LZO is enabled and GPL license is not > accepted on server setup. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintResourceProvider.java > 1c127c080cd33314e47bcfff63e0c1e6d58937bc > > ambari-server/src/main/java/org/apache/ambari/server/topology/Blueprint.java > c9dc7281a092ff9f22551c9f8fc64ebca15fda36 > > ambari-server/src/main/java/org/apache/ambari/server/topology/BlueprintImpl.java > 1d917e0940b1ba5988a740005017c9d228e46f96 > > ambari-server/src/main/java/org/apache/ambari/server/topology/BlueprintValidator.java > 206d161fc694f67ae982ef668c5057b4bacec59b > > ambari-server/src/main/java/org/apache/ambari/server/topology/BlueprintValidatorImpl.java > 1aaa5ce66d1a5385018e4f29c92441e157c2be02 > > ambari-server/src/main/java/org/apache/ambari/server/topology/GPLLicenseNotAcceptedException.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/topology/BlueprintImplTest.java > fdbc0807788377f689e19200be9d6cdc2c992690 > > ambari-server/src/test/java/org/apache/ambari/server/topology/BlueprintValidatorImplTest.java > ea6bee314192f0665d4df652cf0c401e0218026f > > > Diff: https://reviews.apache.org/r/63955/diff/1/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Myroslav Papirkovskyy > >
