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.

Sign up for GitHub

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

Update bitnami postgres installation link #5768

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update bitnami postgres installation link #5768

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
e1e8136
Update bitnami postgres installation link
munkiat Nov 29, 2024
8733860
Merge branch 'master' into patch-5
sharadregoti Dec 6, 2024
9ff5191
Merge branch 'master' into patch-5
sharadregoti Dec 17, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion tyk-docs/content/product-stack/tyk-charts/tyk-stack-chart.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -290,7 +290,7 @@ global:

**PostgresSQL Installation**
```bash
helm install tyk-postgres bitnami/postgresql --set "auth.database=tyk_analytics" -n tyk --version 12.12.10
helm install tyk-postgres oci://registry-1.docker.io/bitnamicharts/postgresql --set "auth.database=tyk_analytics" -n tyk
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
helm install tyk-postgres oci://registry-1.docker.io/bitnamicharts/postgresql --set "auth.database=tyk_analytics" -n tyk
helm install tyk-postgres oci://registry-1.docker.io/bitnamicharts/postgresql --set "auth.database=tyk_analytics" -n tyk --version 16.2.3

I think we should restrict to the current version of chart, Have you tested with this version?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@sharadregoti yes makes sense. I've used the latest; not sure if our automated tests uses v16.2.3 exactly. Would something like v15 work? https://tyk.io/docs/planning-for-production/database-settings/postgresql/

Copy link
Contributor

Choose a reason for hiding this comment

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

16.2.3 is the helm chart version. In this chart version the default value for postgres is 17.2.0-debian-12-r2. We need to select a chart version which uses postgres version 16.x or 15.x as per this page.

Copy link
Contributor

Choose a reason for hiding this comment

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

Or alternatively we can fixate on chat version 16.2.3 and set the image version --set image.tag="16.6.0"
I used this postgres version

```

Follow the notes from the installation output to get connection details.
Expand Down
Loading