> On Dec. 9, 2015, 3:41 p.m., Nate Cole wrote: > > ambari-server/src/main/java/org/apache/ambari/server/checks/InstallPackagesCheck.java, > > line 68 > > <https://reviews.apache.org/r/41115/diff/2/?file=1156722#file1156722line68> > > > > Failed on is dependent on the check type (cluster, host, service). > > This check is a cluster one, so this should be empty or the cluster name. > > > > In either case, the call would be > > prerequisiteCheck.getFailedOn().add(...)
Will fix. - Alejandro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41115/#review109528 ----------------------------------------------------------- On Dec. 9, 2015, 2:27 a.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41115/ > ----------------------------------------------------------- > > (Updated Dec. 9, 2015, 2:27 a.m.) > > > Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, Jonathan > Hurley, Nate Cole, and Swapan Shridhar. > > > Bugs: AMBARI-14281 > https://issues.apache.org/jira/browse/AMBARI-14281 > > > Repository: ambari > > > Description > ------- > > If a cluster has version x (e.g., HDP 2.3.2.0-2950) as CURRENT, and the user > registers a Repository Version such as 2.3.xx that uses different build URLs > but results in the same build number, then installing packages will be a > no-op. > This means that no new version will be detected using the "deltas" method and > the repo_version entity will keep the "version" field without a build number, > and the Cluster Version entity will transition into INSTALLED (since > technically it is already installed). > > This incorrect setup will allow the user to start a RU/EU, which will > immediately fail since the Repo Version has no build number, which is needed > by hdp-select. > > The fix is to add a PreCheck that ensures that all repos contain a build > number. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/checks/InstallPackagesCheck.java > 15b6140 > > ambari-server/src/test/java/org/apache/ambari/server/checks/InstallPackagesCheckTest.java > 56aa1f3 > > Diff: https://reviews.apache.org/r/41115/diff/ > > > Testing > ------- > > mvn clean test > Tested that a build must contain a build number before starting RU/EU. > > > Thanks, > > Alejandro Fernandez > >
