ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Myroslav Papirkovskyy" <mpapyrkovs...@hortonworks.com>
Subject Re: Review Request 31192: Cannot apply new property set for slider-client through api
Date Tue, 03 Mar 2015 17:10:01 GMT

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

Ship it!


Ship It!

- Myroslav Papirkovskyy


On Березень 3, 2015, 7:08 після полудня, Andrew Onischuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31192/
> -----------------------------------------------------------
> 
> (Updated Березень 3, 2015, 7:08 після полудня)
> 
> 
> Review request for Ambari and Myroslav Papirkovskyy.
> 
> 
> Bugs: AMBARI-9703
>     https://issues.apache.org/jira/browse/AMBARI-9703
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> **STR**
> 
>   1. Post new property set <http://%s/api/v1/clusters/cl1/configurations>
>     
>         {"type":"slider-client","Config":{"cluster_name":"cl1"},"tag":"customTestVersion","properties":{"customKey":"customValue"},"properties_attributes":{"final":{"customKey":true}}}
> 
>   2. apply new property set <http://%s/api/v1/clusters/cl1>
>     
>         { "Clusters": {  "desired_configs" : {  "type": "slider-client",
>           "tag": "customTestVersion"} }}
> 
>   3. check new property set applied <http://%s/api/v1/clusters/cl1?desired_configs>
 
> **ER**  
> set applied, teg customTestVersion  
> **AR**
>     
>               "slider-client" : {
>             "tag" : "version1",
>             "user" : "admin",
>             "version" : 1
>           }
> 
> in response
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
2bf0cbf 
> 
> Diff: https://reviews.apache.org/r/31192/diff/
> 
> 
> Testing
> -------
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message