----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55396/#review161225 -----------------------------------------------------------
Ship it! Assuming that there's no concurrency issue - I'll mark it Ship It! - Jonathan Hurley On Jan. 10, 2017, 6:59 p.m., Nate Cole wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55396/ > ----------------------------------------------------------- > > (Updated Jan. 10, 2017, 6:59 p.m.) > > > Review request for Ambari, Dmitro Lisnichenko and Jonathan Hurley. > > > Bugs: AMBARI-19450 > https://issues.apache.org/jira/browse/AMBARI-19450 > > > Repository: ambari > > > Description > ------- > > On initial install, we are creating host_version records that may or may not > have any versionable components. As we add host components, check if there > are any versionable components left on the host. If there are none, mark the > entity with a repo state of NOT_REQUIRED. > > (Removing the host_version records seemed very dangerous to do at this stage > of the release, so I'll make a new jira for that). > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/events/listeners/upgrade/HostVersionOutOfSyncListener.java > dabd348 > > ambari-server/src/test/java/org/apache/ambari/server/events/listeners/upgrade/HostVersionOutOfSyncListenerTest.java > 3592668 > > Diff: https://reviews.apache.org/r/55396/diff/ > > > Testing > ------- > > Manual. Automated pending. > > > Thanks, > > Nate Cole > >