Skip to content

DOCS-3088: Fix inaccuracies on the IPAMConfiguration reference - #3126

Merged
ctauchen merged 2 commits into
tigera:mainfrom
ctauchen:DOCS-3088-fact-check-ipamconfig
Oct 19, 2026
Merged

ctauchen merged 2 commits into
tigera:mainfrom
ctauchen:DOCS-3088-fact-check-ipamconfig

Conversation

@ctauchen

@ctauchen ctauchen commented Oct 16, 2026 •

Copy link
Copy Markdown
Collaborator

Fixes inaccuracies on the IPAMConfiguration reference page, found and verified with the calico-fact-check skill.

  • maxBlocksPerHost accepts 0 to 1000000. Native v3 CRD mode, the default for new 3.33 installs, rejects larger values. Every value in that range is also accepted in API server mode.
  • Adds the missing ipCooldownSeconds row: 0 to 1200 seconds, default 0.

Changes next and 3.33, reproduced on kind clusters running v3.33.0. 3.32 and 3.31 are unchanged: ipCooldownSeconds does not exist at those releases, and their default installs accept the documented maxBlocksPerHost range.

Not changed, for follow-up:

  • The table omits autoAllocateBlocks. In API server mode the API server forces it to true, and in native v3 CRD mode false stops new block claims, so it needs an engineering decision on whether it is user-facing.
  • At 3.32, the 1000000 limit applies only to the tech-preview native v3 CRD mode. The page could give the limit per mode.
  • calicoctl ipam configure refuses maxBlocksPerHost when strictAffinity is false, but the sample sets that combination with kubectl.

Previous attempt: #3078.

Preview the changed page:

Copilot AI lite review requested due to automatic review settings October 16, 2026 15:56

Copilot AI 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.

🟢 Approval recommended

The documentation corrections are focused and no blocking issues were identified.

0 open findings

What changed in this PR

Updates IPAMConfiguration reference documentation for current and Calico 3.33 versions.

Changes:

  • Corrects maxBlocksPerHost range to 0–1000000.
  • Documents ipCooldownSeconds, including its range and default.
File Description
calico/​reference/​resources/​ipamconfig.mdx Updates the current IPAMConfiguration reference.
calico_versioned_docs/​version-3.33/​reference/​resources/​ipamconfig.mdx Updates the Calico 3.33 IPAMConfiguration reference.

🧠 Review effort: Lite


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@netlify

netlify Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for calico-docs-preview-next ready!

Name Link
🔨 Latest commit 3a4b798
🔍 Latest deploy log https://app.netlify.com/projects/calico-docs-preview-next/deploys/6ad24947fc70a163343de86e
😎 Deploy Preview https://deploy-preview-3126--calico-docs-preview-next.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview succeeded!

Built without sensitive environment variables

Name Link
🔨 Latest commit 3a4b798
🔍 Latest deploy log https://app.netlify.com/projects/tigera/deploys/6ad24947a617195bd20f010e
😎 Deploy Preview https://deploy-preview-3126--tigera.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 65 (🔴 down 8 from production)
Accessibility: 98 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@ctauchen
ctauchen marked this pull request as ready for review October 19, 2026 14:03
@ctauchen
ctauchen requested a review from a team as a code owner October 19, 2026 14:03

@ctauchen ctauchen left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

LGTM

@ctauchen
ctauchen merged commit b5c2728 into tigera:main Oct 19, 2026
11 checks passed
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.

2 participants