Skip to content

Conversation

@Victoremepunto
Copy link
Contributor

No description provided.

@Victoremepunto Victoremepunto force-pushed the remove-postgresql-10-support branch from c1df416 to 0dfb6db Compare February 5, 2024 19:28
@Victoremepunto Victoremepunto changed the title refactor(db): Remove Postgresql 10 support RHCLOUD-30831 - refactor(db): Remove Postgresql 10 support Feb 5, 2024
Copy link
Collaborator

@psav psav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You also need to find and change this kubebuilder annotation line

	// +kubebuilder:validation:Enum:=10;12;13;14;15

@psav psav added technical debt Technical Debt related pr-architectural-change Required architect sign-off labels Feb 7, 2024
@psav psav changed the title RHCLOUD-30831 - refactor(db): Remove Postgresql 10 support [RHCLOUD-30831] - refactor(db): Remove Postgresql 10 support Feb 7, 2024
@Victoremepunto
Copy link
Contributor Author

This is waiting on RedHatInsights/config-manager#139

@Victoremepunto Victoremepunto force-pushed the remove-postgresql-10-support branch from 5056f53 to f09a07d Compare March 19, 2024 20:18
@Victoremepunto
Copy link
Contributor Author

/retest

@Victoremepunto Victoremepunto requested a review from psav March 26, 2024 09:29
@Victoremepunto Victoremepunto force-pushed the remove-postgresql-10-support branch 2 times, most recently from ee44c8a to d310cc2 Compare March 26, 2024 10:30
@bsquizz bsquizz self-assigned this Mar 27, 2024
Copy link
Contributor

@bsquizz bsquizz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still one test referencing version 10:

10:48:23 [2024-03-26T10:48:23.157Z]     logger.go:42: 10:48:23 | test-multi-app-interface-db/1-pods | ClowdApp:test-multi-app-interface-db/app-e created
10:48:23 [2024-03-26T10:48:23.157Z]     case.go:364: failed in step 1-pods
10:48:23 [2024-03-26T10:48:23.157Z]     case.go:366: ClowdApp.cloud.redhat.com "app-default-ca" is invalid: spec.database.version: Unsupported value: 10: supported values: "12", "13", "14", "15", "16"
10:48:23 [2024-03-26T10:48:23.157Z]     case.go:366: ClowdApp.cloud.redhat.com "app-b" is invalid: spec.database.version: Unsupported value: 10: supported values: "12", "13", "14", "15", "16"
10:48:23 [2024-03-26T10:48:23.157Z]     case.go:366: ClowdApp.cloud.redhat.com "app-c" is invalid: spec.database.version: Unsupported value: 10: supported values: "12", "13", "14", "15", "16"
10:48:23 [2024-03-26T10:48:23.157Z]     case.go:366: ClowdApp.cloud.redhat.com "app-d" is invalid: spec.database.version: Unsupported value: 10: supported values: "12", "13", "14", "15", "16"

@Victoremepunto Victoremepunto force-pushed the remove-postgresql-10-support branch from f3c05ba to 1e513ce Compare March 27, 2024 14:59
@Victoremepunto
Copy link
Contributor Author

Still one test referencing version 10:

10:48:23 [2024-03-26T10:48:23.157Z]     logger.go:42: 10:48:23 | test-multi-app-interface-db/1-pods | ClowdApp:test-multi-app-interface-db/app-e created
10:48:23 [2024-03-26T10:48:23.157Z]     case.go:364: failed in step 1-pods
10:48:23 [2024-03-26T10:48:23.157Z]     case.go:366: ClowdApp.cloud.redhat.com "app-default-ca" is invalid: spec.database.version: Unsupported value: 10: supported values: "12", "13", "14", "15", "16"
10:48:23 [2024-03-26T10:48:23.157Z]     case.go:366: ClowdApp.cloud.redhat.com "app-b" is invalid: spec.database.version: Unsupported value: 10: supported values: "12", "13", "14", "15", "16"
10:48:23 [2024-03-26T10:48:23.157Z]     case.go:366: ClowdApp.cloud.redhat.com "app-c" is invalid: spec.database.version: Unsupported value: 10: supported values: "12", "13", "14", "15", "16"
10:48:23 [2024-03-26T10:48:23.157Z]     case.go:366: ClowdApp.cloud.redhat.com "app-d" is invalid: spec.database.version: Unsupported value: 10: supported values: "12", "13", "14", "15", "16"

Thanks @bsquizz !! fixed!

@Victoremepunto
Copy link
Contributor Author

I think the Konflux pipeline may be fixed by #965

@Victoremepunto Victoremepunto force-pushed the remove-postgresql-10-support branch from 1e513ce to b4ce304 Compare April 30, 2024 07:51
@psav psav force-pushed the remove-postgresql-10-support branch from b4ce304 to 66b2900 Compare June 17, 2024 12:22
@psav psav merged commit 32a5621 into RedHatInsights:master Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-architectural-change Required architect sign-off technical debt Technical Debt related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants