Skip to content

docs(gateway): update systemd service naming docs#6624

Merged
bernie-g merged 3 commits into
mainfrom
feat/match-gateway-systemd-service-names-docs
May 28, 2026
Merged

docs(gateway): update systemd service naming docs#6624
bernie-g merged 3 commits into
mainfrom
feat/match-gateway-systemd-service-names-docs

Conversation

@bernie-g
Copy link
Copy Markdown
Contributor

Summary

  • Update docs to reflect new systemd naming where service name matches gateway name
  • Add systemd uninstall command documentation
  • Update config paths, systemctl/journalctl examples, and FAQ references

Companion to Infisical/cli#244

Test plan

  • Verify docs render correctly on Mintlify

Reflect new naming where the systemd service name matches the gateway
name. Add systemd uninstall command documentation. Update config paths,
systemctl examples, and FAQ references.
@infisical-review-police
Copy link
Copy Markdown

💬 Discussion in Slack: #pr-review-infisical-6624-docs-gateway-update-systemd-service-naming-docs

Posted by Review Police — reviews, comments, new commits, and CI failures will stream into this channel.

@mintlify
Copy link
Copy Markdown

mintlify Bot commented May 27, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
infisical 🟢 Ready View Preview May 27, 2026, 5:06 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@bernie-g bernie-g marked this pull request as ready for review May 27, 2026 17:19
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 27, 2026

Greptile Summary

This PR updates the gateway docs to reflect the new systemd naming convention where the service name matches the gateway name, and adds documentation for the new systemd uninstall subcommand.

  • Service/config paths in gateway.mdx updated from the hardcoded infisical-gateway name to <name>-scoped paths, and systemctl examples now use my-gateway as a concrete placeholder with an explanatory note.
  • gateway-deployment.mdx aligns the systemctl start, journalctl, and config-path references to the new naming; a minor placeholder inconsistency (<name> vs <gateway-name>) exists in the journalctl example.
  • A new infisical gateway systemd uninstall accordion is added with the legacy fallback behavior documented.

Confidence Score: 4/5

Documentation-only change that accurately reflects the new gateway systemd naming behavior; safe to merge.

Both files are documentation with no runtime code. The changes correctly update service paths, config paths, and systemctl/journalctl examples to match the new name-scoped systemd convention. The only rough edge is a minor placeholder name mismatch (<name> vs <gateway-name>) in the journalctl example in the deployment guide.

docs/documentation/platform/gateways/gateway-deployment.mdx — placeholder naming inconsistency in the journalctl example.

Important Files Changed

Filename Overview
docs/cli/commands/gateway.mdx Updates service/config paths to reflect name-scoped systemd naming and adds the new systemd uninstall subcommand; changes are accurate and internally consistent.
docs/documentation/platform/gateways/gateway-deployment.mdx Aligns systemctl/journalctl examples and config path with new naming; minor inconsistency where the journalctl placeholder uses <name> while the rest of the file uses <gateway-name>.

Reviews (1): Last reviewed commit: "docs(gateway): update systemd service na..." | Re-trigger Greptile

Comment thread docs/documentation/platform/gateways/gateway-deployment.mdx Outdated
Comment thread docs/cli/commands/gateway.mdx Outdated
Comment thread docs/documentation/platform/gateways/gateway-deployment.mdx
@veria-ai
Copy link
Copy Markdown

veria-ai Bot commented May 28, 2026

PR overview

All previously flagged issues have been addressed. No open security concerns remain on this pull request.

Security review

No open security issues remain on this pull request.

Fixed/addressed: 2 · PR risk: 0/10

@bernie-g bernie-g merged commit 7bef252 into main May 28, 2026
13 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