You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by ma...@apache.org on 2013/06/27 19:53:26 UTC

svn commit: r1497478 - in /incubator/ambari/trunk: ambari-agent/src/main/puppet/modules/hdp/manifests/java/ ambari-agent/src/main/puppet/modules/hdp/manifests/java/jce/ ambari-server/src/main/java/org/apache/ambari/server/controller/

Author: mahadev
Date: Thu Jun 27 17:53:26 2013
New Revision: 1497478

URL: http://svn.apache.org/r1497478
Log:
AMBARI-2510. Can't complete install after executing "ambari-server setup-https" (Dmitry Sen via mahadev)

Modified:
    incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp/manifests/java/jce/package.pp
    incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp/manifests/java/package.pp
    incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java

Modified: incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp/manifests/java/jce/package.pp
URL: http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp/manifests/java/jce/package.pp?rev=1497478&r1=1497477&r2=1497478&view=diff
==============================================================================
--- incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp/manifests/java/jce/package.pp (original)
+++ incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp/manifests/java/jce/package.pp Thu Jun 27 17:53:26 2013
@@ -38,7 +38,8 @@ define hdp::java::jce::package(
   #TODO:SUHAS how to avoid redownload and install if correct version already present.
   # may be check the file sizes for local_policy and export_US policy jars? 
   # UNLESS  => "test -e ${java_exec}"
-  $curl_cmd = "mkdir -p ${artifact_dir}; curl -f --retry 10 ${jce_location}/${jce_policy_zip} -o ${jce_curl_target}"
+  # curl -k - ignoring unverified server ssl sertificate,
+  $curl_cmd = "mkdir -p ${artifact_dir}; curl -kf --retry 10 ${jce_location}/${jce_policy_zip} -o ${jce_curl_target}"
   exec{ "jce-download ${name}":
     command => $curl_cmd,
     creates => $jce_curl_target,

Modified: incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp/manifests/java/package.pp
URL: http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp/manifests/java/package.pp?rev=1497478&r1=1497477&r2=1497478&view=diff
==============================================================================
--- incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp/manifests/java/package.pp (original)
+++ incubator/ambari/trunk/ambari-agent/src/main/puppet/modules/hdp/manifests/java/package.pp Thu Jun 27 17:53:26 2013
@@ -38,8 +38,9 @@ define hdp::java::package(
   }
   $java_exec = "${java_home}/bin/java"
   $java_dir = regsubst($java_home,'/[^/]+$','')
-   
-  $curl_cmd = "mkdir -p ${artifact_dir} ; curl -f --retry 10 ${jdk_location}/${jdk_bin} -o ${jdk_curl_target}"
+
+  # curl -k - ignoring unverified server ssl sertificate,
+  $curl_cmd = "mkdir -p ${artifact_dir} ; curl -kf --retry 10 ${jdk_location}/${jdk_bin} -o ${jdk_curl_target}"
   exec{ "${curl_cmd} ${name}":
     command => $curl_cmd,
     creates => $jdk_curl_target,

Modified: incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
URL: http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java?rev=1497478&r1=1497477&r2=1497478&view=diff
==============================================================================
--- incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java (original)
+++ incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java Thu Jun 27 17:53:26 2013
@@ -164,14 +164,23 @@ public class AmbariManagementControllerI
     this.masterHostname =  InetAddress.getLocalHost().getCanonicalHostName();
 
     if (configs != null) {
-      this.jdkResourceUrl = "http://" + masterHostname + ":"
-          + configs.getClientApiPort()
+      String protocol, port;
+      if (configs.getApiSSLAuthentication() == true) {
+        protocol = "https";
+        port = String.valueOf(configs.getClientSSLApiPort());
+      } else {
+        protocol = "http";
+        port = String.valueOf(configs.getClientApiPort());
+      }
+
+      this.jdkResourceUrl = protocol + "://" + masterHostname + ":"
+          + port
           + JDK_RESOURCE_LOCATION;
-      this.ojdbcUrl = "http://" + masterHostname + ":" + 
-          + configs.getClientApiPort() + JDK_RESOURCE_LOCATION + "/" + configs.getOjdbcJarName();
+      this.ojdbcUrl = protocol + "://" + masterHostname + ":"
+          + port + JDK_RESOURCE_LOCATION + "/" + configs.getOjdbcJarName();
 
-      this.mysqljdbcUrl = "http://" + masterHostname + ":" + 
-          + configs.getClientApiPort() + JDK_RESOURCE_LOCATION + "/" + configs.getMySQLJarName();
+      this.mysqljdbcUrl = protocol + "://" + masterHostname + ":"
+          + port + JDK_RESOURCE_LOCATION + "/" + configs.getMySQLJarName();
       this.serverDB = configs.getServerDBName();
     } else {
       this.jdkResourceUrl = null;
@@ -179,8 +188,6 @@ public class AmbariManagementControllerI
       this.mysqljdbcUrl = null;
       this.serverDB = null;
     }
-    
-   
   }
 
   @Override