Skip to content

docs: rename the gateway's keep alive values to health check - #3451

Merged
flin-8 merged 7 commits into
mainfrom
frank/gateway-health-check-values
Sep 9, 2026
Merged

docs: rename the gateway's keep alive values to health check#3451
flin-8 merged 7 commits into
mainfrom
frank/gateway-health-check-values

Conversation

@flin-8

@flin-8 flin-8 commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Follows the gateway chart rename in octopus-argocd-gateway#276: gateway.octopus.keepAlive.{intervalSeconds,maxConsecutiveFailures} became gateway.octopus.healthCheck.{interval,giveUpAfter}, and both are now duration strings rather than integers.

  • Gateway chart values — the two rows replaced, pasted from the chart's helm-docs output
  • Argo CD Instances overview — the load balancer hint points at the new key
  • Troubleshooting — the connection-drops section reworded, and the --set example carries the unit (interval="15s")

The quoted log line becomes Health check failed - cancelling subscribers on both this page and the Kubernetes monitor's troubleshooting page. octopus-grpc#11 renamed the library's log lines, released as go/v0.5.1, so the old string no longer appears in either pod's logs once their pins move (lobster-watcher#366, gateway#276).

⚠️ Please hold this until the gateway chart ships the rename.

Comment thread src/pages/docs/argo-cd/troubleshooting.md Outdated
Comment thread src/pages/docs/argo-cd/troubleshooting.md
@team-marketing-branch-protections

Copy link
Copy Markdown

Pull request environment is available at https://stoctodocspr3451.z22.web.core.windows.net/.

You can view the ephemeral environment status in Octopus Deploy.

This environment will be automatically deprovisioned when the pull request is closed, or after 7 days of inactivity.

Comment thread src/pages/docs/packaging-applications/build-servers/buildkite.md
flin-8 and others added 3 commits September 9, 2026 09:40
…check-values

# Conflicts:
#	src/pages/docs/packaging-applications/build-servers/buildkite.md
octopus-grpc 0.5.1 logs "Health check failed - cancelling subscribers"; the
old "keep alive check failed" string is what operators were told to grep for.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The --set flag would not parse, and it named the gateway's value key on the
monitor's page. Monitor values are nested under kubernetesMonitor when the
agent chart deploys the monitor as a subchart.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@flin-8 flin-8 self-assigned this Sep 9, 2026
@flin-8
flin-8 merged commit 3028f47 into main Sep 9, 2026
7 checks passed
@flin-8
flin-8 deleted the frank/gateway-health-check-values branch September 9, 2026 02:57
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.

3 participants