Skip to content

TKE-46: fix delete node pool parameters#58

Draft
wenjiehs wants to merge 1 commit into
tke-workshop:mainfrom
wenjiehs:agent/tke-workshop-doc-change-agent/c506760b
Draft

TKE-46: fix delete node pool parameters#58
wenjiehs wants to merge 1 commit into
tke-workshop:mainfrom
wenjiehs:agent/tke-workshop-doc-change-agent/c506760b

Conversation

@wenjiehs

Copy link
Copy Markdown
Collaborator

Doc Change Report

Source issue: TKE-46

Scope

  • Updated src/content/docs/basics/node-pool/04-delete-node-pool.md only.
  • Corrected DeleteClusterNodePool examples from singular NodePoolId to array NodePoolIds.
  • Added required KeepInstance choice in CLI and Python SDK examples.
  • Added retain-vs-release reader guidance and verification wording.

Document Identity

  • canonical_doc_path: src/content/docs/basics/node-pool/04-delete-node-pool.md
  • operation_id: tke.nodepool.delete

No duplicate operation entry was changed in this PR.

Fact Sources

  • Tencent Cloud official API documentation: https://intl.cloud.tencent.com/ind/document/product/457/38779
    • Official page update time: 2026-05-27 13:00:05
    • Adopted because it defines DeleteClusterNodePool request parameters as ClusterId, NodePoolIds.N, and required KeepInstance, and describes KeepInstance=true as retaining instances while removing nodes from the cluster.
  • Repository source page: src/content/docs/basics/node-pool/04-delete-node-pool.md
    • Adopted because it is the issue-scoped document containing the incorrect examples.

Static Compile

  • git diff --check passed.
  • Markdown/code-block language inspection passed for the changed page.
  • Command examples were checked against the official API parameter names.
  • Python SDK field names were checked against the official API model shape.

Dry Run Compile

  • node --test test/navigation.test.mjs test/cookbooks.test.mjs test/create-cluster-autoupgrade.test.mjs passed: 10 tests, 10 pass.
  • python3 -m py_compile cookbook/common/auth.py cookbook/common/logger.py cookbook/cluster/create_cluster.py cookbook/cluster/delete_cluster.py cookbook/cluster/describe_clusters.py cookbook/supernode/deploy_gpu_pod.py cookbook/workload/deploy_nginx.py passed.
  • npm run build passed: 184 pages built.
  • MkDocs strict build skipped: no mkdocs.yml or mkdocs.yaml exists in this repo.

Live Verify

  • Live Verify required: no. This change corrects documented API parameter names and examples only; no Tencent Cloud resource creation/deletion was performed or needed.

Remaining Human Confirmation

  • None.

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.

1 participant