Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

Context

This PR fixes issues in the self-hosting overview page identified during review of PR #5106:

  1. Bug fix: The GCP card was incorrectly linking to docker-swarm instead of gcp-native
  2. Content fix: The Kubernetes card description was duplicated from the Docker card - now has a unique description
  3. Formatting: Made all card descriptions consistent (all ending with periods)
  4. Descriptions: Updated AWS and GCP card descriptions to be more specific about the deployment method (ECS with Fargate, GKE)

Also includes the expanded intro content and new deployment option cards (Docker Swarm, AWS, GCP) from the parent PR.

Steps to verify the change

  1. Navigate to the self-hosting overview page in the docs
  2. Verify the GCP card links to /self-hosting/deployment-options/gcp-native
  3. Verify all card descriptions are accurate and end with periods
  4. Verify the Kubernetes card description mentions "Helm chart" (not the Docker image text)

Type

  • Fix
  • Feature
  • Improvement
  • Breaking
  • Docs
  • Chore

Checklist


Requested by: ashwin@infisical.com
Link to Devin run: https://app.devin.ai/sessions/de8b36ae4d4c4287ace71741aef029fe

Co-Authored-By: ashwin@infisical.com <ashwin@infisical.com>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@devin-ai-integration devin-ai-integration bot changed the title Fix self-hosting overview page: correct GCP link and update card descriptions docs: fix self-hosting overview page GCP link and card descriptions Jan 28, 2026
@maidul98
Copy link
Collaborator

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 28, 2026

Greptile Overview

Greptile Summary

This PR fixes documentation issues in the self-hosting overview page, specifically correcting the GCP card link from docker-swarm to gcp-native and updating the Kubernetes card with a unique description mentioning the Helm chart. The changes also expand the intro content with compliance and flexibility details, and add new deployment option cards for Docker Swarm, AWS, and GCP.

  • Fixed GCP card link to point to correct gcp-native path
  • Updated Kubernetes description to mention Helm chart deployment
  • Added expanded intro explaining compliance and flexibility benefits
  • Added new deployment cards for Docker Swarm, AWS (ECS with Fargate), and GCP (GKE)
  • The AWS and GCP documentation files (aws-native and gcp-native) could not be verified to exist in the repository

Confidence Score: 4/5

  • This documentation-only PR is safe to merge with minimal risk.
  • The changes are limited to documentation content (MDX file) with no code logic changes. The fixes correctly address the issues described in the PR (GCP link, Kubernetes description). One concern is that the AWS and GCP documentation files referenced may not exist yet, which should be verified before merging.
  • Verify that target documentation files for AWS and GCP deployment options exist before merging.

Important Files Changed

Filename Overview
docs/self-hosting/overview.mdx Documentation updates with expanded intro content, fixed GCP link, unique Kubernetes description, and added deployment option cards

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@vmatsiiako vmatsiiako merged commit 5d95e04 into main Feb 4, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants