-
Notifications
You must be signed in to change notification settings - Fork 0
Use fakechris Docker namespace and publish multi-arch images #22
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -142,8 +142,8 @@ pnpm compose:down | |||||||||||
| If you want to run the published Docker Hub images instead of building from source, use: | ||||||||||||
|
|
||||||||||||
| ```bash | ||||||||||||
| INVOLUTE_IMAGE_NAMESPACE=turnkeyai INVOLUTE_IMAGE_TAG=latest pnpm compose:pull | ||||||||||||
| INVOLUTE_IMAGE_NAMESPACE=turnkeyai INVOLUTE_IMAGE_TAG=latest pnpm compose:pull:up | ||||||||||||
| INVOLUTE_IMAGE_NAMESPACE=fakechris INVOLUTE_IMAGE_TAG=latest pnpm compose:pull | ||||||||||||
| INVOLUTE_IMAGE_NAMESPACE=fakechris INVOLUTE_IMAGE_TAG=latest pnpm compose:pull:up | ||||||||||||
| ``` | ||||||||||||
|
|
||||||||||||
| ## VPS deployment (fresh install) | ||||||||||||
|
|
@@ -442,22 +442,22 @@ This repo ships one multi-target `Dockerfile` with `server`, `web-dev`, `web`, a | |||||||||||
| Published images: | ||||||||||||
|
|
||||||||||||
| ```bash | ||||||||||||
| docker pull turnkeyai/involute-server:latest | ||||||||||||
| docker pull turnkeyai/involute-web:latest | ||||||||||||
| docker pull turnkeyai/involute-cli:latest | ||||||||||||
| docker pull fakechris/involute-server:latest | ||||||||||||
| docker pull fakechris/involute-web:latest | ||||||||||||
| docker pull fakechris/involute-cli:latest | ||||||||||||
| ``` | ||||||||||||
|
|
||||||||||||
| Run the compose stack from published images: | ||||||||||||
|
|
||||||||||||
| ```bash | ||||||||||||
| INVOLUTE_IMAGE_NAMESPACE=turnkeyai INVOLUTE_IMAGE_TAG=latest \ | ||||||||||||
| INVOLUTE_IMAGE_NAMESPACE=fakechris INVOLUTE_IMAGE_TAG=latest \ | ||||||||||||
| docker compose -f docker-compose.images.yml up -d db server web | ||||||||||||
|
Comment on lines
+453
to
454
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. These environment variables are redundant as they match the new defaults in the compose configuration. The command can be simplified by removing them.
Suggested change
|
||||||||||||
| ``` | ||||||||||||
|
|
||||||||||||
| Production compose can use the same published images: | ||||||||||||
|
|
||||||||||||
| ```bash | ||||||||||||
| INVOLUTE_IMAGE_NAMESPACE=turnkeyai INVOLUTE_IMAGE_TAG=latest \ | ||||||||||||
| INVOLUTE_IMAGE_NAMESPACE=fakechris INVOLUTE_IMAGE_TAG=latest \ | ||||||||||||
| docker compose --env-file .env.production \ | ||||||||||||
| -f docker-compose.prod.images.yml up -d | ||||||||||||
|
Comment on lines
+460
to
462
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. These environment variables are redundant as they match the new defaults in the compose configuration. Removing them makes the production example cleaner and easier to follow.
Suggested change
|
||||||||||||
| ``` | ||||||||||||
|
|
||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The environment variables
INVOLUTE_IMAGE_NAMESPACE=fakechrisandINVOLUTE_IMAGE_TAG=latestare now redundant in these examples because they match the default values defined in the updateddocker-compose.images.ymlanddocker-compose.prod.images.ymlfiles. Removing them simplifies the commands for users.