Skip to content

K8SPG-647: Fix major-upgrade test #988

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.

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

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from 1 commit
Commits
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
Prev Previous commit
Next Next commit
fix
  • Loading branch information
egegunes committed Apr 9, 2025
commit c15dfb12bb1fff502af2ed1868480f0365fcad63
1 change: 1 addition & 0 deletions e2e-tests/tests/major-upgrade/01-create-cluster.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ commands:
.spec.postgresVersion = 13 |
.spec.image = "perconalab/percona-postgresql-operator:main-ppg13-postgres" |
.spec.instances[0].dataVolumeClaimSpec.resources.requests.storage = "3Gi" |
.spec.patroni.createReplicaMethods = ["basebackup","pgbackrest"] |
.spec.proxy.pgBouncer.image = "perconalab/percona-postgresql-operator:main-ppg13-pgbouncer" |
.spec.backups.pgbackrest.image = "perconalab/percona-postgresql-operator:main-ppg13-pgbackrest"' \
| kubectl -n "${NAMESPACE}" apply -f -
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,6 @@ commands:
done

run_psql_local \
'\c myapp \\\ INSERT INTO myApp (id) VALUES (100502)' \
'\c myapp \\\ INSERT INTO myApp (id) VALUES (100501)' \
"postgres:$(get_psql_user_pass major-upgrade-pguser-postgres)@$(get_psql_user_host major-upgrade-pguser-postgres)"

28 changes: 7 additions & 21 deletions e2e-tests/tests/major-upgrade/28-run-restore.yaml
Original file line number Diff line number Diff line change
@@ -1,21 +1,7 @@
apiVersion: kuttl.dev/v1beta1
kind: TestStep
commands:
- timeout: 600
script: |-
set -o errexit
set -o xtrace

backup_name=$(kubectl -n ${NAMESPACE} get pg-backup backup-after-13-to-14 -o jsonpath={.status.backupName})

cat <<-EOF | kubectl -n ${NAMESPACE} apply -f -
apiVersion: pgv2.percona.com/v2
kind: PerconaPGRestore
metadata:
name: restore-after-13-to-14
spec:
pgCluster: major-upgrade
repoName: repo1
options:
- --set=${backup_name}
EOF
apiVersion: pgv2.percona.com/v2
kind: PerconaPGRestore
metadata:
name: restore-after-13-to-14
spec:
pgCluster: major-upgrade
repoName: repo1
1 change: 0 additions & 1 deletion e2e-tests/tests/major-upgrade/29-assert.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,3 @@ data:
data: |2-
100500
100501
100502
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,6 @@ commands:
done

run_psql_local \
'\c myapp \\\ INSERT INTO myApp (id) VALUES (100503)' \
'\c myapp \\\ INSERT INTO myApp (id) VALUES (100502)' \
"postgres:$(get_psql_user_pass major-upgrade-pguser-postgres)@$(get_psql_user_host major-upgrade-pguser-postgres)"

28 changes: 7 additions & 21 deletions e2e-tests/tests/major-upgrade/38-run-restore.yaml
Original file line number Diff line number Diff line change
@@ -1,21 +1,7 @@
apiVersion: kuttl.dev/v1beta1
kind: TestStep
commands:
- timeout: 600
script: |-
set -o errexit
set -o xtrace

backup_name=$(kubectl -n ${NAMESPACE} get pg-backup backup-after-14-to-15 -o jsonpath={.status.backupName})

cat <<-EOF | kubectl -n ${NAMESPACE} apply -f -
apiVersion: pgv2.percona.com/v2
kind: PerconaPGRestore
metadata:
name: restore-after-14-to-15
spec:
pgCluster: major-upgrade
repoName: repo1
options:
- --set=${backup_name}
EOF
apiVersion: pgv2.percona.com/v2
kind: PerconaPGRestore
metadata:
name: restore-after-14-to-15
spec:
pgCluster: major-upgrade
repoName: repo1
1 change: 0 additions & 1 deletion e2e-tests/tests/major-upgrade/39-assert.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,4 +11,3 @@ data:
100500
100501
100502
100503
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,6 @@ commands:
done

run_psql_local \
'\c myapp \\\ INSERT INTO myApp (id) VALUES (100504)' \
'\c myapp \\\ INSERT INTO myApp (id) VALUES (100503)' \
"postgres:$(get_psql_user_pass major-upgrade-pguser-postgres)@$(get_psql_user_host major-upgrade-pguser-postgres)"

28 changes: 7 additions & 21 deletions e2e-tests/tests/major-upgrade/48-run-restore.yaml
Original file line number Diff line number Diff line change
@@ -1,21 +1,7 @@
apiVersion: kuttl.dev/v1beta1
kind: TestStep
commands:
- timeout: 600
script: |-
set -o errexit
set -o xtrace

backup_name=$(kubectl -n ${NAMESPACE} get pg-backup backup-after-15-to-16 -o jsonpath={.status.backupName})

cat <<-EOF | kubectl -n ${NAMESPACE} apply -f -
apiVersion: pgv2.percona.com/v2
kind: PerconaPGRestore
metadata:
name: restore-after-15-to-16
spec:
pgCluster: major-upgrade
repoName: repo1
options:
- --set=${backup_name}
EOF
apiVersion: pgv2.percona.com/v2
kind: PerconaPGRestore
metadata:
name: restore-after-15-to-16
spec:
pgCluster: major-upgrade
repoName: repo1
1 change: 0 additions & 1 deletion e2e-tests/tests/major-upgrade/49-assert.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,4 +12,3 @@ data:
100501
100502
100503
100504
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,19 @@ commands:
set -o errexit
set -o xtrace

sleep 30
source ../../functions

sleep 90

for pod in $(kubectl -n ${NAMESPACE} get pods -l postgres-operator.crunchydata.com/data=postgres --no-headers | awk '{print $1}'); do
echo "Checking the status of ${pod}:"
kubectl -n ${NAMESPACE} exec -it ${pod} -- psql -c "SELECT version()"
kubectl -n ${NAMESPACE} exec -it ${pod} -- psql -c "SELECT timeline_id FROM pg_control_checkpoint()"
kubectl -n ${NAMESPACE} exec -it ${pod} -- pgbackrest --stanza=db --log-level-console=detail check
kubectl -n ${NAMESPACE} exec -it ${pod} -- ls -l /pgdata/pg16_wal
done

run_psql_local \
'\c myapp \\\ INSERT INTO myApp (id) VALUES (100504)' \
"postgres:$(get_psql_user_pass major-upgrade-pguser-postgres)@$(get_psql_user_host major-upgrade-pguser-postgres)"

1 change: 0 additions & 1 deletion e2e-tests/tests/major-upgrade/58-run-restore.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,3 @@ metadata:
spec:
pgCluster: major-upgrade
repoName: repo1

7 changes: 6 additions & 1 deletion e2e-tests/tests/major-upgrade/59-assert.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,9 @@ apiVersion: v1
metadata:
name: 49-read-from-primary
data:
data: ' 100500'
data: |2-
100500
100501
100502
100503
100504