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

Modified docker documentation #233

Merged
merged 3 commits into from
Jul 15, 2024
Merged

Modified docker documentation #233

merged 3 commits into from
Jul 15, 2024

Conversation

dutow
Copy link
Collaborator

@dutow dutow commented Jul 9, 2024

No description provided.

@dutow dutow requested a review from nastena1606 July 9, 2024 09:47
@@ -70,7 +70,9 @@ Install `pg_tde` using one of available installation methods:
docker run --name pg-tde -e POSTGRES_PASSWORD=mysecretpassword -d perconalab/pg_tde
```

It builds and adds `pg_tde` extension to PostgreSQL 16. Relevant `postgresql.conf` and `tde_conf.json` are created in `/etc/postgresql/` inside the container. This directory is exposed as a volume.
It builds and adds `pg_tde` extension to PostgreSQL 16. The `postgresql.conf` contains the required modifications, and the extension is added to `template1` - with this, all new databases automatically have the `pg_tde` extension loaded.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Don't we mention where the postgresql.conf is stored inside the container anymore?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

That's a feature of the "upstream" docker image - do we want to mention it?

Copy link
Collaborator

Choose a reason for hiding this comment

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

We just mentioned it before. If it's not important / well known for Docker users/ brings no value to the user, we can omit it

documentation/docs/install.md Outdated Show resolved Hide resolved
documentation/docs/install.md Outdated Show resolved Hide resolved
dutow and others added 2 commits July 15, 2024 08:01
Co-authored-by: Anastasia Alexandrova <[email protected]>
Co-authored-by: Anastasia Alexandrova <[email protected]>
@dutow dutow requested a review from nastena1606 July 15, 2024 07:02
Copy link
Collaborator

@nastena1606 nastena1606 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@dutow dutow merged commit b5ef829 into Percona-Lab:main Jul 15, 2024
10 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.

None yet

2 participants