Skip to content

Adopting according to new readability and code style #553

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 15 commits into from
Sep 3, 2024
Merged

Adopting according to new readability and code style #553

Changes from 13 commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
197aa9b
Adopting according to new readability and code style
Maleware Aug 29, 2024
4070830
Removing whitespace
Maleware Aug 29, 2024
fcd0901
Making shellcheck happy
Maleware Aug 29, 2024
10d00cc
chore(test): revert previous styling changes
NickLarsenNZ Aug 29, 2024
6e99f7f
chore(test): wrap variables in double quotes
NickLarsenNZ Aug 29, 2024
e43955d
chore(test): use bash strict mode
NickLarsenNZ Aug 29, 2024
0826712
chore(test): add comments to tests
NickLarsenNZ Aug 29, 2024
4943445
chore(test): make assertions more precise with exact matching with grep
NickLarsenNZ Aug 29, 2024
730d506
chore(test): split long commands into multi-line commands, split labe…
NickLarsenNZ Aug 29, 2024
fc6fd79
chore(test): suggestion: use long-options where possible
NickLarsenNZ Aug 29, 2024
24ed1ed
chore(test): fix yamllint
NickLarsenNZ Aug 29, 2024
5f0b874
Revert "chore(test): suggestion: use long-options where possible"
NickLarsenNZ Sep 2, 2024
75d6eb1
chore(test): fix yamllint
NickLarsenNZ Sep 2, 2024
62dd3cb
chore(test): remove -o pipefail bash option
NickLarsenNZ Sep 3, 2024
980a1c6
chore(test): undo ORed labels
NickLarsenNZ Sep 3, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
117 changes: 101 additions & 16 deletions tests/templates/kuttl/overrides/31-assert.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,23 +3,108 @@ apiVersion: kuttl.dev/v1beta1
kind: TestAssert
timeout: 30
commands:
# master, default RG
- script: |
POD=$(kubectl -n $NAMESPACE get pod -l app.kubernetes.io/component=master,app.kubernetes.io/role-group=default -o name | head -n 1 | sed -e 's#pod/##')
kubectl -n $NAMESPACE get pod $POD -o yaml | yq '.spec.containers[0].env[] | select (.name == "TEST_VAR_FROM_MASTER").value' | grep 'MASTER'
kubectl -n $NAMESPACE get pod $POD -o yaml | yq '.spec.containers[0].env[] | select (.name == "TEST_VAR_FROM_MRG").value' | grep 'MASTER'
kubectl -n $NAMESPACE get pod $POD -o yaml | yq '.spec.containers[0].env[] | select (.name == "TEST_VAR").value' | grep 'MASTER_RG'
set -euo pipefail

# Get the name of the first pod by labels
POD=$(
kubectl get pod -n "$NAMESPACE" \
-l app.kubernetes.io/component=master \
-l app.kubernetes.io/role-group=default \
-o name \
| head -n 1 \
| sed -e 's#pod/##'
)

# Assert that environment variables have the correct values
kubectl get pod "$POD" -n "$NAMESPACE" -o yaml \
| yq -r '.spec.containers[0].env[] | select (.name == "TEST_VAR_FROM_MASTER").value' \
| grep '^MASTER$'

kubectl get pod "$POD" -n "$NAMESPACE" -o yaml \
| yq -r '.spec.containers[0].env[] | select (.name == "TEST_VAR_FROM_MRG").value' \
| grep '^MASTER$'

kubectl get pod "$POD" -n "$NAMESPACE" -o yaml \
| yq -r '.spec.containers[0].env[] | select (.name == "TEST_VAR").value' \
| grep '^MASTER_RG$'

# regionserver, resources-from-role RG
- script: |
POD=$(kubectl -n $NAMESPACE get pod -l app.kubernetes.io/component=regionserver,app.kubernetes.io/role-group=resources-from-role -o name | head -n 1 | sed -e 's#pod/##')
kubectl -n $NAMESPACE get pod $POD -o yaml | yq '.spec.containers[0].env[] | select (.name == "TEST_VAR_FROM_RS").value' | grep 'REGIONSERVER'
kubectl -n $NAMESPACE get pod $POD -o yaml | yq '.spec.containers[0].env[] | select (.name == "TEST_VAR_FROM_RFR").value' | grep 'REGIONSERVER'
kubectl -n $NAMESPACE get pod $POD -o yaml | yq '.spec.containers[0].env[] | select (.name == "TEST_VAR").value' | grep 'REGIONSERVER_RFR'
set -euo pipefail

