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.

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

PG-1012 Documented set up of the_heap access method #305

Merged
merged 5 commits into from
Oct 16, 2024

Conversation

nastena1606
Copy link
Collaborator

Added How to decrypt a table

Copy link

github-actions bot commented Oct 8, 2024

Performance test results:
Normal queries: 9323
TDE queries: 8653
Percentage: 92%

Copy link
Collaborator

@janwieremjewicz janwieremjewicz left a comment

Choose a reason for hiding this comment

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

LGTM

sudo systemctl restart postgresql-17
```

After you enabled `pg_tde` and started the Percona Server for PostgreSQL, a principal key and a keyring for WAL are created. We highly recommend you to create your own keyring and rotate the principal key. This is because the default principal key is created from the local keyfile and is stored unencrypted.
Copy link
Member

Choose a reason for hiding this comment

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

In fact, right now we set up global keys on the first server start in any case (even when pg_tde.wal_encrypt is off)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Reworked

SELECT pg_tde_add_key_provider_file('provider-name','/path/to/the/keyring/data.file');
```

2. Rotate the principal key. Don't forget to specify the `PG_TDE_GLOBAL` constant to rotate only the principal key for WAL
Copy link
Member

Choose a reason for hiding this comment

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

ALTER SYSTEM set pg_tde.wal_encrypt = on;
```

2. Restart the server to apply the changes.
Copy link
Member

Choose a reason for hiding this comment

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

Most probably the restart is not needed

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Let's have @shahidullah79's confirmation on that

@nastena1606 nastena1606 merged commit 0995806 into main Oct 16, 2024
12 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.

4 participants