Skip to content

Commit 7e403ab

Browse files
committed
fix: allow to set architecture also when compute-sizes are defined
previously architecture could only be configured when compute-sizes were not defined Signed-off-by: Pavol Pitonak <ppitonak@redhat.com>
1 parent e7abf72 commit 7e403ab

8 files changed

Lines changed: 8 additions & 8 deletions

tkn/infra-aws-fedora.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -236,7 +236,6 @@ spec:
236236
if [[ "$(params.compute-sizes)" != "" ]]; then
237237
cmd+="--compute-sizes '$(params.compute-sizes)' "
238238
else
239-
cmd+="--arch '$(params.arch)' "
240239
cmd+="--cpus '$(params.cpus)' "
241240
cmd+="--memory '$(params.memory)' "
242241
cmd+="--gpus '$(params.gpus)' "
@@ -245,6 +244,7 @@ spec:
245244
if [[ "$(params.nested-virt)" == "true" ]]; then
246245
cmd+="--nested-virt "
247246
fi
247+
cmd+="--arch '$(params.arch)' "
248248
cmd+="--version '$(params.version)' "
249249
if [[ -f /opt/rh-account-secret/user ]]; then
250250
cmd+="--rh-subscription-username '$(cat /opt/rh-account-secret/user)' "

tkn/infra-aws-rhel.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,6 @@ spec:
258258
if [[ "$(params.compute-sizes)" != "" ]]; then
259259
cmd+="--compute-sizes '$(params.compute-sizes)' "
260260
else
261-
cmd+="--arch '$(params.arch)' "
262261
cmd+="--cpus '$(params.cpus)' "
263262
cmd+="--memory '$(params.memory)' "
264263
cmd+="--gpus '$(params.gpus)' "
@@ -267,6 +266,7 @@ spec:
267266
if [[ "$(params.nested-virt)" == "true" ]]; then
268267
cmd+="--nested-virt "
269268
fi
269+
cmd+="--arch '$(params.arch)' "
270270
cmd+="--version '$(params.version)' "
271271
if [[ -f /opt/rh-account-secret/user ]]; then
272272
cmd+="--rh-subscription-username '$(cat /opt/rh-account-secret/user)' "

tkn/infra-azure-fedora.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -223,7 +223,6 @@ spec:
223223
if [[ "$(params.compute-sizes)" != "" ]]; then
224224
cmd+="--compute-sizes '$(params.compute-sizes)' "
225225
else
226-
cmd+="--arch $(params.arch) "
227226
cmd+="--cpus '$(params.cpus)' "
228227
cmd+="--memory '$(params.memory)' "
229228
cmd+="--gpus '$(params.gpus)' "
@@ -232,6 +231,7 @@ spec:
232231
if [[ "$(params.nested-virt)" == "true" ]]; then
233232
cmd+="--nested-virt "
234233
fi
234+
cmd+="--arch '$(params.arch)' "
235235
cmd+="--version '$(params.version)' "
236236
if [[ "$(params.spot)" == "true" ]]; then
237237
cmd+="--spot "

tkn/infra-azure-rhel.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,6 @@ spec:
227227
if [[ "$(params.compute-sizes)" != "" ]]; then
228228
cmd+="--compute-sizes '$(params.compute-sizes)' "
229229
else
230-
cmd+="--arch '$(params.arch)' "
231230
cmd+="--cpus '$(params.cpus)' "
232231
cmd+="--memory '$(params.memory)' "
233232
cmd+="--gpus '$(params.gpus)' "
@@ -236,6 +235,7 @@ spec:
236235
if [[ "$(params.nested-virt)" == "true" ]]; then
237236
cmd+="--nested-virt "
238237
fi
238+
cmd+="--arch '$(params.arch)' "
239239
cmd+="--version '$(params.version)' "
240240
if [[ "$(params.spot)" == "true" ]]; then
241241
cmd+="--spot "

tkn/template/infra-aws-fedora.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -236,7 +236,6 @@ spec:
236236
if [[ "$(params.compute-sizes)" != "" ]]; then
237237
cmd+="--compute-sizes '$(params.compute-sizes)' "
238238
else
239-
cmd+="--arch '$(params.arch)' "
240239
cmd+="--cpus '$(params.cpus)' "
241240
cmd+="--memory '$(params.memory)' "
242241
cmd+="--gpus '$(params.gpus)' "
@@ -245,6 +244,7 @@ spec:
245244
if [[ "$(params.nested-virt)" == "true" ]]; then
246245
cmd+="--nested-virt "
247246
fi
247+
cmd+="--arch '$(params.arch)' "
248248
cmd+="--version '$(params.version)' "
249249
if [[ -f /opt/rh-account-secret/user ]]; then
250250
cmd+="--rh-subscription-username '$(cat /opt/rh-account-secret/user)' "

tkn/template/infra-aws-rhel.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,6 @@ spec:
258258
if [[ "$(params.compute-sizes)" != "" ]]; then
259259
cmd+="--compute-sizes '$(params.compute-sizes)' "
260260
else
261-
cmd+="--arch '$(params.arch)' "
262261
cmd+="--cpus '$(params.cpus)' "
263262
cmd+="--memory '$(params.memory)' "
264263
cmd+="--gpus '$(params.gpus)' "
@@ -267,6 +266,7 @@ spec:
267266
if [[ "$(params.nested-virt)" == "true" ]]; then
268267
cmd+="--nested-virt "
269268
fi
269+
cmd+="--arch '$(params.arch)' "
270270
cmd+="--version '$(params.version)' "
271271
if [[ -f /opt/rh-account-secret/user ]]; then
272272
cmd+="--rh-subscription-username '$(cat /opt/rh-account-secret/user)' "

tkn/template/infra-azure-fedora.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -223,7 +223,6 @@ spec:
223223
if [[ "$(params.compute-sizes)" != "" ]]; then
224224
cmd+="--compute-sizes '$(params.compute-sizes)' "
225225
else
226-
cmd+="--arch $(params.arch) "
227226
cmd+="--cpus '$(params.cpus)' "
228227
cmd+="--memory '$(params.memory)' "
229228
cmd+="--gpus '$(params.gpus)' "
@@ -232,6 +231,7 @@ spec:
232231
if [[ "$(params.nested-virt)" == "true" ]]; then
233232
cmd+="--nested-virt "
234233
fi
234+
cmd+="--arch '$(params.arch)' "
235235
cmd+="--version '$(params.version)' "
236236
if [[ "$(params.spot)" == "true" ]]; then
237237
cmd+="--spot "

tkn/template/infra-azure-rhel.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,6 @@ spec:
227227
if [[ "$(params.compute-sizes)" != "" ]]; then
228228
cmd+="--compute-sizes '$(params.compute-sizes)' "
229229
else
230-
cmd+="--arch '$(params.arch)' "
231230
cmd+="--cpus '$(params.cpus)' "
232231
cmd+="--memory '$(params.memory)' "
233232
cmd+="--gpus '$(params.gpus)' "
@@ -236,6 +235,7 @@ spec:
236235
if [[ "$(params.nested-virt)" == "true" ]]; then
237236
cmd+="--nested-virt "
238237
fi
238+
cmd+="--arch '$(params.arch)' "
239239
cmd+="--version '$(params.version)' "
240240
if [[ "$(params.spot)" == "true" ]]; then
241241
cmd+="--spot "

0 commit comments

Comments
 (0)