# Get the name of the first pod by labels
POD=$(
kubectl get pod -n "$NAMESPACE" \
-l app.kubernetes.io/component=regionserver \
-l app.kubernetes.io/role-group=resources-from-role \
-o name \
| head -n 1 \
| sed -e 's#pod/##'
)

# Assert that environment variables have the correct values
kubectl get pod "$POD" -n "$NAMESPACE" -o yaml \
| yq -r '.spec.containers[0].env[] | select (.name == "TEST_VAR_FROM_RS").value' \
| grep '^REGIONSERVER$'

kubectl get pod "$POD" -n "$NAMESPACE" -o yaml \
| yq -r '.spec.containers[0].env[] | select (.name == "TEST_VAR_FROM_RFR").value' \
| grep '^REGIONSERVER$'

kubectl get pod "$POD" -n "$NAMESPACE" -o yaml \
| yq -r '.spec.containers[0].env[] | select (.name == "TEST_VAR").value' \
| grep '^REGIONSERVER_RFR$'

# regionserver, resources-from-role-group RG
- script: |
POD=$(kubectl -n $NAMESPACE get pod -l app.kubernetes.io/component=regionserver,app.kubernetes.io/role-group=resources-from-role-group -o name | head -n 1 | sed -e 's#pod/##')
kubectl -n $NAMESPACE get pod $POD -o yaml | yq '.spec.containers[0].env[] | select (.name == "TEST_VAR_FROM_RS").value' | grep 'REGIONSERVER'
kubectl -n $NAMESPACE get pod $POD -o yaml | yq '.spec.containers[0].env[] | select (.name == "TEST_VAR_FROM_RFRG").value' | grep 'REGIONSERVER'
kubectl -n $NAMESPACE get pod $POD -o yaml | yq '.spec.containers[0].env[] | select (.name == "TEST_VAR").value' | grep 'REGIONSERVER_RFRG'
set -euo pipefail

# Get the name of the first pod by labels
POD=$(
kubectl get pod \
-n "$NAMESPACE" \
-l app.kubernetes.io/component=regionserver \
-l app.kubernetes.io/role-group=resources-from-role-group \
-o name | head -n 1 | sed -e 's#pod/##'
)

# Assert that environment variables have the correct values
kubectl get pod "$POD" -n "$NAMESPACE" -o yaml \
| yq -r '.spec.containers[0].env[] | select (.name == "TEST_VAR_FROM_RS").value' \
| grep '^REGIONSERVER$'

kubectl get pod "$POD" -n "$NAMESPACE" -o yaml \
| yq -r '.spec.containers[0].env[] | select (.name == "TEST_VAR_FROM_RFRG").value' \
| grep '^REGIONSERVER$'

kubectl get pod "$POD" -n "$NAMESPACE" -o yaml \
| yq -r '.spec.containers[0].env[] | select (.name == "TEST_VAR").value' \
| grep '^REGIONSERVER_RFRG$'

# restserver, default RG
- script: |
POD=$(kubectl -n $NAMESPACE get pod -l app.kubernetes.io/component=restserver,app.kubernetes.io/role-group=default -o name | head -n 1 | sed -e 's#pod/##')
kubectl -n $NAMESPACE get pod $POD -o yaml | yq '.spec.containers[0].env[] | select (.name == "TEST_VAR_FROM_REST").value' | grep 'RESTSERVER'
kubectl -n $NAMESPACE get pod $POD -o yaml | yq '.spec.containers[0].env[] | select (.name == "TEST_VAR_FROM_REST_RG").value' | grep 'RESTSERVER'
kubectl -n $NAMESPACE get pod $POD -o yaml | yq '.spec.containers[0].env[] | select (.name == "TEST_VAR").value' | grep 'RESTSERVER_RG'
set -euo pipefail

# Get the name of the first pod by labels
POD=$(
kubectl get pod \
-n "$NAMESPACE" \
-l app.kubernetes.io/component=restserver \
-l app.kubernetes.io/role-group=default \
-o name | head -n 1 | sed -e 's#pod/##'
)

# Assert that environment variables have the correct values
kubectl get pod "$POD" -n "$NAMESPACE" -o yaml \
| yq -r '.spec.containers[0].env[] | select (.name == "TEST_VAR_FROM_REST").value' \
| grep '^RESTSERVER$'

kubectl get pod "$POD" -n "$NAMESPACE" -o yaml \
| yq -r '.spec.containers[0].env[] | select (.name == "TEST_VAR_FROM_REST_RG").value' \
| grep '^RESTSERVER$'

kubectl get pod "$POD" -n "$NAMESPACE" -o yaml \
| yq -r '.spec.containers[0].env[] | select (.name == "TEST_VAR").value' \
| grep '^RESTSERVER_RG$'
Loading