Skip to content

Conversation

@fulghum
Copy link
Contributor

@fulghum fulghum commented Dec 19, 2024

Previously, superusers were persisted to disk, but never loaded back again when the database was restarted. This essentially made all superusers ephemeral, since they only lasted for the duration of a SQL server process.

This change loads persisted superusers from disk, and also adds a new function to create ephemeral superusers that do not get persisted to disk.

This also includes a fix for the event scheduler to use a privileged account so that it can load events from all databases.

@fulghum fulghum force-pushed the fulghum/persist_root_superuser branch from 5f1e3d7 to c353c46 Compare December 23, 2024 22:27
@fulghum fulghum force-pushed the fulghum/persist_root_superuser branch from c353c46 to e4a10fd Compare January 3, 2025 23:55
@fulghum fulghum marked this pull request as ready for review January 4, 2025 00:11
@fulghum fulghum force-pushed the fulghum/persist_root_superuser branch from 68e36a6 to 2efaf30 Compare January 8, 2025 20:00
@fulghum fulghum requested a review from zachmu January 9, 2025 18:21
Copy link
Member

@zachmu zachmu left a comment

Choose a reason for hiding this comment

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

LGTM

…utility function to create locked superuser accounts
@fulghum fulghum force-pushed the fulghum/persist_root_superuser branch from 2efaf30 to 57e5666 Compare January 15, 2025 01:48
@fulghum fulghum merged commit 41a0994 into main Jan 15, 2025
7 of 8 checks passed
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