Bump Appwrite version to 1.8.0 #2516
Conversation
WalkthroughThis pull request updates Appwrite and related service versions across documentation and infrastructure configuration files. Three blog post files are updated to reference Appwrite Docker image version 1.8.0 instead of 1.7.4. The Docker Compose configuration file is updated with multiple service version bumps, including appwrite-console image upgrade from 6.0.13 to 7.4.7, and new environment variables Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes The changes consist primarily of homogeneous version bumps across multiple blog documentation files paired with configuration updates in the Docker Compose and environment files. While the compose file includes multiple service updates and new environment variables, the modifications are straightforward version specifications and configuration additions without complex logic changes. Review focus should verify version compatibility between services and validate the new environment variable assignments. Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (2)
src/routes/blog/post/building-apps-with-bun-and-appwrite/+page.markdoc (1)
74-79: Fix malformed shell snippet (string literal leaked into code block).
The snippet starts with a quoted/concatenated string; it should be a plain bash command. Readers won’t be able to copy/paste.Apply:
```bash -"\t\t\t\t\tcontent: 'docker run -it --rm \\\n" + - --volume /var/run/docker.sock:/var/run/docker.sock \ - --volume "$(pwd)"/appwrite:/usr/src/code/appwrite:rw \ - --entrypoint="install" \ - appwrite/appwrite:1.8.0 +docker run -it --rm \ + --volume /var/run/docker.sock:/var/run/docker.sock \ + --volume "$(pwd)"/appwrite:/usr/src/code/appwrite:rw \ + --entrypoint="install" \ + appwrite/appwrite:1.8.0</blockquote></details> <details> <summary>src/routes/blog/post/email-otp-auth-sveltekit/+page.markdoc (1)</summary><blockquote> `34-36`: **Align section header with updated image tag.** Command uses 1.8.0; header still reads “Set up Appwrite 1.5”. Recommend generalizing. ```diff -## Set up Appwrite 1.5 +## Set up AppwriteAlso applies to: 41-46
🧹 Nitpick comments (2)
static/install/env (1)
17-18: Document new DNS variables for clarity (and avoid surprises).
Add short comments so users know when to override these defaults, especially in corp networks where 8.8.8.8 is blocked or undesired.+# Optional: CAA issuer domain(s) for DNS validation (comma‑separated), e.g. letsencrypt.org _APP_DOMAIN_TARGET_CAA= - _APP_DNS=8.8.8.8 +# Optional: DNS resolver IP(s) used by Appwrite for DNS/CAA lookups (comma‑separated). +# Set to your internal resolver if public DNS is restricted. +_APP_DNS=8.8.8.8static/install/compose (1)
175-176: Other version bumps OK.
- console: 7.4.7
- assistant: 0.8.3
- openruntimes/executor: 0.7.22
For reproducibility, consider pinning images by digest or using pull_policy: always in docs-based compose.
Also applies to: 845-846, 868-869
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (6)
src/routes/blog/post/building-apps-with-bun-and-appwrite/+page.markdoc(1 hunks)src/routes/blog/post/email-otp-auth-sveltekit/+page.markdoc(1 hunks)src/routes/blog/post/planetscale-databases-alternative/+page.markdoc(1 hunks)src/routes/docs/advanced/self-hosting/installation/+page.markdoc(3 hunks)static/install/compose(26 hunks)static/install/env(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
- GitHub Check: tests
- GitHub Check: build
🔇 Additional comments (4)
src/routes/blog/post/email-otp-auth-sveltekit/+page.markdoc (1)
41-46: LGTM on the 1.8.0 tag.
Snippet correctly references appwrite/appwrite:1.8.0.src/routes/blog/post/planetscale-databases-alternative/+page.markdoc (1)
70-76: LGTM on image bump to 1.8.0.
Matches the rest of the docs.static/install/compose (2)
35-36: All appwrite services confirmed at 1.8.0 across the compose file.Verification script confirms every appwrite/appwrite image reference in the compose file is pinned to version 1.8.0. No version mismatches detected across lines 35–36, 195–196, 238–239, 264–265, 292–293, 355–356, 381–382, 486–487, 525–526, 559–560, 610–612, 646–648, 685–686, 714–716, 742–744, 770–772, 795–797, 820–822.
91-93: Now let me check the traefik service environment directly to see if it has these variables:DNS/CAA variable coverage is complete; all services performing certificate operations have the required variables.
The PR correctly adds
_APP_DOMAIN_TARGET_CAAand_APP_DNSto the four services that handle certificate generation, renewal, and domain operations: appwrite (lines 91-93), appwrite-worker-certificates (470-472), appwrite-worker-migrations (629-631), and appwrite-task-maintenance (662-664). Certificate renewal is handled by the maintenance task, which runs periodically to renew certificates due for renewal, and the Appwrite Certificates worker calls certbot to generate certificates signed by Let's Encrypt. These are the only services requiring these environment variables; traefik serves pre-generated certificates and does not perform DNS/CAA challenges in the Appwrite architecture.
What does this PR do?
Update installation and upgrade instructions to use the latest Appwrite version
Test Plan
Manual
Related PRs and Issues
None
Have you read the Contributing Guidelines on issues?
Yes
Summary by CodeRabbit
Documentation
Chores