Skip to content

Commit a5a87ff

Browse files
authored
Apply suggestions from code review
Co-authored-by: Cody Maffucci <46459665+Maffooch@users.noreply.github.com>
1 parent 61fca47 commit a5a87ff

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

docs/content/releases/pro/ddorch-database.md

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ This guide walks through adding `dojodb-ddorch` to an existing self-hosted Postg
1818
- `listen_addresses` in `postgresql.conf` is already configured for remote access.
1919
- You have upgraded to the release that ships the `ddorch` and `ddorch-workers` services.
2020

21-
> **A note on the database name:** `dojodb-ddorch` contains a hyphen, so it must be double-quoted in every SQL statement (`"dojodb-ddorch"`). If you prefer to avoid the quoting, use `dojodb_ddorch` (underscore) and drop the quotes throughout the rest of this guide.
21+
> **A note on the database name:** `dojodb-ddorch` contains a hyphen, so it must be double-quoted in every SQL statement (`"dojodb-ddorch"`).
2222
2323
## Part 1: Provision the Database
2424

@@ -179,8 +179,7 @@ docker compose logs ddorch-workers --tail=50
179179
Finally, confirm the main Django container can reach the orchestrator. From the `dojo` container:
180180

181181
```bash
182-
docker compose exec dojo curl -kf https://ddorch:9871/health
183-
^^^ PAUL CONFIRM THIS IS REAL
182+
docker compose exec -u root dojo bash -c "apt update && apt install curl -y && echo -e '\n\nReady if 200 is returned\n' && curl -kf -w "%{http_code}" https://ddorch:9871/readyz"
184183
```
185184

186185
A `200 OK` response confirms the orchestrator is healthy and the main app can talk to it. Your installation is now using the new `dojodb-ddorch` database.

0 commit comments

Comments
 (0)