Skip to content

[DNS] Document child zone delegation behavior for subdomain setup decommissioning#31638

Open
ngayerie wants to merge 1 commit into
productionfrom
ngayerie/DEE-3626
Open

[DNS] Document child zone delegation behavior for subdomain setup decommissioning#31638
ngayerie wants to merge 1 commit into
productionfrom
ngayerie/DEE-3626

Conversation

@ngayerie

Copy link
Copy Markdown
Collaborator

Summary

When a subdomain is set up as a child zone on Cloudflare, NS delegation is served automatically based on the child zone's active status — not based on NS records stored in the parent zone. Customers may expect that deleting NS records from the parent zone stops the delegation, but it doesn't. The child zone must be deleted to stop serving delegation.

This PR adds a "Decommission a subdomain setup" section to the subdomain setup page with clear steps.

Changes

File Change
dns/zone-setups/subdomain-setup/setup/index.mdx New section: "Decommission a subdomain setup" with explanation and 4-step procedure

Stats

1 file changed, +13 insertions

Resolves DEE-3626

@github-actions

github-actions Bot commented Jun 23, 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/dns/ @RebeccaTamachiro, @hannes-cf, @cloudflare/product-owners, @esomoza-cf, @fattouche, @xofyarg, @dklbreitling, @chreo, @svenr-cf, @kerolasa, @matildeopbravo, @vavrusa, @mworsley-cloudflare, @sebastiaanyn, @vendemiat, @Woutifier

@ngayerie ngayerie changed the title Document child zone delegation behavior for subdomain setup decommissioning [DNS] Document child zone delegation behavior for subdomain setup decommissioning Jun 23, 2026
@ask-bonk ask-bonk Bot added the documentation Documentation edits label Jun 23, 2026

@ask-bonk ask-bonk Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

1 issue to address before merge.

MEDIUM — The 4-step procedure must use the Steps component (mandatory for multi-step procedures per the style guide). Also add Steps to the component import on line 14. LOW — Remove redundant "If needed" from the optional step.

Posted 1 inline suggestion.

Comment thread src/content/docs/dns/zone-setups/subdomain-setup/setup/index.mdx Outdated
@ask-bonk

ask-bonk Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Review complete.

Labels applied: documentation (added), product:dns (already present).

PR summary: Adds a "Decommission a subdomain setup" section to /dns/zone-setups/subdomain-setup/setup/ explaining that NS delegation for child zones is served automatically based on zone status, not parent-zone NS records, so the child zone must be deleted to stop delegation. The addition is clear, technically accurate, and fills a documentation gap.

Issues flagged:

  1. MEDIUMsrc/content/docs/dns/zone-setups/subdomain-setup/setup/index.mdx:80-83: The 4-step procedure must be wrapped in the <Steps> component, which is mandatory for multi-step procedures per the style guide.
  2. LOW — Same file, line 83: "(Optional) If needed," is redundant. (Optional) already signals that the step is conditional; remove "If needed."
  3. LOW — Same file, line 14: The Steps component must be added to the existing import from ~/components.

Posted a review with an inline suggestion covering the <Steps> wrapper and the redundant wording fix. The import addition was noted in the review summary (GitHub does not allow inline suggestions on unchanged lines outside the diff context).

github run

@ngayerie ngayerie force-pushed the ngayerie/DEE-3626 branch from 2e0fcf9 to 5384b30 Compare June 23, 2026 07:22
…ioning

When a subdomain is set up as a child zone, Cloudflare serves NS delegation
based on the zone's active status, not based on NS records in the parent zone.
Customers must delete the child zone to stop delegation — removing NS records
from the parent zone alone does not work.

Resolves DEE-3626
@ngayerie ngayerie force-pushed the ngayerie/DEE-3626 branch from 5384b30 to 3f92282 Compare June 23, 2026 07:24
@github-actions

Copy link
Copy Markdown
Contributor

@ngayerie ngayerie marked this pull request as ready for review June 23, 2026 07:59
@ngayerie ngayerie requested review from a team, Woutifier and vendemiat as code owners June 23, 2026 07:59
@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Review

✅ No issues found in commit 3f92282.

Code Review

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

✅ No code review issues found.

Style Guide Review

✅ No style-guide issues 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Documentation edits product:dns Issues or PRs related to DNS size/s

Projects

None yet

Development

Successfully merging this pull request may close these issues.