----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64809/#review194447 -----------------------------------------------------------
Ship it! Ship It! - Tim Thorpe On Dec. 22, 2017, 2:48 p.m., Di Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64809/ > ----------------------------------------------------------- > > (Updated Dec. 22, 2017, 2:48 p.m.) > > > Review request for Ambari and Tim Thorpe. > > > Bugs: AMBARI-22685 > https://issues.apache.org/jira/browse/AMBARI-22685 > > > Repository: ambari > > > Description > ------- > > Even when a user had remove say Diebian , SLES OS/repos from cluster > installation UI after selecting a stack, they still showed up on the Versions > tab after cluster was installed. > > > Diffs > ----- > > ambari-web/app/controllers/installer.js f49fc11 > ambari-web/app/controllers/wizard/step8_controller.js e64a09a > > > Diff: https://reviews.apache.org/r/64809/diff/1/ > > > Testing > ------- > > existing unit tests, patch a trunk cluster with the changes, remove some of > the OS from the stack page during cluster installation, run thru cluster > install. From Ambari web UI, on the Versions tab, check os/repos shown on the > cluster, verify only the ones selected during the installation time remain. > > > Thanks, > > Di Li > >
