adding the allocated service port to the property in the correct way

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

Branch: refs/heads/master
Commit: fd916b41cee96de4e0ae4d30507129b8d1bd2a0b
Parents: edb22c5
Author: R-Rajkumar <[email protected]>
Authored: Sat Nov 8 19:14:53 2014 +0530
Committer: R-Rajkumar <[email protected]>
Committed: Sat Nov 8 19:14:53 2014 +0530

----------------------------------------------------------------------
 .../controller/impl/CloudControllerServiceImpl.java   | 14 +++-----------
 1 file changed, 3 insertions(+), 11 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/stratos/blob/fd916b41/components/org.apache.stratos.cloud.controller/src/main/java/org/apache/stratos/cloud/controller/impl/CloudControllerServiceImpl.java
----------------------------------------------------------------------
diff --git 
a/components/org.apache.stratos.cloud.controller/src/main/java/org/apache/stratos/cloud/controller/impl/CloudControllerServiceImpl.java
 
b/components/org.apache.stratos.cloud.controller/src/main/java/org/apache/stratos/cloud/controller/impl/CloudControllerServiceImpl.java
index 0b73676..cd83a06 100644
--- 
a/components/org.apache.stratos.cloud.controller/src/main/java/org/apache/stratos/cloud/controller/impl/CloudControllerServiceImpl.java
+++ 
b/components/org.apache.stratos.cloud.controller/src/main/java/org/apache/stratos/cloud/controller/impl/CloudControllerServiceImpl.java
@@ -56,8 +56,6 @@ import org.apache.stratos.kubernetes.client.model.Service;
 import org.apache.stratos.messaging.domain.topology.Cluster;
 import org.apache.stratos.messaging.domain.topology.ClusterStatus;
 import org.apache.stratos.messaging.domain.topology.Member;
-import org.apache.stratos.messaging.domain.topology.Cluster;
-import org.apache.stratos.messaging.domain.topology.ClusterStatus;
 import org.apache.stratos.messaging.domain.topology.MemberStatus;
 import org.apache.stratos.messaging.util.Constants;
 import org.jclouds.compute.ComputeService;
@@ -1500,15 +1498,9 @@ public class CloudControllerServiceImpl implements 
CloudControllerService {
             kubApi.createService(service);
 
             // set host port and update
-            //TODO to clean the impl of properties
-            Property[] properties = new Property[1];
-            Property property = new 
Property(StratosConstants.ALLOCATED_SERVICE_HOST_PORT,
-                    String.valueOf(service.getPort()));
-            org.apache.stratos.common.Properties properties1 = new 
org.apache.stratos.common.Properties();
-            properties1.setProperties(properties);
-
-            ctxt.setProperties(properties1);
-            dataHolder.addClusterContext(ctxt);
+            Property allocatedServiceHostPortProp = new Property();
+            
allocatedServiceHostPortProp.setName(StratosConstants.ALLOCATED_SERVICE_HOST_PORT);
+            
allocatedServiceHostPortProp.setValue(String.valueOf(service.getPort()));
 
             if (LOG.isDebugEnabled()) {
                 LOG.debug("Cloud Controller successfully started the service "

Reply via email to