diff --git a/test/acceptance/_helpers.bash b/test/acceptance/_helpers.bash index c13f09d..2f62964 100644 --- a/test/acceptance/_helpers.bash +++ b/test/acceptance/_helpers.bash @@ -23,7 +23,7 @@ helm_install() { helm install -f ${values} \ --name openbao \ - ${BATS_TEST_DIRNAME}/../.. + ${BATS_TEST_DIRNAME}/../../charts/openbao } # helm_install_ha installs the openbao chart using HA mode. This will source @@ -40,7 +40,7 @@ helm_install_ha() { --name openbao \ --set 'server.enabled=false' \ --set 'serverHA.enabled=true' \ - ${BATS_TEST_DIRNAME}/../.. + ${BATS_TEST_DIRNAME}/../../charts/openbao } # wait for consul to be ready diff --git a/test/unit/csi-clusterrole.bats b/test/unit/csi-clusterrole.bats index 73d91ba..60346b8 100644 --- a/test/unit/csi-clusterrole.bats +++ b/test/unit/csi-clusterrole.bats @@ -30,4 +30,4 @@ load _helpers . | tee /dev/stderr | yq -r '.metadata.name' | tee /dev/stderr) [ "${actual}" = "release-name-openbao-csi-provider-clusterrole" ] -} \ No newline at end of file +} diff --git a/test/unit/csi-clusterrolebinding.bats b/test/unit/csi-clusterrolebinding.bats index b97ea3b..522c7ac 100644 --- a/test/unit/csi-clusterrolebinding.bats +++ b/test/unit/csi-clusterrolebinding.bats @@ -61,4 +61,4 @@ load _helpers . | tee /dev/stderr | yq -r '.subjects[0].namespace' | tee /dev/stderr) [ "${actual}" = "bar" ] -} \ No newline at end of file +} diff --git a/test/unit/csi-daemonset.bats b/test/unit/csi-daemonset.bats index 656c136..78daa80 100644 --- a/test/unit/csi-daemonset.bats +++ b/test/unit/csi-daemonset.bats @@ -101,7 +101,7 @@ load _helpers local actual=$(echo $object | yq -r '.[0].image' | tee /dev/stderr) - [ "${actual}" = "Image1:0.0.1" ] + [ "${actual}" = "docker.io/Image1:0.0.1" ] local actual=$(echo $object | yq -r '.[0].imagePullPolicy' | tee /dev/stderr) [ "${actual}" = "PullPolicy1" ] diff --git a/test/unit/csi-rolebinding.bats b/test/unit/csi-rolebinding.bats index d1c7b1a..692ea38 100644 --- a/test/unit/csi-rolebinding.bats +++ b/test/unit/csi-rolebinding.bats @@ -38,4 +38,4 @@ load _helpers . | tee /dev/stderr | yq -r '.metadata.namespace' | tee /dev/stderr) [ "${actual}" = "bar" ] -} \ No newline at end of file +} diff --git a/test/unit/injector-clusterrolebinding.bats b/test/unit/injector-clusterrolebinding.bats index e997ebd..9a99b34 100755 --- a/test/unit/injector-clusterrolebinding.bats +++ b/test/unit/injector-clusterrolebinding.bats @@ -38,4 +38,4 @@ load _helpers . | tee /dev/stderr | yq -r '.subjects[0].namespace' | tee /dev/stderr) [ "${actual}" = "bar" ] -} \ No newline at end of file +} diff --git a/test/unit/injector-deployment.bats b/test/unit/injector-deployment.bats index ba23bd9..f5d9d1e 100755 --- a/test/unit/injector-deployment.bats +++ b/test/unit/injector-deployment.bats @@ -69,7 +69,7 @@ load _helpers --set 'injector.image.tag=1.2.3' \ . | tee /dev/stderr | yq -r '.spec.template.spec.containers[0].image' | tee /dev/stderr) - [ "${actual}" = "foo:1.2.3" ] + [ "${actual}" = "docker.io/foo:1.2.3" ] local actual=$(helm template \ --show-only templates/injector-deployment.yaml \ @@ -77,7 +77,7 @@ load _helpers --set 'injector.image.tag=1.2.3' \ . | tee /dev/stderr | yq -r '.spec.template.spec.containers[0].image' | tee /dev/stderr) - [ "${actual}" = "foo:1.2.3" ] + [ "${actual}" = "docker.io/foo:1.2.3" ] } @test "injector/deployment: default imagePullPolicy" { diff --git a/test/unit/injector-mutating-webhook.bats b/test/unit/injector-mutating-webhook.bats index fcf4e7b..bd0499b 100755 --- a/test/unit/injector-mutating-webhook.bats +++ b/test/unit/injector-mutating-webhook.bats @@ -331,4 +331,4 @@ load _helpers yq '.webhooks[0].objectSelector.matchLabels.injector' | tee /dev/stderr) [ "${actual}" = "true" ] -} \ No newline at end of file +} diff --git a/test/unit/injector-psp-role.bats b/test/unit/injector-psp-role.bats index 3dda504..32bb696 100644 --- a/test/unit/injector-psp-role.bats +++ b/test/unit/injector-psp-role.bats @@ -53,4 +53,4 @@ load _helpers . | tee /dev/stderr | yq -r '.metadata.namespace' | tee /dev/stderr) [ "${actual}" = "bar" ] -} \ No newline at end of file +} diff --git a/test/unit/injector-psp-rolebinding.bats b/test/unit/injector-psp-rolebinding.bats index 62afe7b..f978acf 100644 --- a/test/unit/injector-psp-rolebinding.bats +++ b/test/unit/injector-psp-rolebinding.bats @@ -53,4 +53,4 @@ load _helpers . | tee /dev/stderr | yq -r '.metadata.namespace' | tee /dev/stderr) [ "${actual}" = "bar" ] -} \ No newline at end of file +} diff --git a/test/unit/server-clusterrolebinding.bats b/test/unit/server-clusterrolebinding.bats index 12cd6a9..b5c6930 100755 --- a/test/unit/server-clusterrolebinding.bats +++ b/test/unit/server-clusterrolebinding.bats @@ -87,4 +87,4 @@ load _helpers . | tee /dev/stderr | yq -r '.subjects[0].namespace' | tee /dev/stderr) [ "${actual}" = "bar" ] -} \ No newline at end of file +} diff --git a/test/unit/server-dev-statefulset.bats b/test/unit/server-dev-statefulset.bats index 025495a..19c94a0 100755 --- a/test/unit/server-dev-statefulset.bats +++ b/test/unit/server-dev-statefulset.bats @@ -43,7 +43,7 @@ load _helpers --set 'server.dev.enabled=true' \ . | tee /dev/stderr | yq -r '.spec.template.spec.containers[0].image' | tee /dev/stderr) - [ "${actual}" = "foo:1.2.3" ] + [ "${actual}" = "quay.io/foo:1.2.3" ] } @test "server/ha-StatefulSet: image tag defaults to latest" { @@ -56,7 +56,7 @@ load _helpers --set 'server.dev.enabled=true' \ . | tee /dev/stderr | yq -r '.spec.template.spec.containers[0].image' | tee /dev/stderr) - [ "${actual}" = "foo:latest" ] + [ "${actual}" = "quay.io/foo:latest" ] } #-------------------------------------------------------------------- diff --git a/test/unit/server-discovery-role.bats b/test/unit/server-discovery-role.bats index f17dcf4..16799dc 100755 --- a/test/unit/server-discovery-role.bats +++ b/test/unit/server-discovery-role.bats @@ -57,4 +57,4 @@ load _helpers . | tee /dev/stderr | yq -r '.metadata.namespace' | tee /dev/stderr) [ "${actual}" = "bar" ] -} \ No newline at end of file +} diff --git a/test/unit/server-discovery-rolebinding.bats b/test/unit/server-discovery-rolebinding.bats index 83e8def..dd961f6 100755 --- a/test/unit/server-discovery-rolebinding.bats +++ b/test/unit/server-discovery-rolebinding.bats @@ -57,4 +57,4 @@ load _helpers . | tee /dev/stderr | yq -r '.metadata.namespace' | tee /dev/stderr) [ "${actual}" = "bar" ] -} \ No newline at end of file +} diff --git a/test/unit/server-ha-statefulset.bats b/test/unit/server-ha-statefulset.bats index 84bb21c..f0385bf 100755 --- a/test/unit/server-ha-statefulset.bats +++ b/test/unit/server-ha-statefulset.bats @@ -43,7 +43,7 @@ load _helpers --set 'server.ha.enabled=true' \ . | tee /dev/stderr | yq -r '.spec.template.spec.containers[0].image' | tee /dev/stderr) - [ "${actual}" = "foo:1.2.3" ] + [ "${actual}" = "quay.io/foo:1.2.3" ] } @test "server/ha-StatefulSet: image tag defaults to latest" { @@ -56,7 +56,7 @@ load _helpers --set 'server.ha.enabled=true' \ . | tee /dev/stderr | yq -r '.spec.template.spec.containers[0].image' | tee /dev/stderr) - [ "${actual}" = "foo:latest" ] + [ "${actual}" = "quay.io/foo:latest" ] } #-------------------------------------------------------------------- @@ -71,7 +71,7 @@ load _helpers yq -r '.spec.template.spec.containers[0].env' | tee /dev/stderr) local value=$(echo $object | - yq -r 'map(select(.name=="VAULT_ADDR")) | .[] .value' | tee /dev/stderr) + yq -r 'map(select(.name=="BAO_ADDR")) | .[] .value' | tee /dev/stderr) [ "${value}" = "http://127.0.0.1:8200" ] } @@ -84,7 +84,7 @@ load _helpers yq -r '.spec.template.spec.containers[0].env' | tee /dev/stderr) local value=$(echo $object | - yq -r 'map(select(.name=="VAULT_ADDR")) | .[] .value' | tee /dev/stderr) + yq -r 'map(select(.name=="BAO_ADDR")) | .[] .value' | tee /dev/stderr) [ "${value}" = "https://127.0.0.1:8200" ] } @@ -407,7 +407,7 @@ load _helpers } #-------------------------------------------------------------------- -# VAULT_API_ADDR renders +# BAO_API_ADDR renders @test "server/ha-StatefulSet: api addr renders to Pod IP by default" { cd `chart_dir` @@ -418,7 +418,7 @@ load _helpers yq -r '.spec.template.spec.containers[0].env' | tee /dev/stderr) local value=$(echo $object | - yq -r 'map(select(.name=="VAULT_API_ADDR")) | .[] .value' | tee /dev/stderr) + yq -r 'map(select(.name=="BAO_API_ADDR")) | .[] .value' | tee /dev/stderr) [ "${value}" = 'http://$(POD_IP):8200' ] } @@ -432,12 +432,12 @@ load _helpers yq -r '.spec.template.spec.containers[0].env' | tee /dev/stderr) local value=$(echo $object | - yq -r 'map(select(.name=="VAULT_API_ADDR")) | .[] .value' | tee /dev/stderr) + yq -r 'map(select(.name=="BAO_API_ADDR")) | .[] .value' | tee /dev/stderr) [ "${value}" = "https://example.com:8200" ] } #-------------------------------------------------------------------- -# VAULT_CLUSTER_ADDR renders +# BAO_CLUSTER_ADDR renders @test "server/ha-StatefulSet: clusterAddr not set" { cd `chart_dir` @@ -449,7 +449,7 @@ load _helpers yq -r '.spec.template.spec.containers[0].env' | tee /dev/stderr) local value=$(echo $object | - yq -r 'map(select(.name=="VAULT_CLUSTER_ADDR")) | .[] .value' | tee /dev/stderr) + yq -r 'map(select(.name=="BAO_CLUSTER_ADDR")) | .[] .value' | tee /dev/stderr) [ "${value}" = 'https://$(HOSTNAME).release-name-openbao-internal:8201' ] } @@ -464,7 +464,7 @@ load _helpers yq -r '.spec.template.spec.containers[0].env' | tee /dev/stderr) local value=$(echo $object | - yq -r 'map(select(.name=="VAULT_CLUSTER_ADDR")) | .[] .value' | tee /dev/stderr) + yq -r 'map(select(.name=="BAO_CLUSTER_ADDR")) | .[] .value' | tee /dev/stderr) [ "${value}" = 'https://$(HOSTNAME).release-name-openbao-internal:8201' ] } @@ -479,7 +479,7 @@ load _helpers yq -r '.spec.template.spec.containers[0].env' | tee /dev/stderr) local value=$(echo $object | - yq -r 'map(select(.name=="VAULT_CLUSTER_ADDR")) | .[] .value' | tee /dev/stderr) + yq -r 'map(select(.name=="BAO_CLUSTER_ADDR")) | .[] .value' | tee /dev/stderr) [ "${value}" = 'https://test.example.com:8201' ] } @@ -494,7 +494,7 @@ load _helpers yq -r '.spec.template.spec.containers[0].env' | tee /dev/stderr) local value=$(echo $object | - yq -r 'map(select(.name=="VAULT_CLUSTER_ADDR")) | .[] .value' | tee /dev/stderr) + yq -r 'map(select(.name=="BAO_CLUSTER_ADDR")) | .[] .value' | tee /dev/stderr) [ "${value}" = 'http://$(HOSTNAME).release-name-openbao-internal:8201' ] } @@ -515,7 +515,7 @@ local value=$(echo $rendered | } #-------------------------------------------------------------------- -# VAULT_RAFT_NODE_ID renders +# BAO_RAFT_NODE_ID renders @test "server/ha-StatefulSet: raft node ID renders" { cd `chart_dir` @@ -528,7 +528,7 @@ local value=$(echo $rendered | yq -r '.spec.template.spec.containers[0].env' | tee /dev/stderr) local value=$(echo $object | - yq -r 'map(select(.name=="VAULT_RAFT_NODE_ID")) | .[] .valueFrom.fieldRef.fieldPath' | tee /dev/stderr) + yq -r 'map(select(.name=="BAO_RAFT_NODE_ID")) | .[] .valueFrom.fieldRef.fieldPath' | tee /dev/stderr) [ "${value}" = "metadata.name" ] } diff --git a/test/unit/server-headless-service.bats b/test/unit/server-headless-service.bats index 43713de..7796dca 100644 --- a/test/unit/server-headless-service.bats +++ b/test/unit/server-headless-service.bats @@ -74,25 +74,3 @@ load _helpers yq '.spec.ipFamilies' -c | tee /dev/stderr) [ "${actual}" = '["IPv4","IPv6"]' ] } - -@test "server/headless-Service: Assert ipFamilyPolicy is not set if version below 1.23" { - cd `chart_dir` - local actual=$(helm template \ - --show-only templates/server-headless-service.yaml \ - --kube-version 1.27.0 \ - --set 'server.service.ipFamilyPolicy=PreferDualStack' \ - . | tee /dev/stderr | - yq -r '.spec.ipFamilyPolicy' | tee /dev/stderr) - [ "${actual}" = "null" ] -} - -@test "server/headless-Service: Assert ipFamilies is not set if version below 1.23" { - cd `chart_dir` - local actual=$(helm template \ - --show-only templates/server-headless-service.yaml \ - --kube-version 1.27.0 \ - --set 'server.service.ipFamilies={IPv4,IPv6}' \ - . | tee /dev/stderr | - yq -r '.spec.ipFamilies' | tee /dev/stderr) - [ "${actual}" = "null" ] -} \ No newline at end of file diff --git a/test/unit/server-psp-role.bats b/test/unit/server-psp-role.bats index 28239b0..3e63b3c 100644 --- a/test/unit/server-psp-role.bats +++ b/test/unit/server-psp-role.bats @@ -127,4 +127,4 @@ load _helpers . | tee /dev/stderr | yq -r '.metadata.namespace' | tee /dev/stderr) [ "${actual}" = "bar" ] -} \ No newline at end of file +} diff --git a/test/unit/server-psp-rolebinding.bats b/test/unit/server-psp-rolebinding.bats index 4a4bae3..d57687b 100644 --- a/test/unit/server-psp-rolebinding.bats +++ b/test/unit/server-psp-rolebinding.bats @@ -127,4 +127,4 @@ load _helpers . | tee /dev/stderr | yq -r '.metadata.namespace' | tee /dev/stderr) [ "${actual}" = "bar" ] -} \ No newline at end of file +} diff --git a/test/unit/server-service.bats b/test/unit/server-service.bats index bb3c47a..94ebb6a 100755 --- a/test/unit/server-service.bats +++ b/test/unit/server-service.bats @@ -487,25 +487,3 @@ load _helpers yq '.spec.ipFamilies' -c | tee /dev/stderr) [ "${actual}" = '["IPv4","IPv6"]' ] } - -@test "server/Service: Assert ipFamilyPolicy is not set if version below 1.23" { - cd `chart_dir` - local actual=$(helm template \ - --show-only templates/server-service.yaml \ - --kube-version 1.27.0 \ - --set 'server.service.ipFamilyPolicy=PreferDualStack' \ - . | tee /dev/stderr | - yq -r '.spec.ipFamilyPolicy' | tee /dev/stderr) - [ "${actual}" = "null" ] -} - -@test "server/Service: Assert ipFamilies is not set if version below 1.23" { - cd `chart_dir` - local actual=$(helm template \ - --show-only templates/server-service.yaml \ - --kube-version 1.27.0 \ - --set 'server.service.ipFamilies={IPv4,IPv6}' \ - . | tee /dev/stderr | - yq -r '.spec.ipFamilies' | tee /dev/stderr) - [ "${actual}" = "null" ] -} \ No newline at end of file diff --git a/test/unit/server-serviceaccount.bats b/test/unit/server-serviceaccount.bats index da0dd11..2a8c60c 100755 --- a/test/unit/server-serviceaccount.bats +++ b/test/unit/server-serviceaccount.bats @@ -145,4 +145,4 @@ load _helpers . | tee /dev/stderr | yq -r '.metadata.labels.foo' | tee /dev/stderr) [ "${actual}" = "bar" ] -} \ No newline at end of file +} diff --git a/test/unit/server-statefulset.bats b/test/unit/server-statefulset.bats index 755333a..aafd92d 100755 --- a/test/unit/server-statefulset.bats +++ b/test/unit/server-statefulset.bats @@ -105,7 +105,7 @@ load _helpers --set 'server.image.tag=1.2.3' \ . | tee /dev/stderr | yq -r '.spec.template.spec.containers[0].image' | tee /dev/stderr) - [ "${actual}" = "foo:1.2.3" ] + [ "${actual}" = "quay.io/foo:1.2.3" ] local actual=$(helm template \ --show-only templates/server-statefulset.yaml \ @@ -114,7 +114,7 @@ load _helpers --set 'server.standalone.enabled=true' \ . | tee /dev/stderr | yq -r '.spec.template.spec.containers[0].image' | tee /dev/stderr) - [ "${actual}" = "foo:1.2.3" ] + [ "${actual}" = "quay.io/foo:1.2.3" ] } @test "server/standalone-StatefulSet: image tag defaults to latest" { @@ -125,7 +125,7 @@ load _helpers --set 'server.image.tag=' \ . | tee /dev/stderr | yq -r '.spec.template.spec.containers[0].image' | tee /dev/stderr) - [ "${actual}" = "foo:latest" ] + [ "${actual}" = "quay.io/foo:latest" ] local actual=$(helm template \ --show-only templates/server-statefulset.yaml \ @@ -134,7 +134,7 @@ load _helpers --set 'server.standalone.enabled=true' \ . | tee /dev/stderr | yq -r '.spec.template.spec.containers[0].image' | tee /dev/stderr) - [ "${actual}" = "foo:latest" ] + [ "${actual}" = "quay.io/foo:latest" ] } @test "server/standalone-StatefulSet: default imagePullPolicy" { @@ -224,43 +224,11 @@ load _helpers #-------------------------------------------------------------------- # persistentVolumeClaimRetentionPolicy -@test "server/standalone-StatefulSet: persistentVolumeClaimRetentionPolicy not set by default when kubernetes < 1.23" { - cd `chart_dir` - local actual=$(helm template \ - -s templates/server-statefulset.yaml \ - --kube-version "1.27" \ - . | tee /dev/stderr | - yq -r '.spec.persistentVolumeClaimRetentionPolicy' | tee /dev/stderr) - [ "${actual}" = "null" ] -} - -@test "server/standalone-StatefulSet: unset persistentVolumeClaimRetentionPolicy.whenDeleted when kubernetes < 1.23" { - cd `chart_dir` - local actual=$(helm template \ - -s templates/server-statefulset.yaml \ - --kube-version "1.27" \ - --set 'server.persistentVolumeClaimRetentionPolicy.whenDeleted=Delete' \ - . | tee /dev/stderr | - yq -r '.spec.persistentVolumeClaimRetentionPolicy.whenDeleted' | tee /dev/stderr) - [ "${actual}" = "null" ] -} - -@test "server/standalone-StatefulSet: unset persistentVolumeClaimRetentionPolicy.whenScaled when kubernetes < 1.23" { - cd `chart_dir` - local actual=$(helm template \ - -s templates/server-statefulset.yaml \ - --kube-version "1.27" \ - --set 'server.persistentVolumeClaimRetentionPolicy.whenScaled=Delete' \ - . | tee /dev/stderr | - yq -r '.spec.persistentVolumeClaimRetentionPolicy.whenScaled' | tee /dev/stderr) - [ "${actual}" = "null" ] -} - @test "server/standalone-StatefulSet: persistentVolumeClaimRetentionPolicy not set by default when kubernetes >= 1.23" { cd `chart_dir` local actual=$(helm template \ -s templates/server-statefulset.yaml \ - --kube-version "1.23" \ + --kube-version "1.27" \ . | tee /dev/stderr | yq -r '.spec.persistentVolumeClaimRetentionPolicy' | tee /dev/stderr) [ "${actual}" = "null" ] @@ -270,7 +238,7 @@ load _helpers cd `chart_dir` local actual=$(helm template \ -s templates/server-statefulset.yaml \ - --kube-version "1.23" \ + --kube-version "1.27" \ --set 'server.persistentVolumeClaimRetentionPolicy.whenDeleted=Delete' \ . | tee /dev/stderr | yq -r '.spec.persistentVolumeClaimRetentionPolicy.whenDeleted' | tee /dev/stderr) @@ -281,7 +249,7 @@ load _helpers cd `chart_dir` local actual=$(helm template \ -s templates/server-statefulset.yaml \ - --kube-version "1.23" \ + --kube-version "1.27" \ --set 'server.persistentVolumeClaimRetentionPolicy.whenScaled=Delete' \ . | tee /dev/stderr | yq -r '.spec.persistentVolumeClaimRetentionPolicy.whenScaled' | tee /dev/stderr) @@ -571,7 +539,7 @@ load _helpers yq -r '.spec.template.spec.containers[0].env' | tee /dev/stderr) local value=$(echo $objects | - yq -r 'map(select(.name=="VAULT_LOG_LEVEL")) | .[] .name' | tee /dev/stderr) + yq -r 'map(select(.name=="BAO_LOG_LEVEL")) | .[] .name' | tee /dev/stderr) [ "${value}" = "" ] } @@ -584,7 +552,7 @@ load _helpers yq -r '.spec.template.spec.containers[0].env' | tee /dev/stderr) local value=$(echo $objects | - yq -r 'map(select(.name=="VAULT_LOG_LEVEL")) | .[] .value' | tee /dev/stderr) + yq -r 'map(select(.name=="BAO_LOG_LEVEL")) | .[] .value' | tee /dev/stderr) [ "${value}" = "debug" ] } @@ -599,7 +567,7 @@ load _helpers yq -r '.spec.template.spec.containers[0].env' | tee /dev/stderr) local value=$(echo $objects | - yq -r 'map(select(.name=="VAULT_LOG_FORMAT")) | .[] .name' | tee /dev/stderr) + yq -r 'map(select(.name=="BAO_LOG_FORMAT")) | .[] .name' | tee /dev/stderr) [ "${value}" = "" ] } @@ -612,7 +580,7 @@ load _helpers yq -r '.spec.template.spec.containers[0].env' | tee /dev/stderr) local value=$(echo $objects | - yq -r 'map(select(.name=="VAULT_LOG_FORMAT")) | .[] .value' | tee /dev/stderr) + yq -r 'map(select(.name=="BAO_LOG_FORMAT")) | .[] .value' | tee /dev/stderr) [ "${value}" = "json" ] } @@ -800,7 +768,7 @@ load _helpers yq -r '.spec.template.spec.containers[0].volumeMounts[] | select(.name == "audit")' | tee /dev/stderr) local actual=$(echo $object | yq -r '.mountPath' | tee /dev/stderr) - [ "${actual}" = "/vault/audit" ] + [ "${actual}" = "/openbao/audit" ] } @test "server/standalone-StatefulSet: can configure audit storage mount path" { @@ -825,7 +793,7 @@ load _helpers yq -r '.spec.template.spec.containers[0].volumeMounts[] | select(.name == "data")' | tee /dev/stderr) local actual=$(echo $object | yq -r '.mountPath' | tee /dev/stderr) - [ "${actual}" = "/vault/data" ] + [ "${actual}" = "/openbao/data" ] } @test "server/standalone-StatefulSet: can configure data storage mount path" { @@ -1686,7 +1654,7 @@ load _helpers local actual=$(helm template \ --show-only templates/server-statefulset.yaml \ . | tee /dev/stderr | - yq '.spec.template.metadata.annotations["vault.hashicorp.com/config-checksum"] == null' | tee /dev/stderr) + yq '.spec.template.metadata.annotations["openbao.hashicorp.com/config-checksum"] == null' | tee /dev/stderr) [ "${actual}" = "true" ] } @@ -1705,7 +1673,7 @@ load _helpers --show-only templates/server-statefulset.yaml \ --set 'server.includeConfigAnnotation=true' \ . | tee /dev/stderr | - yq '.spec.template.metadata.annotations["vault.hashicorp.com/config-checksum"] == null' | tee /dev/stderr) + yq '.spec.template.metadata.annotations["openbao.hashicorp.com/config-checksum"] == null' | tee /dev/stderr) [ "${actual}" = "false" ] } diff --git a/test/unit/server-test.bats b/test/unit/server-test.bats index 6619771..f36dc5f 100644 --- a/test/unit/server-test.bats +++ b/test/unit/server-test.bats @@ -52,7 +52,7 @@ load _helpers --show-only templates/tests/server-test.yaml \ . | tee /dev/stderr | yq -r '.metadata.name' | tee /dev/stderr) - [ "${actual}" = "vault-server-test" ] + [ "${actual}" = "openbao-server-test" ] } @test "server/standalone-server-test-Pod: release metadata.name foo" { @@ -61,7 +61,7 @@ load _helpers --show-only templates/tests/server-test.yaml \ . | tee /dev/stderr | yq -r '.metadata.name' | tee /dev/stderr) - [ "${actual}" = "foo-vault-server-test" ] + [ "${actual}" = "foo-openbao-server-test" ] } @test "server/standalone-server-test-Pod: default server.standalone.enabled" { @@ -134,7 +134,7 @@ load _helpers --set 'server.image.tag=1.2.3' \ . | tee /dev/stderr | yq -r '.spec.containers[0].image' | tee /dev/stderr) - [ "${actual}" = "foo:1.2.3" ] + [ "${actual}" = "quay.io/foo:1.2.3" ] local actual=$(helm template \ --show-only templates/tests/server-test.yaml \ @@ -143,7 +143,7 @@ load _helpers --set 'server.standalone.enabled=true' \ . | tee /dev/stderr | yq -r '.spec.containers[0].image' | tee /dev/stderr) - [ "${actual}" = "foo:1.2.3" ] + [ "${actual}" = "quay.io/foo:1.2.3" ] } @test "server/standalone-server-test-Pod: image tag defaults to latest" { @@ -154,7 +154,7 @@ load _helpers --set 'server.image.tag=' \ . | tee /dev/stderr | yq -r '.spec.containers[0].image' | tee /dev/stderr) - [ "${actual}" = "foo:latest" ] + [ "${actual}" = "quay.io/foo:latest" ] local actual=$(helm template \ --show-only templates/tests/server-test.yaml \ @@ -163,7 +163,7 @@ load _helpers --set 'server.standalone.enabled=true' \ . | tee /dev/stderr | yq -r '.spec.containers[0].image' | tee /dev/stderr) - [ "${actual}" = "foo:latest" ] + [ "${actual}" = "quay.io/foo:latest" ] } @test "server/standalone-server-test-Pod: default imagePullPolicy" { diff --git a/test/unit/ui-service.bats b/test/unit/ui-service.bats index 531bc56..42b467e 100755 --- a/test/unit/ui-service.bats +++ b/test/unit/ui-service.bats @@ -406,27 +406,3 @@ load _helpers yq -r '.spec.ipFamilyPolicy' | tee /dev/stderr) [ "${actual}" = "PreferDualStack" ] } - -@test "server/Service: Assert ipFamilyPolicy is not set if version below 1.23" { - cd `chart_dir` - local actual=$(helm template \ - --show-only templates/ui-service.yaml \ - --kube-version 1.27.0 \ - --set 'ui.enabled=true' \ - --set 'ui.serviceIPFamilyPolicy=PreferDualStack' \ - . | tee /dev/stderr | - yq -r '.spec.ipFamilyPolicy' | tee /dev/stderr) - [ "${actual}" = "null" ] -} - -@test "server/Service: Assert ipFamilies is not set if version below 1.23" { - cd `chart_dir` - local actual=$(helm template \ - --show-only templates/ui-service.yaml \ - --kube-version 1.27.0 \ - --set 'ui.enabled=true' \ - --set 'ui.serviceIPFamilies={IPv4,IPv6}' \ - . | tee /dev/stderr | - yq -r '.spec.ipFamilies' | tee /dev/stderr) - [ "${actual}" = "null" ] -} \ No newline at end of file