Skip to content

Commit 76f4e71

Browse files
test(backup-restore): Use truststore ConfigMap instead of TLS volumes
1 parent 0d7ee53 commit 76f4e71

File tree

8 files changed

+28
-122
lines changed

8 files changed

+28
-122
lines changed

tests/templates/kuttl/backup-restore/20-create-opensearch-1-admin-certificate.yaml

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,6 @@ spec:
1414
volumeMounts:
1515
- name: script
1616
mountPath: /stackable/scripts
17-
- name: tls
18-
mountPath: /stackable/tls
1917
securityContext:
2018
allowPrivilegeEscalation: false
2119
capabilities:
@@ -34,19 +32,6 @@ spec:
3432
configMap:
3533
name: create-opensearch-1-admin-certificate-script
3634
defaultMode: 0o770
37-
- name: tls
38-
ephemeral:
39-
volumeClaimTemplate:
40-
metadata:
41-
annotations:
42-
secrets.stackable.tech/class: tls
43-
spec:
44-
storageClassName: secrets.stackable.tech
45-
accessModes:
46-
- ReadWriteOnce
47-
resources:
48-
requests:
49-
storage: "1"
5035
serviceAccountName: test-service-account
5136
securityContext:
5237
fsGroup: 1000

tests/templates/kuttl/backup-restore/22-create-testuser.yaml

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -49,18 +49,8 @@ spec:
4949
configMap:
5050
name: create-testuser-script
5151
- name: tls
52-
ephemeral:
53-
volumeClaimTemplate:
54-
metadata:
55-
annotations:
56-
secrets.stackable.tech/class: tls
57-
spec:
58-
storageClassName: secrets.stackable.tech
59-
accessModes:
60-
- ReadWriteOnce
61-
resources:
62-
requests:
63-
storage: "1"
52+
configMap:
53+
name: truststore-pem
6454
serviceAccountName: test-service-account
6555
securityContext:
6656
fsGroup: 1000

tests/templates/kuttl/backup-restore/23-create-data.yaml

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -49,18 +49,8 @@ spec:
4949
configMap:
5050
name: create-data-script
5151
- name: tls
52-
ephemeral:
53-
volumeClaimTemplate:
54-
metadata:
55-
annotations:
56-
secrets.stackable.tech/class: tls
57-
spec:
58-
storageClassName: secrets.stackable.tech
59-
accessModes:
60-
- ReadWriteOnce
61-
resources:
62-
requests:
63-
storage: "1"
52+
configMap:
53+
name: truststore-pem
6454
serviceAccountName: test-service-account
6555
securityContext:
6656
fsGroup: 1000

tests/templates/kuttl/backup-restore/30-create-snapshot.yaml

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -49,18 +49,8 @@ spec:
4949
configMap:
5050
name: create-snapshot-script
5151
- name: tls
52-
ephemeral:
53-
volumeClaimTemplate:
54-
metadata:
55-
annotations:
56-
secrets.stackable.tech/class: tls
57-
spec:
58-
storageClassName: secrets.stackable.tech
59-
accessModes:
60-
- ReadWriteOnce
61-
resources:
62-
requests:
63-
storage: "1"
52+
configMap:
53+
name: truststore-pem
6454
serviceAccountName: test-service-account
6555
securityContext:
6656
fsGroup: 1000

tests/templates/kuttl/backup-restore/50-create-opensearch-2-admin-certificate.yaml

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,6 @@ spec:
1414
volumeMounts:
1515
- name: script
1616
mountPath: /stackable/scripts
17-
- name: tls
18-
mountPath: /stackable/tls
1917
securityContext:
2018
allowPrivilegeEscalation: false
2119
capabilities:
@@ -34,19 +32,6 @@ spec:
3432
configMap:
3533
name: create-opensearch-2-admin-certificate-script
3634
defaultMode: 0o770
37-
- name: tls
38-
ephemeral:
39-
volumeClaimTemplate:
40-
metadata:
41-
annotations:
42-
secrets.stackable.tech/class: tls
43-
spec:
44-
storageClassName: secrets.stackable.tech
45-
accessModes:
46-
- ReadWriteOnce
47-
resources:
48-
requests:
49-
storage: "1"
5035
serviceAccountName: test-service-account
5136
securityContext:
5237
fsGroup: 1000

tests/templates/kuttl/backup-restore/61-restore-snapshot.yaml

Lines changed: 20 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -49,18 +49,8 @@ spec:
4949
configMap:
5050
name: restore-snapshot-script
5151
- name: tls
52-
ephemeral:
53-
volumeClaimTemplate:
54-
metadata:
55-
annotations:
56-
secrets.stackable.tech/class: tls
57-
spec:
58-
storageClassName: secrets.stackable.tech
59-
accessModes:
60-
- ReadWriteOnce
61-
resources:
62-
requests:
63-
storage: "1"
52+
configMap:
53+
name: truststore-pem
6454
serviceAccountName: test-service-account
6555
securityContext:
6656
fsGroup: 1000
@@ -108,27 +98,21 @@ data:
10898
# Restore snapshot
10999
snapshot = "test_snapshot"
110100
111-
try:
112-
response = client.snapshot.restore(
113-
repository='snapshot_repository',
114-
snapshot=snapshot,
115-
# Do not restore the following indices:
116-
# - .opensearch_security will be restored with securityadmin.sh
117-
# - .plugins-ml-config already was created by the plugin
118-
# - .opensearch-sap-log-types-config already was created by the plugin
119-
#
120-
# see also https://github.com/opensearch-project/security-analytics/issues/1352
121-
body={
122-
'indices': ','.join([
123-
'-.opendistro_security',
124-
'-.plugins-ml-config',
125-
'-.opensearch-sap-log-types-config'
126-
])
127-
}
128-
)
129-
print(f'Restoring snapshot; {response=}')
130-
except RequestError as err:
131-
if err.error == 'invalid_snapshot_name_exception':
132-
print(f'The snapshot was already restored in a prior test run; {snapshot=}')
133-
else:
134-
raise
101+
response = client.snapshot.restore(
102+
repository='snapshot_repository',
103+
snapshot=snapshot,
104+
# Do not restore the following indices:
105+
# - .opensearch_security will be restored with securityadmin.sh
106+
# - .plugins-ml-config already was created by the plugin
107+
# - .opensearch-sap-log-types-config already was created by the plugin
108+
#
109+
# see also https://github.com/opensearch-project/security-analytics/issues/1352
110+
body={
111+
'indices': ','.join([
112+
'-.opendistro_security',
113+
'-.plugins-ml-config',
114+
'-.opensearch-sap-log-types-config'
115+
])
116+
}
117+
)
118+
print(f'Restoring snapshot; {response=}')

tests/templates/kuttl/backup-restore/70-test-opensearch-2.yaml

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -49,18 +49,8 @@ spec:
4949
configMap:
5050
name: test-opensearch-2
5151
- name: tls
52-
ephemeral:
53-
volumeClaimTemplate:
54-
metadata:
55-
annotations:
56-
secrets.stackable.tech/class: tls
57-
spec:
58-
storageClassName: secrets.stackable.tech
59-
accessModes:
60-
- ReadWriteOnce
61-
resources:
62-
requests:
63-
storage: "1"
52+
configMap:
53+
name: truststore-pem
6454
serviceAccountName: test-service-account
6555
securityContext:
6656
fsGroup: 1000

tests/templates/kuttl/backup-restore/README.md

Lines changed: 0 additions & 8 deletions
This file was deleted.

0 commit comments

Comments
 (0)