fix(docs): fix slack webhook backtick (backport of #28085 to release/2.29) - #28177
Open
nickvigilante wants to merge 1 commit into
Open
fix(docs): fix slack webhook backtick (backport of #28085 to release/2.29)#28177nickvigilante wants to merge 1 commit into
nickvigilante wants to merge 1 commit into
Conversation
Two small monitoring-doc fixes surfaced by the runtime drift sweep. The native-histograms list showed `coderd_template_coderd_template_workspace_build_duration_seconds` (doubled `coderd_template_` prefix). The correct metric name, per the metrics table earlier on the same page and the generated metrics, is `coderd_template_workspace_build_duration_seconds`. The `CODER_NOTIFICATIONS_WEBHOOK_ENDPOINT` export ended with a stray backtick: ``` export CODER_NOTIFICATIONS_WEBHOOK_ENDPOINT=http://localhost:6000/v1/webhook` ``` On paste, bash treats the trailing backtick as an unterminated command substitution and errors. Removed it. Both reproduced during the runtime drift sweep and verified against `main`. Linear: [DOCS-647](https://linear.app/codercom/issue/DOCS-647/docs-fix-monitoring-examples-prometheus-metric-name-slack-webhook) > This PR was created with AI assistance (Coder Agents). (cherry picked from commit 3145cc8)
|
👋 Hey @nickvigilante! This PR is targeting the Only bug fixes should be cherry-picked to release branches. If this is a bug fix, please update the PR title to match the conventional commit format: If this is not a bug fix, it likely should not target a release branch. |
nickvigilante
marked this pull request as ready for review
August 14, 2026 17:46
nickvigilante
enabled auto-merge (squash)
August 14, 2026 17:46
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Backport of #28085 to
release/2.29(ESR).Replaces the stale automated backport #28164, whose branch was an empty placeholder based on an older
release/2.29. This branch is cut from the currentrelease/2.29tip and cherry-picks3145cc8386f9withgit cherry-pick -x.Scope note: only the
slack.mdfix (removing the stray trailing backtick from theCODER_NOTIFICATIONS_WEBHOOK_ENDPOINTexport) applies to 2.29. #28085 also fixed a doubled-prefix metric name inprometheus.md, butrelease/2.29's native-histograms list does not includecoderd_template_workspace_build_duration_secondsat all, so that hunk is intentionally dropped rather than adding a metric 2.29 does not document. Theslack.mdconflict kept 2.29'sbashfence.Once reviewed, #28164 can be closed in favor of this PR.