----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41115/#review109528 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/checks/InstallPackagesCheck.java (line 68) <https://reviews.apache.org/r/41115/#comment169095> 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(...) - Nate Cole On Dec. 8, 2015, 9:27 p.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41115/ > ----------------------------------------------------------- > > (Updated Dec. 8, 2015, 9:27 p.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 > >
