Skip to content

Commit 3dae8f3

Browse files
authored
Add kroxylicious.io namespace to route label (kroxylicious#3540)
This will help users to identify that the label is owned by our operator. Signed-off-by: Robert Young <robertyoungnz@gmail.com>
1 parent 8d36842 commit 3dae8f3

13 files changed

Lines changed: 19 additions & 13 deletions

File tree

kroxylicious-operator/src/main/java/io/kroxylicious/kubernetes/operator/model/RouteHostDetails.java

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,12 +50,18 @@ public record RouteHostDetails(
5050

5151
/**
5252
* Label used to identify a {@link Route} target.
53+
* <p>
54+
* The Operator manifests the Routes for an Ingress + VirtualKafkaCluster and
55+
* needs to react once the 'ingress[0].host' becomes available in the Routes' status.
56+
* This label enables the reconciler to discriminate between the bootstrap route and
57+
* the node routes when it is constructing the bootstrapAddress and advertisedBrokerAddressPattern
58+
* for the Gateway of the VirtualKafkaCluster.
5359
*/
5460
public enum RouteFor {
5561
BOOTSTRAP,
5662
NODE;
5763

58-
public static final String LABEL_KEY = "route-for";
64+
public static final String LABEL_KEY = "kroxylicious.io/route-for";
5965

6066
@Override
6167
public String toString() {

kroxylicious-operator/src/test/resources/DerivedResourcesTest/route-ingress-hosts-ready/in-Route-one-0.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ metadata:
1717
app.kubernetes.io/name: "kroxylicious"
1818
app.kubernetes.io/component: "proxy"
1919
app.kubernetes.io/instance: "minimal"
20-
route-for: "node"
20+
kroxylicious.io/route-for: "node"
2121
name: "one-0"
2222
namespace: "proxy-ns"
2323
ownerReferences:

kroxylicious-operator/src/test/resources/DerivedResourcesTest/route-ingress-hosts-ready/in-Route-one-1.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ metadata:
1717
app.kubernetes.io/name: "kroxylicious"
1818
app.kubernetes.io/component: "proxy"
1919
app.kubernetes.io/instance: "minimal"
20-
route-for: "node"
20+
kroxylicious.io/route-for: "node"
2121
name: "one-1"
2222
namespace: "proxy-ns"
2323
ownerReferences:

kroxylicious-operator/src/test/resources/DerivedResourcesTest/route-ingress-hosts-ready/in-Route-one-2.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ metadata:
1717
app.kubernetes.io/name: "kroxylicious"
1818
app.kubernetes.io/component: "proxy"
1919
app.kubernetes.io/instance: "minimal"
20-
route-for: "node"
20+
kroxylicious.io/route-for: "node"
2121
name: "one-2"
2222
namespace: "proxy-ns"
2323
ownerReferences:

kroxylicious-operator/src/test/resources/DerivedResourcesTest/route-ingress-hosts-ready/in-Route-one-bootstrap.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ metadata:
1717
app.kubernetes.io/name: "kroxylicious"
1818
app.kubernetes.io/component: "proxy"
1919
app.kubernetes.io/instance: "minimal"
20-
route-for: "bootstrap"
20+
kroxylicious.io/route-for: "bootstrap"
2121
name: "one-bootstrap"
2222
namespace: "proxy-ns"
2323
ownerReferences:

kroxylicious-operator/src/test/resources/DerivedResourcesTest/route-ingress-hosts-ready/out-Route-one-0.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ metadata:
1717
app.kubernetes.io/name: "kroxylicious"
1818
app.kubernetes.io/component: "proxy"
1919
app.kubernetes.io/instance: "minimal"
20-
route-for: "node"
20+
kroxylicious.io/route-for: "node"
2121
name: "one-0"
2222
namespace: "proxy-ns"
2323
ownerReferences:

kroxylicious-operator/src/test/resources/DerivedResourcesTest/route-ingress-hosts-ready/out-Route-one-1.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ metadata:
1717
app.kubernetes.io/name: "kroxylicious"
1818
app.kubernetes.io/component: "proxy"
1919
app.kubernetes.io/instance: "minimal"
20-
route-for: "node"
20+
kroxylicious.io/route-for: "node"
2121
name: "one-1"
2222
namespace: "proxy-ns"
2323
ownerReferences:

kroxylicious-operator/src/test/resources/DerivedResourcesTest/route-ingress-hosts-ready/out-Route-one-2.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ metadata:
1717
app.kubernetes.io/name: "kroxylicious"
1818
app.kubernetes.io/component: "proxy"
1919
app.kubernetes.io/instance: "minimal"
20-
route-for: "node"
20+
kroxylicious.io/route-for: "node"
2121
name: "one-2"
2222
namespace: "proxy-ns"
2323
ownerReferences:

kroxylicious-operator/src/test/resources/DerivedResourcesTest/route-ingress-hosts-ready/out-Route-one-bootstrap.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ metadata:
1717
app.kubernetes.io/name: "kroxylicious"
1818
app.kubernetes.io/component: "proxy"
1919
app.kubernetes.io/instance: "minimal"
20-
route-for: "bootstrap"
20+
kroxylicious.io/route-for: "bootstrap"
2121
name: "one-bootstrap"
2222
namespace: "proxy-ns"
2323
ownerReferences:

kroxylicious-operator/src/test/resources/DerivedResourcesTest/route-ingress/out-Route-one-0.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ metadata:
1717
app.kubernetes.io/name: "kroxylicious"
1818
app.kubernetes.io/component: "proxy"
1919
app.kubernetes.io/instance: "minimal"
20-
route-for: "node"
20+
kroxylicious.io/route-for: "node"
2121
name: "one-0"
2222
namespace: "proxy-ns"
2323
ownerReferences:

0 commit comments

Comments
 (0)