Skip to content

Commit 4e375f9

Browse files
committed
Fix the review comments
1 parent e61f314 commit 4e375f9

File tree

11 files changed

+50
-28
lines changed

11 files changed

+50
-28
lines changed

config/manager/patches/image.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -188,6 +188,8 @@ spec:
188188
value: cockroachdb/cockroach:v22.1.15
189189
- name: RELATED_IMAGE_COCKROACH_v22_1_16
190190
value: cockroachdb/cockroach:v22.1.16
191+
- name: RELATED_IMAGE_COCKROACH_v22_1_18
192+
value: cockroachdb/cockroach:v22.1.18
191193
- name: RELATED_IMAGE_COCKROACH_v22_2_0
192194
value: cockroachdb/cockroach:v22.2.0
193195
- name: RELATED_IMAGE_COCKROACH_v22_2_1
@@ -202,3 +204,5 @@ spec:
202204
value: cockroachdb/cockroach:v22.2.5
203205
- name: RELATED_IMAGE_COCKROACH_v22_2_6
204206
value: cockroachdb/cockroach:v22.2.6
207+
- name: RELATED_IMAGE_COCKROACH_v22_2_7
208+
value: cockroachdb/cockroach:v22.2.7

config/manifests/bases/cockroach-operator.clusterserviceversion.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -345,6 +345,8 @@ spec:
345345
image: registry.connect.redhat.com/cockroachdb/cockroach@sha256:0ec1dc184a2199736b4bd7de3241a357be7427caea5da1c931b08bc06f5c3dc0
346346
- name: RELATED_IMAGE_COCKROACH_v22_1_16
347347
image: registry.connect.redhat.com/cockroachdb/cockroach@sha256:1d590946b2f7c7ec0c7ec057a338e81f864d6a9e3d3ac4e0769aa5756a8e13fc
348+
- name: RELATED_IMAGE_COCKROACH_v22_1_18
349+
image: registry.connect.redhat.com/cockroachdb/cockroach@sha256:363d8feb560324a848c5771edd224dc19020013af7641979ee81ae8ee536d7e3
348350
- name: RELATED_IMAGE_COCKROACH_v22_2_0
349351
image: registry.connect.redhat.com/cockroachdb/cockroach@sha256:e50aab39722af22ccbf3d1db19e37972d52b0fc9a40998a64618dc966b2bac57
350352
- name: RELATED_IMAGE_COCKROACH_v22_2_1
@@ -359,3 +361,5 @@ spec:
359361
image: registry.connect.redhat.com/cockroachdb/cockroach@sha256:0fea6ed8a78ea624240ee6ffb3573d5cf1115186aa9180fcd7d1273351deaaa3
360362
- name: RELATED_IMAGE_COCKROACH_v22_2_6
361363
image: registry.connect.redhat.com/cockroachdb/cockroach@sha256:53c7b960a2ed70a0998a4f8e28a4f78f7f676e01cffebe665e91e3a7629d88ed
364+
- name: RELATED_IMAGE_COCKROACH_v22_2_7
365+
image: registry.connect.redhat.com/cockroachdb/cockroach@sha256:d14f77f70aea0422a906510c1ac2652ff76e131ed705cb002f80e13e887b0cd9

config/manifests/patches/deployment_patch.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -197,6 +197,8 @@ spec:
197197
value: registry.connect.redhat.com/cockroachdb/cockroach@sha256:0ec1dc184a2199736b4bd7de3241a357be7427caea5da1c931b08bc06f5c3dc0
198198
- name: RELATED_IMAGE_COCKROACH_v22_1_16
199199
value: registry.connect.redhat.com/cockroachdb/cockroach@sha256:1d590946b2f7c7ec0c7ec057a338e81f864d6a9e3d3ac4e0769aa5756a8e13fc
200+
- name: RELATED_IMAGE_COCKROACH_v22_1_18
201+
value: registry.connect.redhat.com/cockroachdb/cockroach@sha256:363d8feb560324a848c5771edd224dc19020013af7641979ee81ae8ee536d7e3
200202
- name: RELATED_IMAGE_COCKROACH_v22_2_0
201203
value: registry.connect.redhat.com/cockroachdb/cockroach@sha256:e50aab39722af22ccbf3d1db19e37972d52b0fc9a40998a64618dc966b2bac57
202204
- name: RELATED_IMAGE_COCKROACH_v22_2_1
@@ -211,4 +213,6 @@ spec:
211213
value: registry.connect.redhat.com/cockroachdb/cockroach@sha256:0fea6ed8a78ea624240ee6ffb3573d5cf1115186aa9180fcd7d1273351deaaa3
212214
- name: RELATED_IMAGE_COCKROACH_v22_2_6
213215
value: registry.connect.redhat.com/cockroachdb/cockroach@sha256:53c7b960a2ed70a0998a4f8e28a4f78f7f676e01cffebe665e91e3a7629d88ed
216+
- name: RELATED_IMAGE_COCKROACH_v22_2_7
217+
value: registry.connect.redhat.com/cockroachdb/cockroach@sha256:d14f77f70aea0422a906510c1ac2652ff76e131ed705cb002f80e13e887b0cd9
214218
image: RH_COCKROACH_OP_IMAGE_PLACEHOLDER

