Skip to content

fix the namespace of the add-on in vks cluster.yaml example#2847

Merged
sujeet-kr merged 1 commit into
mainfrom
sk-carvel-example-vks-ns
Jul 10, 2026
Merged

fix the namespace of the add-on in vks cluster.yaml example#2847
sujeet-kr merged 1 commit into
mainfrom
sk-carvel-example-vks-ns

Conversation

@sujeet-kr

Copy link
Copy Markdown
Collaborator

Product Version(s):

Issue:

Link to docs preview:

SME review:

  • An SME has approved this change.

DOCS review:

  • A member of the docs team has approved this change.

Additional information:

Merge checklist:

  • Deploy preview inspected wherever changes were made
  • Build completed successfully
  • Test have passed

Copilot AI review requested due to automatic review settings July 10, 2026 16:52
@sujeet-kr sujeet-kr requested a review from a team as a code owner July 10, 2026 16:52
@netlify

netlify Bot commented Jul 10, 2026

Copy link
Copy Markdown

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

Name Link
🔨 Latest commit 971a003
🔍 Latest deploy log https://app.netlify.com/projects/calico-docs-preview-next/deploys/6a5123401e9fe20008051a53
😎 Deploy Preview https://deploy-preview-2847--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

netlify Bot commented Jul 10, 2026

Copy link
Copy Markdown

Deploy Preview succeeded!

Name Link
🔨 Latest commit 971a003
🔍 Latest deploy log https://app.netlify.com/projects/tigera/deploys/6a512340c3ab4f000888d468
😎 Deploy Preview https://deploy-preview-2847--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: 66 (🔴 down 28 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.

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.

Pull request overview

Updates the Calico Enterprise VKS installation documentation to use the correct namespace for the add-on reference in the cluster.yaml example, aligning it with the namespace used elsewhere on the page.

Changes:

  • Fix cniRef.namespace in the VKS Cluster manifest example to vmware-system-vks-public.
  • Apply the same correction to the corresponding versioned docs page (3.23-2).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
calico-enterprise/getting-started/install-on-clusters/vks.mdx Corrects the add-on cniRef.namespace in the Cluster YAML example.
calico-enterprise_versioned_docs/version-3.23-2/getting-started/install-on-clusters/vks.mdx Backports the same namespace correction to the 3.23-2 versioned page.

@ctauchen ctauchen left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@sujeet-kr sujeet-kr merged commit 91f7925 into main Jul 10, 2026
12 checks passed
@sujeet-kr sujeet-kr deleted the sk-carvel-example-vks-ns branch July 10, 2026 20:44
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