Skip to content

docs(tutorials): fix errors in the tutorial content vaultwarden-serverless #5374

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
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
11 changes: 7 additions & 4 deletions tutorials/vaultwarden-serverless/index.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,9 @@ Vaultwarden relies on a database to store all password data securely. The databa
|--------------|-----------------------------------------------------------------------------------------------------|
| Image origin | External registry |
| Image URL | `vaultwarden/server:latest` |
| CPU | 1 000 mVCPU |
| RAM | 2 048 MB |
| Container port | 80 |
| CPU | 100 mVCPU |
| RAM | 128 MB |
Comment on lines +72 to +73
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove CPU/RAM requirements as it depends on actual usage, lowest values may harm performance in several scenarios

| Min scale | `0` ((set a value of 1 or more to avoid [cold starts](/serverless-containers/concepts/#cold-start)) |
| Max scale | `1` or more, according to your workload |

Expand All @@ -79,14 +80,16 @@ Vaultwarden relies on a database to store all password data securely. The databa
|-------------------------|--------|
| `DATA_FOLDER` | `/tmp` |
| `ORG_ATTACHMENT_LIMIT` | `0` |
| `SIGNUPS_ALLOWED` | `true` |
| `USER_ATTACHMENT_LIMIT` | `0` |
| `SENDS_ALLOWED` | `false`|
Copy link
Collaborator

Choose a reason for hiding this comment

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

Personal configuration, should be removed from this table, and can be added as a note instead

| `SIGNUPS_ALLOWED` | `true` |

4. Set the following ¨¨**secrets**:
| Key | Value |
|----------------|-----------------------------------------------------------------------------------------------------------------------------------|
| `DATABASE_URL` | connection string retrieved during the [previous step](#creating-a-serverless-sql-database) |
| `DATABASE_URL` | connection string retrieved during the [previous step](#creating-a-serverless-sql-database) (IPv4 only) |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove this change as the tutorial relies on a Serverless Database and there is no notion of IP involved

| `ADMIN_TOKEN` | authentication token retrieved by following [This procedure](https://github.com/dani-garcia/vaultwarden/wiki/Enabling-admin-page) |

5. Click **Deploy container**. The container **Overview** tab displays.

## Finalizing the container configuration
Expand Down