config/samples/crdb-tls-example.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ kind: CrdbCluster
1919
metadata:
2020
name: crdb-tls-example
2121
spec:
22-
cockroachDBVersion: v22.2.6
22+
cockroachDBVersion: v22.2.7
2323
dataStore:
2424
pvc:
2525
spec:

crdb-versions.yaml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -265,6 +265,9 @@ CrdbVersions:
265265
- image: cockroachdb/cockroach:v22.1.16
266266
redhatImage: registry.connect.redhat.com/cockroachdb/cockroach@sha256:1d590946b2f7c7ec0c7ec057a338e81f864d6a9e3d3ac4e0769aa5756a8e13fc
267267
tag: v22.1.16
268+
- image: cockroachdb/cockroach:v22.1.18
269+
redhatImage: registry.connect.redhat.com/cockroachdb/cockroach@sha256:363d8feb560324a848c5771edd224dc19020013af7641979ee81ae8ee536d7e3
270+
tag: v22.1.18
268271
- image: cockroachdb/cockroach:v22.2.0
269272
redhatImage: registry.connect.redhat.com/cockroachdb/cockroach@sha256:e50aab39722af22ccbf3d1db19e37972d52b0fc9a40998a64618dc966b2bac57
270273
tag: v22.2.0
@@ -286,3 +289,6 @@ CrdbVersions:
286289
- image: cockroachdb/cockroach:v22.2.6
287290
redhatImage: registry.connect.redhat.com/cockroachdb/cockroach@sha256:53c7b960a2ed70a0998a4f8e28a4f78f7f676e01cffebe665e91e3a7629d88ed
288291
tag: v22.2.6
292+
- image: cockroachdb/cockroach:v22.2.7
293+
redhatImage: registry.connect.redhat.com/cockroachdb/cockroach@sha256:d14f77f70aea0422a906510c1ac2652ff76e131ed705cb002f80e13e887b0cd9
294+
tag: v22.2.7

examples/client-secure-operator.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ spec:
2323
serviceAccountName: cockroachdb-sa
2424
containers:
2525
- name: cockroachdb-client-secure
26-
image: cockroachdb/cockroach:v22.2.6
26+
image: cockroachdb/cockroach:v22.2.7
2727
imagePullPolicy: IfNotPresent
2828
volumeMounts:
2929
- name: client-certs

examples/example.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,9 +40,9 @@ spec:
4040
memory: 8Gi
4141
tlsEnabled: true
4242
# You can set either a version of the db or a specific image name
43-
# cockroachDBVersion: v22.2.6
43+
# cockroachDBVersion: v22.2.7
4444
image:
45-
name: cockroachdb/cockroach:v22.2.6
45+
name: cockroachdb/cockroach:v22.2.7
4646
# nodes refers to the number of crdb pods that are created
4747
# via the statefulset
4848
nodes: 3

examples/smoketest.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,5 +39,5 @@ spec:
3939
memory: 300Mi
4040
tlsEnabled: true
4141
image:
42-
name: cockroachdb/cockroach:v22.2.6
42+
name: cockroachdb/cockroach:v22.2.7
4343
nodes: 3

hack/bundle.sh

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -70,17 +70,17 @@ generate_bundle() {
7070

7171
# Generate the new package bundle
7272
kustomize build config/manifests | operator-sdk generate bundle -q \
73-
--version "${version}" \
74-
${opts} \
75-
--output-dir "${dir}"
73+
--version "${version}" \
74+
${opts} \
75+
--output-dir "${dir}"
7676

7777
# Ensure package name is correct for the specific bundle and that the CSV name matches the package name. Also removing
7878
# the testing annotations since these are handled automatically upstream.
7979
sed \
80-
-e "s/package.v1: cockroach-operator/package.v1: ${pkg}/g" \
81-
-e "/\s*# Annotations for testing/d" \
82-
-e "/\s*operators.operatorframework.io.test/d" \
83-
"${dir}/metadata/annotations.yaml" > "${dir}/metadata/annotations.yaml.new"
80+
-e "s/package.v1: cockroach-operator/package.v1: ${pkg}/g" \
81+
-e "/\s*# Annotations for testing/d" \
82+
-e "/\s*operators.operatorframework.io.test/d" \
83+
"${dir}/metadata/annotations.yaml" > "${dir}/metadata/annotations.yaml.new"
8484

8585
mv "${dir}/metadata/annotations.yaml.new" "${dir}/metadata/annotations.yaml"
8686

@@ -89,9 +89,9 @@ generate_bundle() {
8989

9090
# move the dockerfile into the bundle directory and make it valid
9191
sed \
92-
-e "/\s*tests\/scorecard/d" \
93-
-e "s+${dir}/++g" \
94-
bundle.Dockerfile > "${dir}/Dockerfile"
92+
-e "/\s*tests\/scorecard/d" \
93+
-e "s+${dir}/++g" \
94+
bundle.Dockerfile > "${dir}/Dockerfile"
9595

9696
rm bundle.Dockerfile
9797
rm "${dir}/manifests/cockroach-operator-webhook-service_v1_service.yaml"
@@ -108,9 +108,9 @@ adapt_csv() {
108108

109109
# replace RH_COCKROACH_OP_IMAGE_PLACEHOLDER with the proper image and CREATED_AT_PLACEHOLDER with the current time
110110
sed \
111-
-e "s+RH_COCKROACH_OP_IMAGE_PLACEHOLDER+${img}+g" \
112-
-e "s+CREATED_AT_PLACEHOLDER+"$(date +"%FT%H:%M:%SZ")"+g" \
113-
"${csv}" > "${csv}.new"
111+
-e "s+RH_COCKROACH_OP_IMAGE_PLACEHOLDER+${img}+g" \
112+
-e "s+CREATED_AT_PLACEHOLDER+"$(date +"%FT%H:%M:%SZ")"+g" \
113+
"${csv}" > "${csv}.new"
114114

115115
mv "${csv}.new" "${csv}"
116116
}
@@ -122,7 +122,7 @@ patch_marketplace_csv() {
122122

123123
local csv="${1}/cockroachdb-certified-rhmp/manifests/${2}.clusterserviceversion.yaml"
124124
sed "s+ annotations:+ annotations:\n${rhmp_annotations}+" \
125-
"${csv}" > "${csv}.new"
125+
"${csv}" > "${csv}.new"
126126
mv "${csv}.new" "${csv}"
127127
}
128128

hack/redhat.sh

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ main() {
4343

4444
case "${1:-}" in
4545
operator)
46-
publish_operator_image
46+
publish_operator_image
4747
preflight_operator;;
4848
bundle)
4949
publish_bundle_image "${REGISTRY}/${BUNDLE_IMAGE}" "bundle/cockroachdb-certified"
@@ -65,15 +65,15 @@ main() {
6565
publish_operator_image() {
6666
echo "Publishing operator image to local repo..."
6767
APP_VERSION="v${VERSION}" \
68-
DOCKER_REGISTRY="${REGISTRY}" \
69-
DOCKER_IMAGE_REPOSITORY="${IMAGE%:*}" \
70-
bazel run --stamp --platforms=@io_bazel_rules_go//go/toolchain:linux_amd64 //:push_operator_image
68+
DOCKER_REGISTRY="${REGISTRY}" \
69+
DOCKER_IMAGE_REPOSITORY="${IMAGE%:*}" \
70+
bazel run --stamp --platforms=@io_bazel_rules_go//go/toolchain:linux_amd64 //:push_operator_image
7171
}
7272

7373
preflight_operator() {
7474
echo "Running preflight checks on operator image..."
7575
preflight check container "${REGISTRY}/${IMAGE}" \
76-
--docker-config "${HOME}/.docker/config.json"
76+
--docker-config "${HOME}/.docker/config.json"
7777
}
7878

7979
publish_bundle_image() {
@@ -93,17 +93,17 @@ publish_bundle_index() {
9393

9494
echo "Publishing ${index_img}..."
9595
opm index add \
96-
--container-tool docker \
97-
--bundles "${bundle_img}" \
98-
--tag "${index_img}"
96+
--container-tool docker \
97+
--bundles "${bundle_img}" \
98+
--tag "${index_img}"
9999
}
100100

101101
preflight_bundle() {
102102
local bundle_img="${1}"
103103
local index_img="${2}"
104104

105105
echo "Running preflight checks on bundle image..."
106-
echo " IMAGE: ${bundle_img}"
106+
echo "IMAGE: ${bundle_img}"
107107

108108
PFLT_INDEXIMAGE="${index_img}" preflight check operator "${bundle_img}"
109109
}

install/operator.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -546,6 +546,8 @@ spec:
546546
value: cockroachdb/cockroach:v22.1.15
547547
- name: RELATED_IMAGE_COCKROACH_v22_1_16
548548
value: cockroachdb/cockroach:v22.1.16
549+
- name: RELATED_IMAGE_COCKROACH_v22_1_18
550+
value: cockroachdb/cockroach:v22.1.18
549551
- name: RELATED_IMAGE_COCKROACH_v22_2_0
550552
value: cockroachdb/cockroach:v22.2.0
551553
- name: RELATED_IMAGE_COCKROACH_v22_2_1
@@ -560,6 +562,8 @@ spec:
560562
value: cockroachdb/cockroach:v22.2.5
561563
- name: RELATED_IMAGE_COCKROACH_v22_2_6
562564
value: cockroachdb/cockroach:v22.2.6
565+
- name: RELATED_IMAGE_COCKROACH_v22_2_7
566+
value: cockroachdb/cockroach:v22.2.7
563567
- name: OPERATOR_NAME
564568
value: cockroachdb
565569
- name: POD_NAME

0 commit comments

Comments
 (0)