----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45544/#review126308 -----------------------------------------------------------
Ship it! Ship It! - Oliver Szabo On March 31, 2016, 2:28 p.m., Sebastian Toader wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45544/ > ----------------------------------------------------------- > > (Updated March 31, 2016, 2:28 p.m.) > > > Review request for Ambari, Daniel Gergely, Oliver Szabo, and Sandor Magyari. > > > Bugs: AMBARI-15643 > https://issues.apache.org/jira/browse/AMBARI-15643 > > > Repository: ambari > > > Description > ------- > > *Problem* > *After the cluster creation template posted to Ambari (via the REST Api) to > provision a cluster using Blueprints is accepted for processing Ambari > returns in the response the url of the request through which the > status/progress of the cluster creation can be tracked. > When querying the status of the request it erroneously returns "COMPLETED" > instead of returning "PENDING" until the first task of the cluster creation > is scheduled to be executed on one of the cluster nodes. Once at least one > task is scheduled to be executed the status of the request should change to > "IN_PROGRESS". The "COMPLETED" state should be reached when all tasks > completed succesfully. > > *Resolution* > Make distinction between requests initiated from the UI and Blueprints. If > the request is a request for Blueprint provisioned clusters and there are no > host level tasks available than the request status derivation logic to > default to PENDING. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestResourceProvider.java > d00ce58 > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/RequestResourceProviderTest.java > 4a76a1c > > Diff: https://reviews.apache.org/r/45544/diff/ > > > Testing > ------- > > Manual testing and unit tests: > > Results : > > Tests run: 3543, Failures: 0, Errors: 0, Skipped: 36 > > > Thanks, > > Sebastian Toader > >