> On Dec. 4, 2017, 4:20 p.m., Nate Cole wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/bootstrap/SshHostInfo.java
> > Lines 130-132 (patched)
> > <https://reviews.apache.org/r/64242/diff/1/?file=1905504#file1905504line130>
> >
> > How many URLs are we talking about here. If more than one, should this
> > bet a Set?
>
> Sonia Garudi wrote:
> ambariRepoUrls is a json string having ambari repo URLs for each os type
> which is sent from the UI.
>
> Nate Cole wrote:
> Can I see an example?
Example :
"[{"os_type":"os1","ambari_repo":"http://ambari-repo1","hosts":["host1","host2"],"ambariRepoUIError":"","hasError":false},{"os_type":"os2","ambari_repo":"http://ambari-repo2","hosts":["host3"],"ambariRepoUIError":"","hasError":false}]";
The above json string consists of list of 2 objects; each containing the
os_type, ambari repository URL and the list of host names for the corresponding
os_type. The 2 other fields(i.e. ambariRepoUIError and hasError) are used for
UI validation.
- Sonia
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64242/#review192705
-----------------------------------------------------------
On Dec. 1, 2017, 7:47 a.m., Sonia Garudi wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64242/
> -----------------------------------------------------------
>
> (Updated Dec. 1, 2017, 7:47 a.m.)
>
>
> Review request for Ambari and Nate Cole.
>
>
> Bugs: AMBARI-22476
> https://issues.apache.org/jira/browse/AMBARI-22476
>
>
> Repository: ambari
>
>
> Description
> -------
>
> The hosts failing with error code 44 are bootstrapped along with the ambari
> repository URLs. The required java beans are changed to include the ambari
> repository URLs to be passed to bootstrap.py.
>
>
> Diffs
> -----
>
>
> ambari-server/src/main/java/org/apache/ambari/server/bootstrap/BSRunner.java
> c7976ee5bb48209478258625532e3420973b4b8c
>
> ambari-server/src/main/java/org/apache/ambari/server/bootstrap/SshHostInfo.java
> 86888f499f15aa23807067ee1af745525b9d4619
> ambari-server/src/main/python/bootstrap.py
> f1c53ce45316843fd38624536eed656b029a230a
>
> ambari-server/src/test/java/org/apache/ambari/server/bootstrap/BootStrapTest.java
> b9da013524fee556c6faa2f0236d2c0c4b032750
> ambari-server/src/test/python/TestBootstrap.py
> bea47f498cbb523fbe24622c63b48918d433c717
> ambari-web/app/controllers/wizard/step3_controller.js
> 5ff2cdc1b4bb9da20565fe387765b4fc6728e096
> ambari-web/test/controllers/wizard/step3_test.js
> be1affe28fdcdb3404ebd3fb3fc6e61315483ed2
>
>
> Diff: https://reviews.apache.org/r/64242/diff/1/
>
>
> Testing
> -------
>
> Tests were successful in ambari-web and ambari-server.
>
>
> Thanks,
>
> Sonia Garudi
>
>
