Fix postgres StatefulSet crash under readOnlyRootFilesystem - #7
Merged
Conversation
The postgres container sets readOnlyRootFilesystem: true but never mounted a writable volume at /var/run/postgresql, so the official postgres image's entrypoint fails to create its Unix socket lock file on every start: FATAL: could not create lock file "/var/run/postgresql/.s.PGSQL.5432.lock": Read-only file system Add an emptyDir volume mounted at /var/run/postgresql, matching the existing dshm pattern.
lookup-based persistence only works against a live cluster (helm upgrade/install). GitOps controllers that render via `helm template` (ArgoCD, Flux) always get an empty lookup result, so these two secrets were regenerating on every render and forcing a server pod restart via the checksum/secret annotation on every sync — same class of issue as the existing postgres.auth.existingSecret / patchmon.jwt.secret escape hatches, just missing for these two fields.
PatchMon v2.1.0 shrinks agent check-in payloads, adds per-host status pills, and speeds up the packages/dashboard pages. The upstream compose reference also moved guacd off :latest onto a pinned 1.6.0 tag. https://github.com/PatchMon/PatchMon/releases/tag/v2.1.0
Rolls up v2.1.1-v2.1.3: fixes the v2.1.0 migration-42 crash loop for host_down alert configs, ADFS/Entra ID SSO fixes, Mattermost/Rocket.Chat webhook fixes, compliance content for non-Docker agents, and stale-host reporting after suspend/resume. No DB or compose changes required. https://github.com/PatchMon/PatchMon/releases/tag/v2.1.3
Contributor
|
Nice one @BryanR77, thanks a lot for this! You're the first person to send a PR to this chart, so this kind of made my day 🎉 It all looks good. There might be a follow up on the changes to OIDC but i'll look into that later. Merging it, thanks again! 🙌 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The postgres container sets readOnlyRootFilesystem: true but never mounted a writable volume at /var/run/postgresql, so the official postgres image's entrypoint fails to create its Unix socket lock file on every start:
FATAL: could not create lock file
"/var/run/postgresql/.s.PGSQL.5432.lock": Read-only file system
Add an emptyDir volume mounted at /var/run/postgresql, matching the existing dshm pattern.