----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39281/#review102482 -----------------------------------------------------------
Ship it! Ship It! - Alejandro Fernandez On Oct. 13, 2015, 5:23 p.m., Jonathan Hurley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39281/ > ----------------------------------------------------------- > > (Updated Oct. 13, 2015, 5:23 p.m.) > > > Review request for Ambari and Alejandro Fernandez. > > > Bugs: AMBARI-13406 > https://issues.apache.org/jira/browse/AMBARI-13406 > > > Repository: ambari > > > Description > ------- > > ``` > 2015-10-13 17:22:13,946 ERROR [qtp-client-87 - /api/v1/clusters/c1/upgrades] > handlers.BaseManagementHandler (CreateHandler.java:persist(65)) - Caught a > runtime exception while attempting to create a resource > java.lang.ClassCastException: > org.apache.ambari.server.state.stack.upgrade.RestartGrouping cannot be cast > to org.apache.ambari.server.state.stack.upgrade.Task > at > org.apache.ambari.server.state.stack.upgrade.Grouping$DefaultBuilder.add(Grouping.java:111) > ``` > > Caused by a grouping being bound to task. We need to change: > <upgrade> > <task xsi:type="restart" /> > </upgrade> > to > <upgrade> > <task xsi:type="restart-task" /> > </upgrade> > > > Diffs > ----- > > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml > 810423d > ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.3.xml > 2c7cc7c > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.3.xml > 4f57978 > > ambari-server/src/test/resources/stacks/HDP/2.2.0/upgrades/upgrade_test_skip_failures.xml > 467e63a > > Diff: https://reviews.apache.org/r/39281/diff/ > > > Testing > ------- > > Performed a rolling upgrade of a ZK cluster. Running through tests now as > well (actually, mostly looking for disabled tests that can be turned back on). > > > Thanks, > > Jonathan Hurley > >
