-
Notifications
You must be signed in to change notification settings - Fork 200
test(cluster): Add support for extension upgrade test under cluster/test/database-management #773
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
base: main
Are you sure you want to change the base?
Conversation
| - name: pg_search | ||
| ensure: absent | ||
| version: "0.15.21" | ||
| version: "0.19.11" |
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.
0.15.21 is pretty ancient for us at ParadeDB, and a version above 0.16.0 is guaranteed to always be available
| metadata: | ||
| name: extension-upgrade-cluster-cluster | ||
| spec: | ||
| name: test-db-icu |
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.
@itay-grudev let me know if this is not right. I based it on 01-database-parameters.
Signed-off-by: Philippe Noël <philippemnoel@gmail.com>
Signed-off-by: Philippe Noël <philippemnoel@gmail.com>
Signed-off-by: Philippe Noël <philippemnoel@gmail.com>
84c8a8e to
465c600
Compare
Over at @paradedb we have been using the CNPG
clusterChart as a baseline for our chart for quite some time. We had some tests around upgrading database extensions intest/database-management.In an effort to 1) contribute our changes to the community and 2) reduce our diff with upstream, I'd like to propose adding these tests to the upstream charts. They confirm that extensions upgrade properly via the database management functionality, and ensure they don't regress.