Skip to content

Commit 6a882e4

Browse files
committed
Use op-rs release. Also fix tests
1 parent e1ff5b1 commit 6a882e4

10 files changed

+82
-66
lines changed

Cargo.lock

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.nix

Lines changed: 10 additions & 10 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ prost = "0.13"
2121
prost-types = "0.13"
2222
serde = "1.0"
2323
snafu = "0.8"
24-
stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", tag = "stackable-operator-0.80.0" }
24+
stackable-operator = { git = "https://github.com/stackabletech/operator-rs.git", tag = "stackable-operator-0.81.0" }
2525
strum = { version = "0.26", features = ["derive"] }
2626
socket2 = { version = "0.5", features = ["all"] }
2727
tokio = { version = "1.40", features = ["full"] }
@@ -34,4 +34,3 @@ tracing = "0.1.40"
3434
[patch."https://github.com/stackabletech/operator-rs.git"]
3535
# stackable-operator = { path = "../operator-rs/crates/stackable-operator" }
3636
# stackable-operator = { git = "https://github.com/stackabletech//operator-rs.git", branch = "main" }
37-
stackable-operator = { git = "https://github.com/stackabletech//operator-rs.git", branch = "feature/smart-preferred-address-type" }

crate-hashes.json

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
apiVersion: kuttl.dev/v1beta1
3+
kind: TestStep
4+
commands:
5+
- script: envsubst '$NAMESPACE' < 05_listenerclass.yaml | kubectl apply -n $NAMESPACE -f -

tests/templates/kuttl/smoke-nodeport/05-create-listenerclass.yaml.j2 renamed to tests/templates/kuttl/smoke-nodeport/05_listenerclass.yaml.j2

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
apiVersion: listeners.stackable.tech/v1alpha1
33
kind: ListenerClass
44
metadata:
5-
name: listener-operator-test-smoke-nodeport
5+
name: listener-operator-test-smoke-nodeport-$NAMESPACE
66
spec:
77
serviceType: NodePort
88
preferredAddressType: {{ test_scenario['values']['addressType'] }}

tests/templates/kuttl/smoke-nodeport/10-assert.yaml.j2

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,4 +13,10 @@ metadata:
1313
name: listener-nginx-long-name-approaching-k8s-limits-0
1414
status:
1515
ingressAddresses:
16+
# FIXME: This test seems to assume that Nodes always have a hostname *and* a IP address (which should be the case on
17+
# e.g. kind)! We probably need to relax this assertion at a later point of time
18+
{% if test_scenario['values']['addressType'] == 'HostnameConservative' %}
19+
- addressType: IP
20+
{% else %}
1621
- addressType: {{ test_scenario['values']['addressType'] }}
22+
{% endif %}
Lines changed: 4 additions & 46 deletions
Original file line numberDiff line numberDiff line change
@@ -1,47 +1,5 @@
11
---
2-
apiVersion: apps/v1
3-
kind: StatefulSet
4-
metadata:
5-
# Intentionally long name to trigger #110
6-
name: nginx-long-name-approaching-k8s-limits
7-
spec:
8-
serviceName: nginx
9-
selector:
10-
matchLabels:
11-
app: nginx
12-
replicas: 2
13-
template:
14-
metadata:
15-
labels:
16-
app: nginx
17-
spec:
18-
containers:
19-
- name: nginx
20-
image: nginxinc/nginx-unprivileged:1.25.2
21-
ports:
22-
- name: http
23-
containerPort: 8080
24-
volumeMounts:
25-
- name: listener
26-
mountPath: /listener
27-
- name: metadata
28-
mountPath: /usr/share/nginx/html/
29-
volumes:
30-
- name: metadata
31-
downwardAPI:
32-
items:
33-
- path: pod-name
34-
fieldRef:
35-
fieldPath: metadata.name
36-
volumeClaimTemplates:
37-
- metadata:
38-
name: listener
39-
annotations:
40-
listeners.stackable.tech/listener-class: listener-operator-test-smoke-nodeport
41-
spec:
42-
accessModes:
43-
- ReadWriteMany
44-
storageClassName: listeners.stackable.tech
45-
resources:
46-
requests:
47-
storage: 1
2+
apiVersion: kuttl.dev/v1beta1
3+
kind: TestStep
4+
commands:
5+
- script: envsubst '$NAMESPACE' < 10_nginx-statefulset.yaml | kubectl apply -n $NAMESPACE -f -
Lines changed: 47 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,47 @@
1+
---
2+
apiVersion: apps/v1
3+
kind: StatefulSet
4+
metadata:
5+
# Intentionally long name to trigger #110
6+
name: nginx-long-name-approaching-k8s-limits
7+
spec:
8+
serviceName: nginx
9+
selector:
10+
matchLabels:
11+
app: nginx
12+
replicas: 2
13+
template:
14+
metadata:
15+
labels:
16+
app: nginx
17+
spec:
18+
containers:
19+
- name: nginx
20+
image: nginxinc/nginx-unprivileged:1.25.2
21+
ports:
22+
- name: http
23+
containerPort: 8080
24+
volumeMounts:
25+
- name: listener
26+
mountPath: /listener
27+
- name: metadata
28+
mountPath: /usr/share/nginx/html/
29+
volumes:
30+
- name: metadata
31+
downwardAPI:
32+
items:
33+
- path: pod-name
34+
fieldRef:
35+
fieldPath: metadata.name
36+
volumeClaimTemplates:
37+
- metadata:
38+
name: listener
39+
annotations:
40+
listeners.stackable.tech/listener-class: listener-operator-test-smoke-nodeport-$NAMESPACE
41+
spec:
42+
accessModes:
43+
- ReadWriteMany
44+
storageClassName: listeners.stackable.tech
45+
resources:
46+
requests:
47+
storage: 1

tests/test-definition.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ dimensions:
77
values:
88
- IP
99
- Hostname
10+
- HostnameConservative
1011
tests:
1112
- name: smoke-nodeport
1213
dimensions:

0 commit comments

Comments
 (0)