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 docs for Package Manager Postgres.UsageDataPassword bug #561

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

glin
Copy link
Contributor

@glin glin commented Sep 26, 2024

Postgres.UsageDataPassword is supposed to default to Postgres.Password, but a bug in recent PPM versions prevents this from working. So as a temporary workaround, the Postgres.UsageDataPassword setting must be explicitly set to Postgres.Password when using Postgres with usage data enabled.

This will be fixed in the next Package Manager release. The config can be removed after the next release, but it also won't cause any issues if you keep it around for the next release.

@@ -1,5 +1,9 @@
# Changelog

## 0.5.35

- Update PostgreSQL configuration documentation to temporarily work around bug with `Postgres.UsageDataPassword` in Package Manager 2024.08.2.
Copy link
Member

Choose a reason for hiding this comment

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

Where is this update?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh what, wow, the automated helm-docs rebuild overwrote my readme change from 0995998

I might have to recreate this PR 🤔. I messed up originally by not bumping the chart version when making the readme change, and that might have messed things up?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I totally missed that there was a readme template file. Should look right now.

@glin glin force-pushed the glin-ppm-db branch 2 times, most recently from 4b2c2bd to d170d9e Compare September 27, 2024 19:43
@glin glin closed this Sep 27, 2024
@glin glin deleted the glin-ppm-db branch September 27, 2024 19:43
@glin glin restored the glin-ppm-db branch September 27, 2024 19:43
@glin glin reopened this Sep 27, 2024
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.

4 participants