-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53373/
-----------------------------------------------------------

Review request for Ambari, Dmitro Lisnichenko and Nate Cole.


Bugs: AMBARI-18685
    https://issues.apache.org/jira/browse/AMBARI-18685


Repository: ambari


Description
-------

Unit tests are in progress, but seeing as though this is a feature branch, I 
didn't think I needed to wait to post the review. I mostly wanted feedback on 
the approach before I invested time in the tests.

Augment the API of creating an Upgrade with the following:

```
POST api/v1/clusters/c1/upgrades
{
  "Upgrade": {
    "repository_version": "2.5.0.0-965",
    "upgrade_type": "HOST_ORDERED",     <---------- new value other than 
ROLLING/NON_ROLLING
    "skip_failures": "false",
    "skip_prerequisite_checks": "false",
    "skip_manual_verification": "false",
    "host_order": [            <------ new
       {
         "hosts": [ "c6401.ambari.apache.org, "c6402.ambari.apache.org", 
"c6403.ambari.apache.org" ],
         "service_checks": ["ZOOKEEPER"]
       },
       {
         "hosts": [ "c6404.ambari.apache.org, "c6405.ambari.apache.org"],
         "service_checks": ["ZOOKEEPER", "KAFKA"]
       }
    ]
  }
}
```

{{skip_failures}} must be {{false}} when the upgrade_type is HOST_ORDERED
{{skip_manual_verification}} must be omitted or {{false}} when upgrade_type is 
HOST_ORDERED


Diffs
-----

  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
 d83aaa2 
  
ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/HostOrderGrouping.java
 9c75344 
  
ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/HostOrderItem.java
 PRE-CREATION 

Diff: https://reviews.apache.org/r/53373/diff/


Testing
-------

PENDING


Thanks,

Jonathan Hurley

Reply via email to