-
Notifications
You must be signed in to change notification settings - Fork 82
K8SPG-1051 add pg 19 #1699
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
K8SPG-1051 add pg 19 #1699
Changes from all commits
ca1b520
5c88988
349c4b4
ee53942
1ecf6b2
e6624fd
47cf684
e80c71e
3e24bdb
d19810d
b8428c5
174acfc
128a58e
38aa057
81415e0
2ecd69d
c24c9f9
2bd7f8c
6176a58
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -19,6 +19,7 @@ major-upgrade-14-to-15 | |
| major-upgrade-15-to-16 | ||
| major-upgrade-16-to-17 | ||
| major-upgrade-17-to-18 | ||
| major-upgrade-18-to-19 | ||
| major-upgrade-14-to-15-postgis | ||
| major-upgrade-15-to-16-postgis | ||
| major-upgrade-16-to-17-postgis | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. do we need a postgis test as well? for all the other versions we have it.
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @nmarukovich @gkech we have PostGIS in the community image by default. Maybe we need to have a major-upgrade-18-to-19-postgis test only for now?
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yeap I agree! Let's have it
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we don't have postgis for community images. |
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,24 @@ | ||
| apiVersion: kuttl.dev/v1beta1 | ||
| kind: TestAssert | ||
| timeout: 120 | ||
| --- | ||
| apiVersion: apiextensions.k8s.io/v1 | ||
| kind: CustomResourceDefinition | ||
| metadata: | ||
| name: perconapgclusters.pgv2.percona.com | ||
| spec: | ||
| group: pgv2.percona.com | ||
| names: | ||
| kind: PerconaPGCluster | ||
| listKind: PerconaPGClusterList | ||
| plural: perconapgclusters | ||
| singular: perconapgcluster | ||
| scope: Namespaced | ||
| --- | ||
| apiVersion: kuttl.dev/v1beta1 | ||
| kind: TestAssert | ||
| metadata: | ||
| name: check-operator-deploy-status | ||
| timeout: 120 | ||
| commands: | ||
| - script: kubectl assert exist-enhanced deployment percona-postgresql-operator -n ${OPERATOR_NS:-$NAMESPACE} --field-selector status.readyReplicas=1 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| apiVersion: kuttl.dev/v1beta1 | ||
| kind: TestStep | ||
| commands: | ||
| - script: |- | ||
| set -o errexit | ||
| set -o xtrace | ||
|
|
||
| source ../../functions | ||
| init_temp_dir # do this only in the first TestStep | ||
|
|
||
| deploy_operator | ||
| deploy_client | ||
| deploy_s3_secrets |
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.
should we run this test also on every pr?
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.
added