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

(Updated April 27, 2016, 12:11 a.m.)


Review request for Ambari, Jaimin Jetly, Nate Cole, Richard Zang, Xi Wang, and 
Yusaku Sako.


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


Repository: ambari


Description
-------

Provide RedHat satellite option. It can be used wherever we show redhat6 urls, 
and applicable to redhat6/redhat7. For example, registering repos, etc.
The API call to update links when using VDF can specify the new property. In 
order to make it more generic (and default true), it is called 
"ambari_managed_repositories". API looks like:
PUT /api/v1/stacks/HDP/versions/2.3/repository_versions/2
{
  "RepositoryVersions" : {
    "id" : 2
  },
  "operating_systems" : [
    {
      "OperatingSystems" : {
        "ambari_managed_repositories": false,
        "os_type" : "redhat6",
        "repository_version_id" : 2,
        "stack_name" : "HDP",
        "stack_version" : "2.3"
      },
      "repositories" : [
        {
          "Repositories" : {
            "base_url" : "http://repos.ambari.apache.org/hdp/HDP-2.3.4.14-4";,
            "repo_id" : "HDP-2.3",
            "repo_name" : "HDP"
          }
        },
        {
          "Repositories" : {
            "base_url" : 
"http://repos.ambari.apache.org/hdp/HDP-UTILS-1.1.0.20";,
            "repo_id" : "HDP-UTILS-1.1.0.20",
            "repo_name" : "HDP-UTILS"
          }
        }
      ]
    }
  ]
}


Diffs
-----

  ambari-web/app/controllers/main/admin/stack_and_upgrade_controller.js 7a7d050 
  ambari-web/app/mappers/repository_version_mapper.js 816308b 
  ambari-web/app/models/stack_version/repository_version.js e02999f 
  ambari-web/app/templates/main/admin/stack_upgrade/edit_repositories.hbs 
35677ab 
  ambari-web/app/views/main/admin/stack_upgrade/upgrade_version_box_view.js 
88bfec2 
  ambari-web/test/controllers/main/admin/stack_and_upgrade_controller_test.js 
00d7ce8 

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


Testing
-------

Modified unit test.
Local ambari-web test passed.
25660 tests complete (24 seconds)
154 tests pending
Manual testing done.


Thanks,

Zhe (Joe) Wang

Reply via email to