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

Upgrade Postgres Helm Chart to v11 #1894

Conversation

KevinMellott91
Copy link
Contributor

Problem

The renovate dependency upgrade of the Postgres Helm chart (#1878) resulted in a failed PR check. This is because the Postgres chart contained a major version upgrade with breaking changes in how authentication is configured.

Solution

Configuration of Postgres chart authentication was modified in accordance with the Bitnami upgrade guidance. The underlying version of Postgres has been pinned to 12.1 to match the Marquez project; these configuration changes are specific to the Helm chart itself.

Checklist

  • You've signed-off your work
  • Your changes are accompanied by tests (if relevant)
  • Your change contains a small diff and is self-contained
  • You've updated any relevant documentation (if relevant)
  • You've updated the CHANGELOG.md with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary)
  • You've versioned your .sql database schema migration according to Flyway's naming convention (if relevant)

Kevin Mellott and others added 2 commits March 3, 2022 16:50
@codecov
Copy link

codecov bot commented Mar 3, 2022

Codecov Report

Merging #1894 (94f3a4b) into main (6d67ad7) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##               main    #1894   +/-   ##
=========================================
  Coverage     77.90%   77.90%           
  Complexity      937      937           
=========================================
  Files           193      193           
  Lines          5218     5218           
  Branches        418      418           
=========================================
  Hits           4065     4065           
  Misses          706      706           
  Partials        447      447           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6d67ad7...94f3a4b. Read the comment docs.

Copy link
Member

@wslulciuc wslulciuc left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@wslulciuc wslulciuc merged commit 6c312d9 into MarquezProject:main Mar 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants