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

[bitnami/postgresql] fixing tls for cronjobs #24468

Merged
merged 2 commits into from
Mar 21, 2024
Merged

[bitnami/postgresql] fixing tls for cronjobs #24468

merged 2 commits into from
Mar 21, 2024
+5 −4

Conversation

mrlhansen
Copy link
Contributor

Description of the change

This is a recreation of #21078. It fixes the mounting of TLS secrets for the postgresql cronjob.

Benefits

TLS will be working for postgresql cronjobs

Possible drawbacks

None

Applicable issues

Additional information

None

Checklist

  • Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
  • Variables are documented in the values.yaml and added to the README.md using readme-generator-for-helm
  • Title of the pull request follows this pattern [bitnami/<name_of_the_chart>] Descriptive title
  • All commits signed off and in agreement of Developer Certificate of Origin (DCO)

Sorry, something went wrong.

@mrlhansen
postgresql: fixing tls for cronjobs
Loading
Loading status checks…
82545c4 
Signed-off-by: Martin Hansen <hansen@imada.sdu.dk>
@github-actions github-actions bot added postgresql triage Triage is needed labels Mar 15, 2024
@github-actions github-actions bot requested a review from javsalgar March 15, 2024 08:27
@carrodher carrodher added verify Execute verification workflow for these changes in-progress labels Mar 15, 2024
@github-actions github-actions bot removed the triage Triage is needed label Mar 15, 2024
@github-actions github-actions bot removed the request for review from javsalgar March 15, 2024 08:38
@github-actions github-actions bot requested a review from aoterolorenzo March 15, 2024 08:38
@aoterolorenzo
Merge branch 'main' into main

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
GPG key ID: B5690EEEBB952194
Verified
Learn about vigilant mode
Loading
Loading status checks…
69f389e 
Signed-off-by: Alberto Otero <lorenzoa@vmware.com>
@aoterolorenzo aoterolorenzo enabled auto-merge (squash) March 21, 2024 11:40
@aoterolorenzo aoterolorenzo merged commit 5e7f4e1 into bitnami:main Mar 21, 2024
7 checks passed
djjudas21 pushed a commit to djjudas21/bitnami-charts that referenced this pull request Apr 17, 2024
@mrlhansen @aoterolorenzo @djjudas21
[bitnami/postgresql] fixing tls for cronjobs (bitnami#24468)

Verified

This commit was signed with the committer’s verified signature.
djjudas21 Jonathan
GPG key ID: C5D15B16B566F5EE
Verified
Learn about vigilant mode
0b97700 
Signed-off-by: Martin Hansen <hansen@imada.sdu.dk>
Signed-off-by: Alberto Otero <lorenzoa@vmware.com>
Co-authored-by: Alberto Otero <lorenzoa@vmware.com>
Signed-off-by: Jonathan Gazeley <me@jonathangazeley.com>
@pthpth pthpth mentioned this pull request Dec 9, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aoterolorenzo aoterolorenzo

Assignees

@aoterolorenzo aoterolorenzo

Labels
postgresql solved verify Execute verification workflow for these changes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

postgresql: cronjob backup not working with tls
4 participants
@mrlhansen @aoterolorenzo @javsalgar @carrodher