Skip to content
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

Update postgres bp to work with postgresql 13 #1116

Merged
merged 1 commit into from
Oct 12, 2021
Merged

Conversation

PrasadG193
Copy link
Contributor

@PrasadG193 PrasadG193 commented Oct 12, 2021

Signed-off-by: Prasad Ghangal prasad.ghangal@gmail.com

Change Overview

SQL dump created by pg_dump in postgres-bp is not compatible with PostgreSQL 13+. Fix the restore action in postgres BP to get restore working on PostgreSQL 13+
Ref: https://stackoverflow.com/a/66565656/4572187

Pull request type

Please check the type of change your PR introduces:

  • 🚧 Work in Progress
  • 🌈 Refactoring (no functional changes, no api changes)
  • 🐹 Trivial/Minor
  • 🐛 Bugfix
  • 🌻 Feature
  • 🗺️ Documentation
  • 🤖 Test

Test Plan

  • 💪 Manual
  • ⚡ Unit test
  • 💚 E2E

Signed-off-by: Prasad Ghangal <prasad.ghangal@gmail.com>
Copy link
Contributor

@ewhamilton ewhamilton left a comment

Choose a reason for hiding this comment

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

Looks good.

@mergify mergify bot merged commit b5d99b6 into master Oct 12, 2021
@mergify mergify bot deleted the bp-postgres-13 branch October 12, 2021 19:24
viveksinghggits added a commit that referenced this pull request Oct 19, 2021
We recently figure out [an issue](#1116) where the restore command in our
blueprint was not working for postgres applications.
We fixed that for other application in the mentioned PR this
PR fixes the issue for the postgres application that gets
deployed as deployment config in OCP clusters.
mergify bot pushed a commit that referenced this pull request Oct 19, 2021
We recently figure out [an issue](#1116) where the restore command in our
blueprint was not working for postgres applications.
We fixed that for other application in the mentioned PR this
PR fixes the issue for the postgres application that gets
deployed as deployment config in OCP clusters.

Co-authored-by: Tom Manville <tom@kasten.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants