-
Notifications
You must be signed in to change notification settings - Fork 2
Conversation
9189512
to
40f9286
Compare
8c36630
to
1a30512
Compare
1a30512
to
a0b1585
Compare
@cicdteam @sharnoff I'd appreciate your review on this one. UPD switched to |
a0b1585
to
1b0bbd5
Compare
61c0f74
to
b521bd2
Compare
0df91f1
to
4f0fd00
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Broadly looks good. I'm not familiar with kuttl, but if it works it works. One curiosity: why is CGO_ENABLED=0
required? (and why even on go fmt
?) — It'd be good to have a brief comment explaining as well.
4f0fd00
to
cf09e5d
Compare
Actually, it's not required for |
cf09e5d
to
c5ea326
Compare
This PR adds a test for VM migration and checks that the connection doesn't break. It uses
kuttl
as a test tool.pg_isready
pgbench --initialize
pgbench
workload (the duration of the workload should be longer that test duration)pgbench
is still runningHow to run it locally:
kuttl
:brew install kuttl
orgo install github.com/kudobuilder/kuttl/cmd/[email protected]
or download the binary from their release page https://github.com/kudobuilder/kuttl/releases/tag/v0.15.0make local-cluster
make deploy
make e2e
or, if you want to keep pods running after the test, you can usekubectl kuttl test --config tests/e2e/kuttl-test.yaml --skip-delete
(note, that before running the test next time, you'll need to delete namespace to avoid Postgres ports collision:kubectl get namespaces --no-headers | awk '/kuttl-test-/ {print $1}' | xargs --no-run-if-empty -n 1 kubectl delete namespace
)pgbench
output for the migration timeframe looks like this: