Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Added Airgapped to Editions and updated K8s configurations settings #158

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 31, 2025
Merged

Added Airgapped to Editions and updated K8s configurations settings #158

merged 1 commit into from
Oct 31, 2025

Conversation

@danciaclara
Copy link
Collaborator

@danciaclara danciaclara commented Oct 31, 2025 •
edited by coderabbitai bot
Loading

Description

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • Feature (non-breaking change which adds functionality)
  • Improvement (change that would cause existing functionality to not work as expected)
  • Code refactoring
  • Performance improvements
  • Documentation update

Screenshots and Media (if applicable)

Test Scenarios

References

Note

Adds Airgapped edition docs and updates Kubernetes deployment guide to v1.15.0 with airgapped options, new optional services, and expanded configuration controls.

  • Documentation
    • Editions (self-hosting/editions-and-versions.mdx):
      • Add the Airgapped Edition and reference it across edition/version sections.
    • Kubernetes (self-hosting/methods/kubernetes.mdx):
      • Bump PLANE_VERSION/planeVersion to v1.15.0 (Commercial) and standardize Community tables.
      • Add airgapped settings for S3 CA override and isolated deployments.
      • Introduce optional services: email_service, outbox_poller, automation_consumer, and iframely with full config blocks.
      • Expand configs across services: resource requests, pullPolicy, scheduling controls (nodeSelector, tolerations, affinity), and metadata (labels, annotations).
      • Add extraEnv (global envs), new ingress/SSL options (incl. rabbitmqHost), and refine External Secrets tables.
      • Update Community Edition accordion: defaults, resource/scheduling options, and env.default_cluster_domain.

Written by Cursor Bugbot for commit b8ff3a4. This will update automatically on new commits. Configure here.

Summary by CodeRabbit

  • New Features

    • Airgapped Edition now available as a new self-hosted offering with isolated environment, feature parity, and user-controlled version management.

  • Documentation

    • Updated self-hosted editions documentation to reflect the new Airgapped Edition, including detailed feature descriptions and revised version update guidance across all editions.

@vercel
Copy link

vercel bot commented Oct 31, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
developer-docs Error Error Oct 31, 2025 0:28am

@coderabbitai
Copy link

coderabbitai bot commented Oct 31, 2025 •
edited
Loading

Walkthrough

Documentation update expanding the self-hosted product lineup to include a new Airgapped Edition, increasing the total edition count from three to four. Updates include feature descriptions, revised edition references, and adjusted version update guidance for the expanded lineup.

Changes

Cohort / File(s) Summary
Airgapped Edition Documentation
self-hosting/editions-and-versions.mdx		 Added new Airgapped Edition to self-hosted offerings with feature descriptions covering isolation, parity, version updates, self-contained architecture, and compliance. Updated edition count from 3 to 4, expanded self-hosted editions list from 2 to 3, revised "Why we separate editions" section, and adjusted version update guidance to reflect Commercial, Airgapped, and Community Editions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

  • Verify accuracy of Airgapped Edition feature descriptions and compliance claims
  • Confirm edition count references are consistently updated throughout the document
  • Check that version update guidance correctly reflects the new three-edition lineup

Poem

🐰 Four editions now hop through the land,
Airgapped joins, secure and grand!
With isolation's shield held tight,
Self-hosting shines in digital light. 🌟

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Title Check ⚠️ Warning The title contains two components: "Added Airgapped to Editions" and "updated K8s configurations settings." The first component is accurate and well-supported by the raw_summary, which documents the addition of an Airgapped Edition to the self-hosted offerings within the self-hosting/editions-and-versions.mdx file. However, the second component regarding K8s configuration updates is not present in the changeset—the raw_summary shows only changes to the editions documentation file with no mention of any Kubernetes configuration files or settings being modified. This makes the title partially misleading. The title should be revised to accurately reflect the actual changes in the pull request. Consider updating it to "Added Airgapped Edition to self-hosting documentation" or similar, removing the reference to K8s configuration updates that are not present in the changeset. If Kubernetes configuration changes do exist, they should be included in the pull request scope and documented in the raw_summary.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch updates-30-10-25
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3a1c8d5 and b8ff3a4.

📒 Files selected for processing (1)
  • self-hosting/editions-and-versions.mdx (4 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). (1)
  • GitHub Check: Cursor Bugbot
🔇 Additional comments (4)
self-hosting/editions-and-versions.mdx (4)

6-6: Math checks out—four editions properly enumerated.

The intro correctly identifies one hosted Cloud edition and three self-hosted editions (Community, Commercial, Airgapped). No issues.

48-49: Verify clarity of version updates guidance for airgapped users.

Line 49 states "Updates from your own docker registry" but may leave unanswered questions about the actual process (e.g., how to populate the registry, reference to upgrade docs). Consider adding a link or brief note directing users to the upgrade documentation, similar to how the intro mentions Update Plane.

Does the file reference or link to detailed airgapped upgrade instructions elsewhere in the documentation, or would clarifying this line improve usability?

64-66: Updated rationale is clear and consistent.

The new Airgapped bullet correctly positions it as extending Commercial Edition capabilities to isolated environments. Maintains parallel structure and messaging tone.

80-80: Version update guidance correctly includes all self-hosted editions.

The update properly includes Airgapped alongside Commercial and Community Editions for version update control. Consistent with the rest of the document.

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sriramveeraghanta sriramveeraghanta merged commit 99b8abe into preview Oct 31, 2025
5 of 6 checks passed
@sriramveeraghanta sriramveeraghanta deleted the updates-30-10-25 branch October 31, 2025 13:00
@danciaclara danciaclara mentioned this pull request Oct 31, 2025
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sriramveeraghanta sriramveeraghanta sriramveeraghanta approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@danciaclara @sriramveeraghanta