You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by rz...@apache.org on 2017/04/11 20:12:37 UTC

ambari git commit: AMBARI-20700 - UI should use the URL as-given by the backend (rzang)

Repository: ambari
Updated Branches:
  refs/heads/trunk 48ff4c19b -> c0424b9f6


AMBARI-20700 - UI should use the URL as-given by the backend (rzang)

Change-Id: I308c7b8d3eed2ba9218d26d2d2590dba9fe49f74


Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/c0424b9f
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/c0424b9f
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/c0424b9f

Branch: refs/heads/trunk
Commit: c0424b9f621266cd765cbfa2a36d50a9fb502b21
Parents: 48ff4c1
Author: Richard Zang <rz...@apache.org>
Authored: Tue Apr 11 13:10:28 2017 -0700
Committer: Richard Zang <rz...@apache.org>
Committed: Tue Apr 11 13:10:28 2017 -0700

----------------------------------------------------------------------
 ambari-web/app/controllers/installer.js | 14 ++------------
 1 file changed, 2 insertions(+), 12 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/c0424b9f/ambari-web/app/controllers/installer.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/controllers/installer.js b/ambari-web/app/controllers/installer.js
index 369e163..ec280fa 100644
--- a/ambari-web/app/controllers/installer.js
+++ b/ambari-web/app/controllers/installer.js
@@ -707,6 +707,7 @@ App.InstallerController = App.WizardController.extend(App.Persist, {
    */
   getSupportedOSListSuccessCallback: function (response, request, data) {
     var self = this;
+    var stack_default = data.versionDefinition.VersionDefinition.stack_default;
     var existedOS = data.versionDefinition.operating_systems;
     var existedMap = {};
     existedOS.map(function (existedOS) {
@@ -720,8 +721,7 @@ App.InstallerController = App.WizardController.extend(App.Persist, {
           repo.Repositories.base_url = '';
         });
         existedOS.push(supportedOS);
-      }
-      if(existedMap[supportedOS.OperatingSystems.os_type]) {
+      } else if (stack_default) { // only overwrite if it is stack default, otherwise use url from /version_definition
         existedMap[supportedOS.OperatingSystems.os_type].repositories.forEach(function (repo) {
           supportedOS.repositories.forEach(function (supportedRepo) {
             if (supportedRepo.Repositories.repo_id == repo.Repositories.repo_id) {
@@ -754,16 +754,6 @@ App.InstallerController = App.WizardController.extend(App.Persist, {
           this.setSelected(data.stackInfo.isStacksExistInDb);
         }
       }
-      // log diagnosis data for abnormal number of repos
-      var post_diagnosis = false;
-      data.versionDefinition.operating_systems.map(function(item) {
-        if (item.repositories.length > 2) {
-          post_diagnosis = true;
-        }
-      });
-      if (post_diagnosis) {
-        this.postUserPref('stack_response_diagnosis', data);
-      }
     }
   },