You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by da...@apache.org on 2016/04/26 12:24:12 UTC
[1/4] camel git commit: CAMEL-9683: A new toService EIP that uses a
client discovery to lookup alive services and pick a service ip/port to use
when calling the service from Camel route. Allows to plugin different
providers.
Repository: camel
Updated Branches:
refs/heads/kube-lb 3b0b4d036 -> f42f6e3a5
CAMEL-9683: A new toService EIP that uses a client discovery to lookup alive services and pick a service ip/port to use when calling the service from Camel route. Allows to plugin different providers.
Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/68eb0dbc
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/68eb0dbc
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/68eb0dbc
Branch: refs/heads/kube-lb
Commit: 68eb0dbc8d578cf7a5d525b97150de036a77ee13
Parents: 3b0b4d0
Author: Claus Ibsen <da...@apache.org>
Authored: Tue Apr 26 12:00:01 2016 +0200
Committer: Claus Ibsen <da...@apache.org>
Committed: Tue Apr 26 12:01:19 2016 +0200
----------------------------------------------------------------------
.../ServiceCallConfigurationDefinition.java | 36 +++++++++++++++
.../camel/model/ServiceCallDefinition.java | 46 ++++++++++++++++----
.../camel/spi/ServiceCallLoadBalancer.java | 34 +++++++++++++++
.../processor/KubernetesProcessorFactory.java | 17 +++++++-
.../KubernetesServiceCallProcessor.java | 18 +++++++-
.../processor/KubernetesServiceDiscovery.java | 2 +-
.../processor/RandomLoadBalancer.java | 19 +++++---
.../component/kubernetes/processor/Server.java | 5 ++-
.../processor/ServiceCallLoadBalancer.java | 25 -----------
9 files changed, 159 insertions(+), 43 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/camel/blob/68eb0dbc/camel-core/src/main/java/org/apache/camel/model/ServiceCallConfigurationDefinition.java
----------------------------------------------------------------------
diff --git a/camel-core/src/main/java/org/apache/camel/model/ServiceCallConfigurationDefinition.java b/camel-core/src/main/java/org/apache/camel/model/ServiceCallConfigurationDefinition.java
index 2363a63..e84f554 100644
--- a/camel-core/src/main/java/org/apache/camel/model/ServiceCallConfigurationDefinition.java
+++ b/camel-core/src/main/java/org/apache/camel/model/ServiceCallConfigurationDefinition.java
@@ -23,6 +23,7 @@ import javax.xml.bind.annotation.XmlRootElement;
import javax.xml.bind.annotation.XmlTransient;
import org.apache.camel.spi.Metadata;
+import org.apache.camel.spi.ServiceCallLoadBalancer;
@Metadata(label = "eip,routing")
@XmlRootElement(name = "toServiceConfiguration")
@@ -61,6 +62,10 @@ public class ServiceCallConfigurationDefinition extends IdentifiedType {
private String clientKeyPassphrase;
@XmlAttribute @Metadata(label = "security")
private Boolean trustCerts;
+ @XmlAttribute
+ private String loadBalancerRef;
+ @XmlTransient
+ private ServiceCallLoadBalancer loadBalancer;
public ServiceCallConfigurationDefinition() {
}
@@ -192,6 +197,21 @@ public class ServiceCallConfigurationDefinition extends IdentifiedType {
this.trustCerts = trustCerts;
}
+ public String getLoadBalancerRef() {
+ return loadBalancerRef;
+ }
+
+ public void setLoadBalancerRef(String loadBalancerRef) {
+ this.loadBalancerRef = loadBalancerRef;
+ }
+
+ public ServiceCallLoadBalancer getLoadBalancer() {
+ return loadBalancer;
+ }
+
+ public void setLoadBalancer(ServiceCallLoadBalancer loadBalancer) {
+ this.loadBalancer = loadBalancer;
+ }
// Fluent API
// -------------------------------------------------------------------------
@@ -317,6 +337,22 @@ public class ServiceCallConfigurationDefinition extends IdentifiedType {
}
/**
+ * Sets a reference to a custom {@link org.apache.camel.spi.ServiceCallLoadBalancer} to use.
+ */
+ public ServiceCallConfigurationDefinition loadBalancer(String loadBalancerRef) {
+ setLoadBalancerRef(loadBalancerRef);
+ return this;
+ }
+
+ /**
+ * Sets a custom {@link org.apache.camel.spi.ServiceCallLoadBalancer} to use.
+ */
+ public ServiceCallConfigurationDefinition loadBalancer(ServiceCallLoadBalancer loadBalancer) {
+ setLoadBalancer(loadBalancer);
+ return this;
+ }
+
+ /**
* End of configuration
*/
public ServiceCallDefinition end() {
http://git-wip-us.apache.org/repos/asf/camel/blob/68eb0dbc/camel-core/src/main/java/org/apache/camel/model/ServiceCallDefinition.java
----------------------------------------------------------------------
diff --git a/camel-core/src/main/java/org/apache/camel/model/ServiceCallDefinition.java b/camel-core/src/main/java/org/apache/camel/model/ServiceCallDefinition.java
index 7b2d1a5..d1c154d 100644
--- a/camel-core/src/main/java/org/apache/camel/model/ServiceCallDefinition.java
+++ b/camel-core/src/main/java/org/apache/camel/model/ServiceCallDefinition.java
@@ -21,19 +21,19 @@ import javax.xml.bind.annotation.XmlAccessorType;
import javax.xml.bind.annotation.XmlAttribute;
import javax.xml.bind.annotation.XmlElement;
import javax.xml.bind.annotation.XmlRootElement;
+import javax.xml.bind.annotation.XmlTransient;
import org.apache.camel.ExchangePattern;
import org.apache.camel.Processor;
import org.apache.camel.spi.Metadata;
import org.apache.camel.spi.RouteContext;
+import org.apache.camel.spi.ServiceCallLoadBalancer;
@Metadata(label = "eip,routing")
@XmlRootElement(name = "serviceCall")
@XmlAccessorType(XmlAccessType.FIELD)
public class ServiceCallDefinition extends NoOutputDefinition<ServiceCallDefinition> {
- // TODO: load balancing strategy
-
@XmlElement
private ServiceCallConfigurationDefinition serviceCallConfiguration;
@XmlAttribute @Metadata(required = "true")
@@ -48,16 +48,14 @@ public class ServiceCallDefinition extends NoOutputDefinition<ServiceCallDefinit
private String discovery;
@XmlAttribute
private String serviceCallConfigurationRef;
+ @XmlAttribute
+ private String loadBalancerRef;
+ @XmlTransient
+ private ServiceCallLoadBalancer loadBalancer;
public ServiceCallDefinition() {
}
- // serviceCall("myService") (will use http by default)
- // serviceCall("myService/foo") (will use http by default)
- // serviceCall("http:myService/foo")
- // serviceCall("myService", "http:myService.host:myService.port/foo")
- // serviceCall("myService", "netty4:tcp:myService?connectTimeout=1000")
-
@Override
public String toString() {
return "ServiceCall[" + name + "]";
@@ -136,6 +134,22 @@ public class ServiceCallDefinition extends NoOutputDefinition<ServiceCallDefinit
return this;
}
+ /**
+ * Sets a reference to a custom {@link org.apache.camel.spi.ServiceCallLoadBalancer} to use.
+ */
+ public ServiceCallDefinition loadBalancer(String loadBalancerRef) {
+ setLoadBalancerRef(loadBalancerRef);
+ return this;
+ }
+
+ /**
+ * Sets a custom {@link org.apache.camel.spi.ServiceCallLoadBalancer} to use.
+ */
+ public ServiceCallDefinition loadBalancer(ServiceCallLoadBalancer loadBalancer) {
+ setLoadBalancer(loadBalancer);
+ return this;
+ }
+
// Properties
// -------------------------------------------------------------------------
@@ -198,4 +212,20 @@ public class ServiceCallDefinition extends NoOutputDefinition<ServiceCallDefinit
public void setUri(String uri) {
this.uri = uri;
}
+
+ public String getLoadBalancerRef() {
+ return loadBalancerRef;
+ }
+
+ public void setLoadBalancerRef(String loadBalancerRef) {
+ this.loadBalancerRef = loadBalancerRef;
+ }
+
+ public ServiceCallLoadBalancer getLoadBalancer() {
+ return loadBalancer;
+ }
+
+ public void setLoadBalancer(ServiceCallLoadBalancer loadBalancer) {
+ this.loadBalancer = loadBalancer;
+ }
}
http://git-wip-us.apache.org/repos/asf/camel/blob/68eb0dbc/camel-core/src/main/java/org/apache/camel/spi/ServiceCallLoadBalancer.java
----------------------------------------------------------------------
diff --git a/camel-core/src/main/java/org/apache/camel/spi/ServiceCallLoadBalancer.java b/camel-core/src/main/java/org/apache/camel/spi/ServiceCallLoadBalancer.java
new file mode 100644
index 0000000..d9d7650
--- /dev/null
+++ b/camel-core/src/main/java/org/apache/camel/spi/ServiceCallLoadBalancer.java
@@ -0,0 +1,34 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements. See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.camel.spi;
+
+import java.util.Collection;
+
+/**
+ * Allows SPIs to implement custom load balacing strategies for the Service Call EIP.
+ */
+public interface ServiceCallLoadBalancer<T> {
+
+ /**
+ * Chooses one of the servers to use using the implemented strategy.
+ *
+ * @param servers list of servers
+ * @return the choosen server to use.
+ */
+ T chooseServer(Collection<T> servers);
+
+}
http://git-wip-us.apache.org/repos/asf/camel/blob/68eb0dbc/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesProcessorFactory.java
----------------------------------------------------------------------
diff --git a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesProcessorFactory.java b/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesProcessorFactory.java
index 13f77e2..f0b231b 100644
--- a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesProcessorFactory.java
+++ b/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesProcessorFactory.java
@@ -27,6 +27,7 @@ import org.apache.camel.model.ServiceCallConfigurationDefinition;
import org.apache.camel.model.ServiceCallDefinition;
import org.apache.camel.spi.ProcessorFactory;
import org.apache.camel.spi.RouteContext;
+import org.apache.camel.spi.ServiceCallLoadBalancer;
import org.apache.camel.util.CamelContextHelper;
import org.apache.camel.util.IntrospectionSupport;
@@ -79,7 +80,21 @@ public class KubernetesProcessorFactory implements ProcessorFactory {
namespace = kc.getNamespace();
}
- return new KubernetesServiceCallProcessor(name, namespace, uri, mep, kc);
+ // lookup the load balancer to use
+ ServiceCallLoadBalancer lb = ts.getLoadBalancer();
+ if (lb == null && ts.getServiceCallConfigurationRef() != null) {
+ lb = CamelContextHelper.mandatoryLookup(routeContext.getCamelContext(), ts.getLoadBalancerRef(), ServiceCallLoadBalancer.class);
+ }
+ if (lb == null && config != null) {
+ lb = config.getLoadBalancer();
+ }
+ if (lb == null && configRef != null) {
+ lb = CamelContextHelper.mandatoryLookup(routeContext.getCamelContext(), configRef.getLoadBalancerRef(), ServiceCallLoadBalancer.class);
+ }
+
+ KubernetesServiceCallProcessor processor = new KubernetesServiceCallProcessor(name, namespace, uri, mep, kc);
+ processor.setLoadBalancer(lb);
+ return processor;
} else {
return null;
}
http://git-wip-us.apache.org/repos/asf/camel/blob/68eb0dbc/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesServiceCallProcessor.java
----------------------------------------------------------------------
diff --git a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesServiceCallProcessor.java b/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesServiceCallProcessor.java
index 8bdfe09..ff859d6 100644
--- a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesServiceCallProcessor.java
+++ b/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesServiceCallProcessor.java
@@ -34,6 +34,7 @@ import org.apache.camel.component.kubernetes.KubernetesConfiguration;
import org.apache.camel.component.kubernetes.KubernetesConstants;
import org.apache.camel.processor.SendDynamicProcessor;
import org.apache.camel.spi.IdAware;
+import org.apache.camel.spi.ServiceCallLoadBalancer;
import org.apache.camel.support.ServiceSupport;
import org.apache.camel.util.AsyncProcessorHelper;
import org.apache.camel.util.ObjectHelper;
@@ -59,7 +60,7 @@ public class KubernetesServiceCallProcessor extends ServiceSupport implements As
private final KubernetesConfiguration configuration;
private KubernetesServiceDiscovery discovery;
- private ServiceCallLoadBalancer loadBalancer = new RandomLoadBalancer();
+ private ServiceCallLoadBalancer<Server> loadBalancer;
private final ServiceCallExpression serviceCallExpression;
private SendDynamicProcessor processor;
@@ -116,7 +117,7 @@ public class KubernetesServiceCallProcessor extends ServiceSupport implements As
}
// let the client load balancer chose which server to use
- Server server = loadBalancer.choseServer(servers);
+ Server server = loadBalancer.chooseServer(servers);
String ip = server.getIp();
int port = server.getPort();
LOG.debug("Random selected service {} active at server: {}:{}", name, ip, port);
@@ -154,12 +155,25 @@ public class KubernetesServiceCallProcessor extends ServiceSupport implements As
return "kubernetes";
}
+ public ServiceCallLoadBalancer<Server> getLoadBalancer() {
+ return loadBalancer;
+ }
+
+ public void setLoadBalancer(ServiceCallLoadBalancer<Server> loadBalancer) {
+ this.loadBalancer = loadBalancer;
+ }
+
@Override
protected void doStart() throws Exception {
ObjectHelper.notEmpty(name, "name", this);
ObjectHelper.notEmpty(namespace, "namespace", this);
ObjectHelper.notEmpty(configuration.getMasterUrl(), "masterUrl", this);
+ if (loadBalancer == null) {
+ loadBalancer = new RandomLoadBalancer();
+ }
+ LOG.info("KubernetesServiceCall at namespace: {} with service name: {} is using load balancer: {}", namespace, name, loadBalancer);
+
discovery = new KubernetesServiceDiscovery(name, namespace, null, createKubernetesClient());
processor = new SendDynamicProcessor(uri, serviceCallExpression);
processor.setCamelContext(getCamelContext());
http://git-wip-us.apache.org/repos/asf/camel/blob/68eb0dbc/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesServiceDiscovery.java
----------------------------------------------------------------------
diff --git a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesServiceDiscovery.java b/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesServiceDiscovery.java
index ff44e44..75590f7 100644
--- a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesServiceDiscovery.java
+++ b/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesServiceDiscovery.java
@@ -31,7 +31,7 @@ import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
/**
- * Discovers services in Kubernetes.
+ * Discovers where services in Kubernetes are running on which servers.
*/
public class KubernetesServiceDiscovery extends ServiceSupport {
http://git-wip-us.apache.org/repos/asf/camel/blob/68eb0dbc/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/RandomLoadBalancer.java
----------------------------------------------------------------------
diff --git a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/RandomLoadBalancer.java b/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/RandomLoadBalancer.java
index 86a9b6c..5724098 100644
--- a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/RandomLoadBalancer.java
+++ b/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/RandomLoadBalancer.java
@@ -16,16 +16,25 @@
*/
package org.apache.camel.component.kubernetes.processor;
+import java.util.ArrayList;
+import java.util.Collection;
import java.util.List;
import java.util.Random;
-public class RandomLoadBalancer implements ServiceCallLoadBalancer {
+import org.apache.camel.spi.ServiceCallLoadBalancer;
+
+public class RandomLoadBalancer implements ServiceCallLoadBalancer<Server> {
@Override
- public Server choseServer(List<Server> services) {
- int size = services.size();
+ public Server chooseServer(Collection<Server> servers) {
+ List<Server> list = new ArrayList<>(servers);
+ int size = list.size();
int ran = new Random().nextInt(size);
- Server server = services.get(ran);
- return server;
+ return list.get(ran);
+ }
+
+ @Override
+ public String toString() {
+ return "RandomLoadBalancer";
}
}
http://git-wip-us.apache.org/repos/asf/camel/blob/68eb0dbc/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/Server.java
----------------------------------------------------------------------
diff --git a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/Server.java b/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/Server.java
index d0a4471..6a6a913 100644
--- a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/Server.java
+++ b/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/Server.java
@@ -16,7 +16,10 @@
*/
package org.apache.camel.component.kubernetes.processor;
-public class Server {
+/**
+ * Represents a model of a kubernetes server.
+ */
+public final class Server {
private final String ip;
private final int port;
http://git-wip-us.apache.org/repos/asf/camel/blob/68eb0dbc/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/ServiceCallLoadBalancer.java
----------------------------------------------------------------------
diff --git a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/ServiceCallLoadBalancer.java b/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/ServiceCallLoadBalancer.java
deleted file mode 100644
index f2dc7ad..0000000
--- a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/ServiceCallLoadBalancer.java
+++ /dev/null
@@ -1,25 +0,0 @@
-/**
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.apache.camel.component.kubernetes.processor;
-
-import java.util.List;
-
-public interface ServiceCallLoadBalancer {
-
- Server choseServer(List<Server> services);
-
-}
[3/4] camel git commit: Fixed typo
Posted by da...@apache.org.
Fixed typo
Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/7465a91c
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/7465a91c
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/7465a91c
Branch: refs/heads/kube-lb
Commit: 7465a91c63ff8b856206252a1e93424b2694c8e0
Parents: 27bba78
Author: Claus Ibsen <da...@apache.org>
Authored: Tue Apr 26 12:02:40 2016 +0200
Committer: Claus Ibsen <da...@apache.org>
Committed: Tue Apr 26 12:02:40 2016 +0200
----------------------------------------------------------------------
.../main/java/org/apache/camel/spi/ServiceCallLoadBalancer.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/camel/blob/7465a91c/camel-core/src/main/java/org/apache/camel/spi/ServiceCallLoadBalancer.java
----------------------------------------------------------------------
diff --git a/camel-core/src/main/java/org/apache/camel/spi/ServiceCallLoadBalancer.java b/camel-core/src/main/java/org/apache/camel/spi/ServiceCallLoadBalancer.java
index d9d7650..c7b05db 100644
--- a/camel-core/src/main/java/org/apache/camel/spi/ServiceCallLoadBalancer.java
+++ b/camel-core/src/main/java/org/apache/camel/spi/ServiceCallLoadBalancer.java
@@ -27,7 +27,7 @@ public interface ServiceCallLoadBalancer<T> {
* Chooses one of the servers to use using the implemented strategy.
*
* @param servers list of servers
- * @return the choosen server to use.
+ * @return the chosen server to use.
*/
T chooseServer(Collection<T> servers);
[4/4] camel git commit: CAMEL-9683: A new toService EIP that uses a
client discovery to lookup alive services and pick a service ip/port to use
when calling the service from Camel route. Allows to plugin different
providers.
Posted by da...@apache.org.
CAMEL-9683: A new toService EIP that uses a client discovery to lookup alive services and pick a service ip/port to use when calling the service from Camel route. Allows to plugin different providers.
Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/f42f6e3a
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/f42f6e3a
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/f42f6e3a
Branch: refs/heads/kube-lb
Commit: f42f6e3a5c0416d5a004d9e56f049c45de4a55be
Parents: 7465a91
Author: Claus Ibsen <da...@apache.org>
Authored: Tue Apr 26 12:23:45 2016 +0200
Committer: Claus Ibsen <da...@apache.org>
Committed: Tue Apr 26 12:23:45 2016 +0200
----------------------------------------------------------------------
.../processor/KubernetesProcessorFactory.java | 67 +++++++++++++++-----
.../processor/RoundRobinBalancer.java | 44 +++++++++++++
.../processor/ServiceCallRouteTest.java | 2 +
.../src/test/resources/log4j.properties | 2 +-
4 files changed, 98 insertions(+), 17 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/camel/blob/f42f6e3a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesProcessorFactory.java
----------------------------------------------------------------------
diff --git a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesProcessorFactory.java b/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesProcessorFactory.java
index f0b231b..05d979b 100644
--- a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesProcessorFactory.java
+++ b/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/KubernetesProcessorFactory.java
@@ -43,24 +43,25 @@ public class KubernetesProcessorFactory implements ProcessorFactory {
}
@Override
+ @SuppressWarnings("unchecked")
public Processor createProcessor(RouteContext routeContext, ProcessorDefinition<?> definition) throws Exception {
if (definition instanceof ServiceCallDefinition) {
- ServiceCallDefinition ts = (ServiceCallDefinition) definition;
+ ServiceCallDefinition sc = (ServiceCallDefinition) definition;
// discovery must either not be set, or if set then must be us
- if (ts.getDiscovery() != null && !"kubernetes".equals(ts.getDiscovery())) {
+ if (sc.getDiscovery() != null && !"kubernetes".equals(sc.getDiscovery())) {
return null;
}
- String name = ts.getName();
- String namespace = ts.getNamespace();
- String uri = ts.getUri();
- ExchangePattern mep = ts.getPattern();
+ String name = sc.getName();
+ String namespace = sc.getNamespace();
+ String uri = sc.getUri();
+ ExchangePattern mep = sc.getPattern();
- ServiceCallConfigurationDefinition config = ts.getServiceCallConfiguration();
+ ServiceCallConfigurationDefinition config = sc.getServiceCallConfiguration();
ServiceCallConfigurationDefinition configRef = null;
- if (ts.getServiceCallConfigurationRef() != null) {
- configRef = CamelContextHelper.mandatoryLookup(routeContext.getCamelContext(), ts.getServiceCallConfigurationRef(), ServiceCallConfigurationDefinition.class);
+ if (sc.getServiceCallConfigurationRef() != null) {
+ configRef = CamelContextHelper.mandatoryLookup(routeContext.getCamelContext(), sc.getServiceCallConfigurationRef(), ServiceCallConfigurationDefinition.class);
}
// extract the properties from the configuration from the model
@@ -80,16 +81,13 @@ public class KubernetesProcessorFactory implements ProcessorFactory {
namespace = kc.getNamespace();
}
- // lookup the load balancer to use
- ServiceCallLoadBalancer lb = ts.getLoadBalancer();
- if (lb == null && ts.getServiceCallConfigurationRef() != null) {
- lb = CamelContextHelper.mandatoryLookup(routeContext.getCamelContext(), ts.getLoadBalancerRef(), ServiceCallLoadBalancer.class);
- }
+ // lookup the load balancer to use (configured on EIP takes precedence vs configured on configuration)
+ ServiceCallLoadBalancer lb = configureLoadBalancer(routeContext, sc);
if (lb == null && config != null) {
- lb = config.getLoadBalancer();
+ lb = configureLoadBalancer(routeContext, config);
}
if (lb == null && configRef != null) {
- lb = CamelContextHelper.mandatoryLookup(routeContext.getCamelContext(), configRef.getLoadBalancerRef(), ServiceCallLoadBalancer.class);
+ lb = configureLoadBalancer(routeContext, configRef);
}
KubernetesServiceCallProcessor processor = new KubernetesServiceCallProcessor(name, namespace, uri, mep, kc);
@@ -100,4 +98,41 @@ public class KubernetesProcessorFactory implements ProcessorFactory {
}
}
+ private ServiceCallLoadBalancer configureLoadBalancer(RouteContext routeContext, ServiceCallDefinition sd) {
+ ServiceCallLoadBalancer lb = null;
+
+ if (sd != null) {
+ lb = sd.getLoadBalancer();
+ if (lb == null && sd.getLoadBalancerRef() != null) {
+ String ref = sd.getLoadBalancerRef();
+ // special for ref is referring to built-in
+ if ("random".equalsIgnoreCase(ref)) {
+ lb = new RandomLoadBalancer();
+ } else if ("roundrobin".equalsIgnoreCase(ref)) {
+ lb = new RoundRobinBalancer();
+ } else {
+ lb = CamelContextHelper.mandatoryLookup(routeContext.getCamelContext(), ref, ServiceCallLoadBalancer.class);
+ }
+ }
+ }
+
+ return lb;
+ }
+
+ private ServiceCallLoadBalancer configureLoadBalancer(RouteContext routeContext, ServiceCallConfigurationDefinition config) {
+ ServiceCallLoadBalancer lb = config.getLoadBalancer();
+ if (lb == null && config.getLoadBalancerRef() != null) {
+ String ref = config.getLoadBalancerRef();
+ // special for ref is referring to built-in
+ if ("random".equalsIgnoreCase(ref)) {
+ lb = new RandomLoadBalancer();
+ } else if ("roundrobin".equalsIgnoreCase(ref)) {
+ lb = new RoundRobinBalancer();
+ } else {
+ lb = CamelContextHelper.mandatoryLookup(routeContext.getCamelContext(), ref, ServiceCallLoadBalancer.class);
+ }
+ }
+ return lb;
+ }
+
}
http://git-wip-us.apache.org/repos/asf/camel/blob/f42f6e3a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/RoundRobinBalancer.java
----------------------------------------------------------------------
diff --git a/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/RoundRobinBalancer.java b/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/RoundRobinBalancer.java
new file mode 100644
index 0000000..2a2a401
--- /dev/null
+++ b/components/camel-kubernetes/src/main/java/org/apache/camel/component/kubernetes/processor/RoundRobinBalancer.java
@@ -0,0 +1,44 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements. See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.camel.component.kubernetes.processor;
+
+import java.util.ArrayList;
+import java.util.Collection;
+import java.util.List;
+
+import org.apache.camel.spi.ServiceCallLoadBalancer;
+
+public class RoundRobinBalancer implements ServiceCallLoadBalancer<Server> {
+
+ private int counter = -1;
+
+ @Override
+ public Server chooseServer(Collection<Server> servers) {
+ List<Server> list = new ArrayList<>(servers);
+
+ int size = list.size();
+ if (++counter >= size) {
+ counter = 0;
+ }
+ return list.get(counter);
+ }
+
+ @Override
+ public String toString() {
+ return "RoundRobinBalancer";
+ }
+}
http://git-wip-us.apache.org/repos/asf/camel/blob/f42f6e3a/components/camel-kubernetes/src/test/java/org/apache/camel/component/kubernetes/processor/ServiceCallRouteTest.java
----------------------------------------------------------------------
diff --git a/components/camel-kubernetes/src/test/java/org/apache/camel/component/kubernetes/processor/ServiceCallRouteTest.java b/components/camel-kubernetes/src/test/java/org/apache/camel/component/kubernetes/processor/ServiceCallRouteTest.java
index b6b675a..ae26cb6 100644
--- a/components/camel-kubernetes/src/test/java/org/apache/camel/component/kubernetes/processor/ServiceCallRouteTest.java
+++ b/components/camel-kubernetes/src/test/java/org/apache/camel/component/kubernetes/processor/ServiceCallRouteTest.java
@@ -45,6 +45,8 @@ public class ServiceCallRouteTest extends CamelTestSupport {
config.setUsername("admin");
config.setPassword("admin");
config.setNamespace("default");
+ // lets use the built-in round robin (random is default)
+ config.setLoadBalancerRef("roundrobin");
from("direct:start")
.serviceCall("cdi-camel-jetty", null, config)
http://git-wip-us.apache.org/repos/asf/camel/blob/f42f6e3a/components/camel-kubernetes/src/test/resources/log4j.properties
----------------------------------------------------------------------
diff --git a/components/camel-kubernetes/src/test/resources/log4j.properties b/components/camel-kubernetes/src/test/resources/log4j.properties
index 767860e..f88e05c 100644
--- a/components/camel-kubernetes/src/test/resources/log4j.properties
+++ b/components/camel-kubernetes/src/test/resources/log4j.properties
@@ -18,7 +18,7 @@
#
# The logging properties used
#
-log4j.rootLogger=INFO, out
+log4j.rootLogger=INFO, file
log4j.logger.org.apache.camel.component.kubernetes=DEBUG
[2/4] camel git commit: Fixed CS
Posted by da...@apache.org.
Fixed CS
Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/27bba78e
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/27bba78e
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/27bba78e
Branch: refs/heads/kube-lb
Commit: 27bba78ea5c1690e218d15449fce3da02f1351c6
Parents: 68eb0db
Author: Claus Ibsen <da...@apache.org>
Authored: Tue Apr 26 12:02:17 2016 +0200
Committer: Claus Ibsen <da...@apache.org>
Committed: Tue Apr 26 12:02:17 2016 +0200
----------------------------------------------------------------------
.../camel/impl/EndpointRegistryKeepRouteEndpointsTest.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/camel/blob/27bba78e/camel-core/src/test/java/org/apache/camel/impl/EndpointRegistryKeepRouteEndpointsTest.java
----------------------------------------------------------------------
diff --git a/camel-core/src/test/java/org/apache/camel/impl/EndpointRegistryKeepRouteEndpointsTest.java b/camel-core/src/test/java/org/apache/camel/impl/EndpointRegistryKeepRouteEndpointsTest.java
index db6e734..d0894f8 100644
--- a/camel-core/src/test/java/org/apache/camel/impl/EndpointRegistryKeepRouteEndpointsTest.java
+++ b/camel-core/src/test/java/org/apache/camel/impl/EndpointRegistryKeepRouteEndpointsTest.java
@@ -56,11 +56,11 @@ public class EndpointRegistryKeepRouteEndpointsTest extends ContextTestSupport {
int count = 0;
for (int i = 0; i < 50; i++) {
String uri = "mock://unknown" + i;
- if (context.hasEndpoint(uri) != null) {
+ if (context.hasEndpoint(uri) != null) {
count++;
// and it should be dynamic
assertTrue(context.getEndpointRegistry().isDynamic(uri));
- };
+ }
}
assertEquals("Should only be 20 dynamic endpoints in the cache", 20, count);