Skip to content

Add separate CRD chart to enterprise Helm install docs#2557

Merged
caseydavenport merged 2 commits intomainfrom
casey/crd-chart-docs-CORE-12418
Mar 3, 2026
Merged

Add separate CRD chart to enterprise Helm install docs#2557
caseydavenport merged 2 commits intomainfrom
casey/crd-chart-docs-CORE-12418

Conversation

@caseydavenport
Copy link
Member

The tigera-operator chart no longer bundles CRDs (see projectcalico/calico#11727) — they're now in a separate crd.projectcalico.org.v1 chart. The open source Calico install docs already have this, but the enterprise install docs were still missing it in a few places.

This adds the CRD chart download + install step to:

  • Standard enterprise Helm install
  • Management cluster Helm install (both NodePort and LoadBalancer)
  • Managed cluster Helm install

Also fixes the existing CRD install command in the standard enterprise doc (was missing .tgz extension and had an unnecessary --version flag for a local file).

Fixes https://tigera.atlassian.net/browse/CORE-12418

The tigera-operator chart no longer bundles CRDs — they're now in
a separate crd.projectcalico.org.v1 chart. This updates the enterprise
Helm install docs to match the pattern already in place for open source
Calico: download the CRD chart, install it before the operator.

Covers standard install, management cluster, and managed cluster pages.
Copilot AI review requested due to automatic review settings March 3, 2026 15:18
@caseydavenport caseydavenport requested a review from a team as a code owner March 3, 2026 15:18
@netlify
Copy link

netlify bot commented Mar 3, 2026

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

Name Link
🔨 Latest commit eef86bd
🔍 Latest deploy log https://app.netlify.com/projects/calico-docs-preview-next/deploys/69a72999df2f5700082f7247
😎 Deploy Preview https://deploy-preview-2557--calico-docs-preview-next.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@netlify
Copy link

netlify bot commented Mar 3, 2026

Deploy Preview succeeded!

Name Link
🔨 Latest commit eef86bd
🔍 Latest deploy log https://app.netlify.com/projects/tigera/deploys/69a729992073e40008aa62b7
😎 Deploy Preview https://deploy-preview-2557--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: 68 (🔴 down 5 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

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

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Calico Enterprise Helm installation documentation to include the separate crd.projectcalico.org.v1 chart that was split out from the tigera-operator chart (see referenced upstream PR #11727). The open source Calico docs already reflected this change, but the enterprise docs were missing the CRD chart download and install steps in several places.

Changes:

  • Adds helm pull tigera-ee/crd.projectcalico.org.v1 download step to the standard enterprise Helm install, management cluster (both NodePort and LoadBalancer), and managed cluster docs.
  • Adds a new helm install calico-crds crd.projectcalico.org.v1-$[chart_version_name].tgz --namespace tigera-operator step to all three multicluster Helm install docs.
  • Fixes the existing CRD install command in the standard enterprise doc by adding the missing .tgz extension and removing an incorrect --version flag (not applicable when installing from a local .tgz file).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
calico-enterprise/getting-started/install-on-clusters/kubernetes/helm.mdx Adds missing helm pull for CRD chart; fixes existing CRD install command (.tgz extension, removes erroneous --version flag)
calico-enterprise/multicluster/set-up-multi-cluster-management/helm-install/create-a-management-cluster-helm.mdx Adds CRD chart helm pull and CRD helm install steps for both NodePort and LoadBalancer tabs
calico-enterprise/multicluster/set-up-multi-cluster-management/helm-install/create-a-managed-cluster-helm.mdx Adds CRD chart helm pull and CRD helm install steps

Copy link
Collaborator

@ctauchen ctauchen left a comment

Choose a reason for hiding this comment

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

Do the copilot comments merit attention? Otherwise LGTM. @caseydavenport

The CRD chart is installed first, so it needs to create the namespace.
The operator install runs after, so the namespace already exists.
@caseydavenport caseydavenport merged commit 05cb0d0 into main Mar 3, 2026
11 checks passed
@caseydavenport caseydavenport deleted the casey/crd-chart-docs-CORE-12418 branch March 3, 2026 19:05
ctauchen pushed a commit to ctauchen/docs that referenced this pull request Mar 4, 2026
ctauchen pushed a commit to ctauchen/docs that referenced this pull request Mar 4, 2026
ctauchen pushed a commit to ctauchen/docs that referenced this pull request Mar 4, 2026
ctauchen pushed a commit to ctauchen/docs that referenced this pull request Mar 4, 2026
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.

3 participants