Skip to content

Commit 2762688

Browse files
authored
Merge pull request #685 from jgehrcke/jp/tests-v1-exactly
tests: construct ResourceClaim differently on v1
2 parents 1516ec7 + 784ba18 commit 2762688

File tree

2 files changed

+11
-2
lines changed

2 files changed

+11
-2
lines changed

tests/bats/setup_suite.bash

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,13 @@ validate_prerequisites() {
4747

4848
# Examples: v1, or v1beta1
4949
export TEST_K8S_RESOURCE_API_VERSION
50+
51+
# For resource.k8s.io API versions before v1, a ResourceClaim spec looks
52+
# slightly differently. Use flow-style YAML.
53+
export TEST_DEVCLASSNAME_CD_CHANNEL="exactly: {deviceClassName: compute-domain-default-channel.nvidia.com}"
54+
if [[ "${TEST_K8S_RESOURCE_API_VERSION}" != "v1" ]]; then
55+
export TEST_DEVCLASSNAME_CD_CHANNEL="deviceClassName: compute-domain-default-channel.nvidia.com"
56+
fi
5057
}
5158

5259

tests/bats/specs/rc-opaque-cfg-unknown-field.yaml.tmpl

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,10 @@ metadata:
55
spec:
66
devices:
77
requests:
8-
- deviceClassName: compute-domain-default-channel.nvidia.com
9-
name: chan
8+
- name: chan
9+
# This part of the ResourceClaim spec changed in
10+
# resource.k8s.io/v1 compared to previous API versions.
11+
${TEST_DEVCLASSNAME_CD_CHANNEL}
1012
config:
1113
- opaque:
1214
driver: compute-domain.nvidia.com

0 commit comments

Comments
 (0)