Skip to content

Conversation

pcnc
Copy link
Member

@pcnc pcnc commented Sep 11, 2024

  • Saves us from headaches due to external outages, or mismatched installer issues
  • Fixes vacuumdb not being run as superuser post-bootstrap user switch

@pcnc pcnc requested a review from a team as a code owner September 11, 2024 13:40
Copy link
Collaborator

@samrose samrose left a comment

Choose a reason for hiding this comment

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

Seems like a reasonable approach to me.

The one caveat is that there can be security issues with specific versions of nix, which tend to be fixed and then distributed with the installer by DS.

So, we should definitely have a process to keep out self-hosted installer up to date on a weekly basis or so, outside of the operation it is being used for.

@pcnc pcnc changed the title feat(pg_upgrade): use self-hosted nix installer if available feat(pg_upgrade): use self-hosted nix installer if available; move files out of /tmp Sep 11, 2024
@pcnc
Copy link
Member Author

pcnc commented Sep 11, 2024

Moving into draft as it still needs some testing

@pcnc pcnc marked this pull request as draft September 11, 2024 16:53
@pcnc pcnc force-pushed the pcnc/pinned-nix-installer branch from 53970fb to ed63ed7 Compare September 13, 2024 15:44
@pcnc pcnc marked this pull request as ready for review September 13, 2024 15:46
@pcnc pcnc merged commit dd15fb6 into develop Sep 13, 2024
8 checks passed
@pcnc pcnc deleted the pcnc/pinned-nix-installer branch September 13, 2024 16:41
damonrand pushed a commit to cepro/postgres that referenced this pull request Jun 15, 2025
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.

3 participants