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.

Sign up for GitHub

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

Jump to bottom

feat: upgrade postgres dependency from v8 -> v11 #40

Merged
merged 3 commits into from
Jul 15, 2022
Merged

feat: upgrade postgres dependency from v8 -> v11 #40

merged 3 commits into from
Jul 15, 2022

Conversation

Imti
Copy link
Contributor

@Imti Imti commented Jul 15, 2022 •
edited
Loading

What's changed?

  • upgrade postgres dependency from v8 -> v11
  • update values.yaml to match the new v11 postgres values structure
  • use remote bitnami chart instead of the committed local chart (with regenerated chart lock file)
  • re-add resource limits that were previously commented out

@Imti Imti self-assigned this Jul 15, 2022
@Imti Imti changed the title fix: upgrade postgres dependency from v8 -> v11, update to new values, use remote chart feat: upgrade postgres dependency from v8 -> v11 Jul 15, 2022
@Imti Imti marked this pull request as ready for review July 15, 2022 20:45
yourbuddyconner
yourbuddyconner approved these changes Jul 15, 2022
View reviewed changes
Copy link
Contributor

@yourbuddyconner yourbuddyconner left a comment

Choose a reason for hiding this comment

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

more red than green we love it

@Imti Imti force-pushed the imti/upgrade-postgres-dependency branch from d065a13 to 18fd801 Compare July 15, 2022 21:34
Base automatically changed from imti/add-prometheus-postgres-exporter to main July 15, 2022 21:51
@Imti Imti merged commit 4136d6e into main Jul 15, 2022
@Imti Imti deleted the imti/upgrade-postgres-dependency branch July 15, 2022 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yourbuddyconner yourbuddyconner yourbuddyconner approved these changes

@arnaud036 arnaud036 Awaiting requested review from arnaud036

Assignees

@Imti Imti

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Imti @yourbuddyconner