Skip to content

Commit 236ea59

Browse files
authored
Fix SO version in override snapshot generation (#3360)
1 parent 5046379 commit 236ea59

File tree

1 file changed

+18
-14
lines changed

1 file changed

+18
-14
lines changed

hack/generate/override-snapshot.sh

Lines changed: 18 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ EOF
3030
}
3131

3232
function create_component_snapshot {
33-
local snapshot_file so_branch so_version so_semversion serving_tag serving_version_dotted serving_version tmp_catalog_dir max_ocp_version latest_index_image
33+
local snapshot_file so_short_version so_version so_semversion serving_tag serving_version_dotted serving_version tmp_catalog_dir max_ocp_version latest_index_image
3434
snapshot_file="${1}/override-snapshot.yaml"
3535

3636
serving_tag="$(metadata.get dependencies.serving)"
@@ -39,24 +39,26 @@ function create_component_snapshot {
3939
so_branch="$(sobranch --upstream-version "${serving_version_dotted}")"
4040
so_version="$(get_app_version_from_tag "${serving_tag}")"
4141
so_semversion="$(metadata.get project.version)"
42+
so_short_version=${so_semversion/./} # 1.36.0 -> 136.0
43+
so_short_version=${so_short_version%.*} # 136.0 -> 136
4244

4345
cat > "${snapshot_file}" <<EOF
4446
apiVersion: appstudio.redhat.com/v1alpha1
4547
kind: Snapshot
4648
metadata:
47-
name: serverless-operator-${so_version}-override-snapshot
49+
name: serverless-operator-${so_short_version}-override-snapshot
4850
labels:
4951
test.appstudio.openshift.io/type: override
50-
application: serverless-operator-${so_version}
52+
application: serverless-operator-${so_short_version}
5153
branch: ${so_branch}
5254
spec:
53-
application: serverless-operator-${so_version}
55+
application: serverless-operator-${so_short_version}
5456
EOF
5557

5658
tmp_catalog_dir=$(mktemp -d)
5759
max_ocp_version="$(metadata.get requirements.ocpVersion.max)"
5860
max_ocp_version=${max_ocp_version/./}
59-
latest_index_image="${registry_quay}-fbc-${max_ocp_version}/serverless-index-${so_version}-fbc-${max_ocp_version}:latest"
61+
latest_index_image="${registry_quay}-fbc-${max_ocp_version}/serverless-index-${so_short_version}-fbc-${max_ocp_version}:latest"
6062

6163
# get catalog from latest index, so we can get the referenced images from there
6264
opm migrate "${latest_index_image}" "${tmp_catalog_dir}" -o json
@@ -75,10 +77,10 @@ EOF
7577
if [[ $image == "serverless-operator-bundle" ]]; then
7678
# bundle component is named in konflux serverless-bundle-<version>
7779

78-
component_name="serverless-bundle-${so_version}"
80+
component_name="serverless-bundle-${so_short_version}"
7981
component_image_ref="${registry_quay}/serverless-bundle@${image_sha}"
8082
elif [[ $image =~ serverless ]]; then
81-
component_name="${image}-${so_version}"
83+
component_name="${image}-${so_short_version}"
8284
else
8385
component_name="${image}-${serving_version}"
8486
fi
@@ -95,13 +97,15 @@ EOF
9597
}
9698

9799
function create_fbc_snapshots {
98-
local rootdir snapshot_dir so_version so_branch serving_tag
100+
local rootdir snapshot_dir so_short_version so_branch serving_tag
99101
rootdir="$(dirname "$(dirname "$(dirname "$(realpath "${BASH_SOURCE[0]}")")")")"
100102
snapshot_dir="${1}"
101103

102104
serving_tag="$(metadata.get dependencies.serving)"
103105
so_branch="$(sobranch --upstream-version "${serving_tag/knative-v/}")"
104-
so_version=$(get_app_version_from_tag "${serving_tag}")
106+
so_short_version="$(metadata.get project.version)"
107+
so_short_version=${so_short_version/./} # 1.36.0 -> 136.0
108+
so_short_version=${so_short_version%.*} # 136.0 -> 136
105109

106110
while IFS= read -r ocp_version; do
107111
ocp_version=${ocp_version/./}
@@ -111,18 +115,18 @@ function create_fbc_snapshots {
111115
apiVersion: appstudio.redhat.com/v1alpha1
112116
kind: Snapshot
113117
metadata:
114-
name: serverless-operator-${so_version}-fbc-${ocp_version}-override-snapshot
118+
name: serverless-operator-${so_short_version}-fbc-${ocp_version}-override-snapshot
115119
labels:
116120
test.appstudio.openshift.io/type: override
117-
application: serverless-operator-${so_version}-fbc-${ocp_version}
121+
application: serverless-operator-${so_short_version}-fbc-${ocp_version}
118122
branch: ${so_branch}
119123
spec:
120-
application: serverless-operator-${so_version}-fbc-${ocp_version}
124+
application: serverless-operator-${so_short_version}-fbc-${ocp_version}
121125
EOF
122126

123-
index_image="${registry_quay}-fbc-${ocp_version}/serverless-index-${so_version}-fbc-${ocp_version}"
127+
index_image="${registry_quay}-fbc-${ocp_version}/serverless-index-${so_short_version}-fbc-${ocp_version}"
124128
index_image_digest="$(skopeo inspect --retry-times=10 --no-tags docker://"${index_image}:latest" | jq -r .Digest)"
125-
add_component "${snapshot_file}" "serverless-index-${so_version}-fbc-${ocp_version}" "${index_image}@${index_image_digest}"
129+
add_component "${snapshot_file}" "serverless-index-${so_short_version}-fbc-${ocp_version}" "${index_image}@${index_image_digest}"
126130

127131
append_hash_to_snapshot_name "${snapshot_file}"
128132

0 commit comments

Comments
 (0)