----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45306/#review125319 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog222.java (line 196) <https://reviews.apache.org/r/45306/#comment188097> So creating the new service config version will automatically add slider-site to the list? ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog222.java (line 199) <https://reviews.apache.org/r/45306/#comment188096> Nit: Typo - SLIDE to SLIDER in the text. - Sumit Mohanty On March 24, 2016, 7:09 p.m., Vitalyi Brodetskyi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45306/ > ----------------------------------------------------------- > > (Updated March 24, 2016, 7:09 p.m.) > > > Review request for Ambari, Dmytro Sen and Myroslav Papirkovskyy. > > > Bugs: AMBARI-15568 > https://issues.apache.org/jira/browse/AMBARI-15568 > > > Repository: ambari > > > Description > ------- > > STR: Gateway: 172.22.74.38:8080 > 1)Deploy old version > 2)Make Ambari only Upgrade > 4)'ambari-server check-database' > 2016-03-23 10:22:31,276 INFO - ******************************* Check database > started ******************************* > 2016-03-23 10:22:56,229 ERROR - Required config(s): slider-client is(are) not > available for service SLIDER with service config version 1 in cluster cl1 > 2016-03-23 10:22:56,469 INFO - ******************************* Check database > completed ******************************* > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog222.java > 7601e4c > ambari-server/src/main/python/ambari_server/checkDatabase.py 2deca0a > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog222Test.java > 17d7e4e > > Diff: https://reviews.apache.org/r/45306/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Vitalyi Brodetskyi > >