-
Notifications
You must be signed in to change notification settings - Fork 7
[DOCSW-552] Enhance airgapped edition documentation for docker compose deployments #182
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
Conversation
* Fixes from support threads * minor fixes
Merge changes to master branch
Update master branch with changes
docs: restructure API docs and update Work Item terminology (#163)
feat: add options field docs in create property API
Merge to master branch
chore: update MCP server docs with latest http transport changes
…y Docker image preparation, and add OpenSearch as an alternative service. Streamline installation steps and improve environment variable configuration instructions.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughWalkthroughReworks airgapped deployment docs: Kubernetes guide rebranded to Plane Commercial and adds OpenSearch; Docker Compose guide shifts from offline bundle to private-registry image workflow; adds a new airgapped license activation page and updates site navigation. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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 |
…g Docker images, highlight important infrastructure images that require separate handling, and clarify configuration steps for services.
|
Linked to Plane Work Item(s) This comment was auto-generated by Plane |
…into chore-update-airgapped-docker
…s for clarity, removing redundant contact information for license acquisition.
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.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
self-hosting/methods/airgapped-edition-kubernetes.mdx (1)
56-62: Inconsistent Valkey image reference.Line 57 references
valkey:7.2.5-alpinewhile Line 135 (unchanged, but relevant) usesvalkey/valkey:7.2.11-alpine. The Docker Compose guide (airgapped-edition.mdx) consistently usesvalkey/valkey:7.2.11-alpine. This discrepancy could confuse users preparing their airgapped environment.Suggested fix
- `valkey:7.2.5-alpine` + - `valkey/valkey:7.2.11-alpine`
🧹 Nitpick comments (1)
self-hosting/methods/airgapped-edition.mdx (1)
52-62: Consider adding version placeholder clarification.The
<plane-version>placeholder in the download URLs should be explicitly called out, with guidance on where users can find available versions. The Kubernetes guide references Artifact Hub for version information.Suggested addition
2. **Download Docker Compose configuration** Download the Docker Compose configuration files from the official release. + Replace `<plane-version>` with your target version (e.g., `1.6.4`). Check for the latest version at [Prime releases](https://prime.plane.so/releases). ```bash # Download docker-compose.yml
…ore-update-airgapped-docker
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.
Actionable comments posted: 2
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
self-hosting/methods/airgapped-edition-kubernetes.mdx (1)
54-65: Valkey version inconsistency.Line 57 lists
valkey:7.2.5-alpinebut line 135 showsvalkey/valkey:7.2.11-alpine. Additionally, the Docker guide (airgapped-edition.mdx) consistently uses7.2.11-alpine. Consider aligning these versions.📝 Suggested fix
**Important** This process will NOT download or clone these infrastructure images: - - `valkey:7.2.5-alpine` + - `valkey/valkey:7.2.11-alpine` - `postgres:15.7-alpine`
🤖 Fix all issues with AI agents
In `@self-hosting/manage/manage-licenses/activate-airgapped.mdx`:
- Line 18: The sentence in activate-airgapped.mdx contains an orphaned reference
"check the logs using the commands above" that doesn't exist on this page;
update the copy by either removing that clause or replacing it with a link to
the appropriate troubleshooting/log viewing docs (e.g., a "See troubleshooting"
link), and ensure the phrase is adjusted so the page reads correctly (refer to
the exact sentence containing "check the logs using the commands above" for the
edit).
- Around line 8-16: The ordered list in activate-airgapped.mdx has a missing
step number (it jumps from "4." to "6."); update the numbering so steps read 1–8
sequentially by changing the "6. Select **Billing and plans**" entry to "5.
Select **Billing and plans**" (and ensure subsequent steps remain correctly
numbered), locating the items by their text such as "Click **Download
license**", "Navigate to the [Workspace Settings]", "Select **Billing and
plans**", and "Click the **Activate this workspace**" to verify order.
🧹 Nitpick comments (2)
self-hosting/methods/airgapped-edition.mdx (2)
39-50: Consider consistency in image version pinning.The infrastructure images list mixes pinned versions (e.g.,
valkey:7.2.11-alpine,opensearchproject/opensearch:3.3.2) with floating tags (minio/minio:latest,minio/mc:latest). For airgapped environments where reproducibility is critical, consider pinning all image versions.
104-118: OpenSearch missing from infrastructure services example.The infrastructure images note (line 47) lists
opensearchproject/opensearch:3.3.2as requiring separate handling, but the infrastructure services YAML example doesn't include an OpenSearch service configuration. Consider adding it for completeness if users can run OpenSearch locally.📝 Suggested addition
minio: image: minio/minio:latest + + opensearch: + image: opensearchproject/opensearch:3.3.2
Description
This pull request updates the documentation for deploying Plane in airgapped environments, with a focus on clarifying prerequisites, improving instructions for Docker-based deployments, and ensuring OpenSearch is included among recommended infrastructure alternatives. The changes streamline the setup process, emphasize the use of managed services for reliability, and enhance clarity for users preparing and running airgapped installations.
Type of Change
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.