Skip to content

Commit dd4c2f2

Browse files
fix: Fix the key of the restarter annotation
1 parent 0c51fd2 commit dd4c2f2

3 files changed

Lines changed: 22 additions & 10 deletions

File tree

rust/operator-binary/src/controller/build/role_group_builder.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -386,7 +386,7 @@ impl<'a> RoleGroupBuilder<'a> {
386386
.enumerate()
387387
.map(|(i, config_map_name)| {
388388
(
389-
format!("restarter.stackable.tech/ignore-config-map.{i}"),
389+
format!("restarter.stackable.tech/ignore-configmap.{i}"),
390390
config_map_name.to_string(),
391391
)
392392
});
@@ -1787,8 +1787,8 @@ mod tests {
17871787

17881788
let expected_annotations = match security_mode {
17891789
TestSecurityMode::Initializing | TestSecurityMode::Managing => json!({
1790-
"restarter.stackable.tech/ignore-config-map.0": "my-opensearch-cluster-security-config",
1791-
"restarter.stackable.tech/ignore-config-map.1": "opensearch-security-config",
1790+
"restarter.stackable.tech/ignore-configmap.0": "my-opensearch-cluster-security-config",
1791+
"restarter.stackable.tech/ignore-configmap.1": "opensearch-security-config",
17921792
"restarter.stackable.tech/ignore-secret.0": "opensearch-security-config",
17931793
}),
17941794
TestSecurityMode::Disabled | TestSecurityMode::Participating => json!({}),

tests/templates/kuttl/security-config/11-assert.yaml

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,19 @@ kind: StatefulSet
2424
metadata:
2525
name: opensearch-nodes-security-config
2626
annotations:
27-
restarter.stackable.tech/ignore-config-map.0: "opensearch-security-config"
27+
restarter.stackable.tech/ignore-configmap.0: opensearch-security-config
28+
restarter.stackable.tech/ignore-configmap.1: security-config
29+
restarter.stackable.tech/ignore-secret.0: security-config-file-internal-users
30+
labels:
31+
restarter.stackable.tech/enabled: "true"
32+
spec:
33+
template:
34+
metadata:
35+
annotations:
36+
# configmap.restarter.stackable.tech/opensearch-nodes-security-config: <uid>/<resourceVersion>
37+
configmap.restarter.stackable.tech/opensearch-security-config: changes-ignored
38+
configmap.restarter.stackable.tech/security-config: changes-ignored
39+
secret.restarter.stackable.tech/security-config-file-internal-users: changes-ignored
2840
status:
2941
readyReplicas: 1
3042
replicas: 1

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

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ kind: StatefulSet
1111
metadata:
1212
name: opensearch-nodes-cluster-manager
1313
annotations:
14-
restarter.stackable.tech/ignore-config-map.0: opensearch-security-config
14+
restarter.stackable.tech/ignore-configmap.0: opensearch-security-config
1515
labels:
1616
app.kubernetes.io/component: nodes
1717
app.kubernetes.io/instance: opensearch
@@ -40,8 +40,8 @@ spec:
4040
template:
4141
metadata:
4242
annotations:
43-
# configmap.restarter.stackable.tech/opensearch-nodes-cluster-manager: <UID>
44-
# configmap.restarter.stackable.tech/opensearch-security-config: <UID>
43+
# configmap.restarter.stackable.tech/opensearch-nodes-cluster-manager: <uid>/<resourceVersion>
44+
configmap.restarter.stackable.tech/opensearch-security-config: changes-ignored
4545
kubectl.kubernetes.io/default-container: opensearch
4646
labels:
4747
app.kubernetes.io/component: nodes
@@ -493,7 +493,7 @@ kind: StatefulSet
493493
metadata:
494494
name: opensearch-nodes-data
495495
annotations:
496-
restarter.stackable.tech/ignore-config-map.0: opensearch-security-config
496+
restarter.stackable.tech/ignore-configmap.0: opensearch-security-config
497497
labels:
498498
app.kubernetes.io/component: nodes
499499
app.kubernetes.io/instance: opensearch
@@ -522,8 +522,8 @@ spec:
522522
template:
523523
metadata:
524524
annotations:
525-
# configmap.restarter.stackable.tech/opensearch-nodes-data: <UID>
526-
# configmap.restarter.stackable.tech/opensearch-security-config: <UID>
525+
# configmap.restarter.stackable.tech/opensearch-nodes-data: <uid>/<resourceVersion>
526+
configmap.restarter.stackable.tech/opensearch-security-config: changes-ignored
527527
kubectl.kubernetes.io/default-container: opensearch
528528
labels:
529529
app.kubernetes.io/component: nodes

0 commit comments

Comments
 (0)