Skip to content

[bitnami/postgres-repmgr] Remove old pid file if it exists #80514

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ferm10n
Copy link

@ferm10n ferm10n commented Apr 23, 2025

See #999

Additional considerations:

  • would we want to disable the pid file altogether, with --no-pid-file? It doesn't make a lot of sense to have in a containerized environment
  • would we want to try to parse the repmgr config file to know where the pid is configured to be located? probably not but worth mentioning.

@github-actions github-actions bot requested a review from carrodher April 23, 2025 14:37
@carrodher carrodher added verify Execute verification workflow for these changes in-progress labels Apr 23, 2025
@github-actions github-actions bot removed the triage Triage is needed label Apr 23, 2025
@github-actions github-actions bot removed the request for review from carrodher April 23, 2025 15:33
@github-actions github-actions bot requested a review from migruiz4 April 23, 2025 15:33
Copy link

github-actions bot commented May 9, 2025

This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution.

@github-actions github-actions bot added the stale 15 days without activity label May 9, 2025
@ariel-kruszynski-zen-com

Hey @migruiz4 , just checking in on this PR - any idea when you might have a chance to take a look?

@github-actions github-actions bot removed the stale 15 days without activity label May 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in-progress postgresql-repmgr verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants