Skip to content

[Network Interconnect] Remove beta tag from CNI Connection Maintenance Alert#31844

Merged
alpdot merged 1 commit into
productionfrom
steve-cloudflare/cni-maintenance-alert-ga
Jul 2, 2026
Merged

[Network Interconnect] Remove beta tag from CNI Connection Maintenance Alert#31844
alpdot merged 1 commit into
productionfrom
steve-cloudflare/cni-maintenance-alert-ga

Conversation

@steve-cloudflare

@steve-cloudflare steve-cloudflare commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

Removes the "(beta)" tag from the CNI Connection Maintenance Alert now that the feature is GA. Updates the section description and heading in network-interconnect/monitoring-and-alerts.

Documentation checklist

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/content/docs/network-interconnect/ @marciocloudflare, @steve-cloudflare, @jeffh-cloudflare, @alpdot, @cloudflare/product-owners

@ask-bonk

ask-bonk Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Review for PR #31844

Summary
This PR removes the "(beta)" tag from the CNI Connection Maintenance Alert in src/content/docs/network-interconnect/monitoring-and-alerts.mdx. The change is minimal and correct:

  • Removes (beta) from the bold description sentence in the "Alerts (v1 dataplane only)" section.
  • Removes (beta) from the H3 procedure heading "Enable CNI Connection Maintenance Alert".

I verified there are no remaining references to CNI Connection Maintenance Alert (beta) elsewhere in src/content/. pnpm run check passes with no errors related to this change.

Suggested labels

  • network-interconnect
  • content (or documentation)

Non-blocking suggestions / flags for the author

  1. Documentation checklist: The PR body has an unchecked item for style guide adherence. Please check it off if you've confirmed the change follows the style guide.
  2. Optional frontmatter update: Since the page is being actively updated, consider adding reviewed: 2026-07-02 to the frontmatter to record the review date.
  3. No changelog needed: This is a straightforward docs label cleanup following a GA release, so a separate changelog entry is not required here.

Verdict
No blocking issues. LGTM.

github run

@steve-cloudflare steve-cloudflare marked this pull request as ready for review July 2, 2026 04:34
@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Review

💡 1 suggestion found in commit 75d0a14.

Code Review

This code review is in beta and may not always be helpful — use your judgment.

No code review issues found.

Conventions

Checks PR title, description, and redirect checklist.

No convention issues found.

Style Guide Review

Suggestions (1)
File Issue
network-interconnect/monitoring-and-alerts.mdx line 30 Oxford comma before the coordinating conjunction in a list of three or more items — Line lists scheduled, updated, or canceled without a comma before or. Fix: Add a serial comma: scheduled, updated, or canceled.

Redirects

No missing redirect entries found.

Commands

Only codeowners can run commands. Post a comment with the command to trigger it.

Command Description
/review Runs a review now. Incremental if a prior review exists, full if not.
/full-review Re-reviews the entire PR diff from scratch, ignoring incremental history. Useful after a rebase, when you want a fresh review, or if the bot gets out of sync and reports issues that no longer exist.
/ignore-review-limit Permanently lifts the 2-review automatic limit for this PR. Future pushes will trigger reviews as normal.
/disable-auto-review Stops automatic reviews from triggering on future pushes to this PR. Codeowners can still run /review or /full-review manually.

@alpdot alpdot merged commit fc2e11d into production Jul 2, 2026
22 checks passed
@alpdot alpdot deleted the steve-cloudflare/cni-maintenance-alert-ga branch July 2, 2026 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants