Skip to content

Add replication context example#82

Open
sandert-k8s wants to merge 3 commits intoprojectcapsule:release/preview-v0.13from
sandert-k8s:small-updates
Open

Add replication context example#82
sandert-k8s wants to merge 3 commits intoprojectcapsule:release/preview-v0.13from
sandert-k8s:small-updates

Conversation

@sandert-k8s
Copy link
Copy Markdown
Collaborator

2 updates:

  1. Update proxy globalproxySetting, "pods" is incorrect, since it's not clusterwide. Change this to a clusterwide resource, such as globalclusterquota.
  2. Add additional example for the context.

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 3, 2026

Deploy Preview for docs-projectcapsule ready!

Name Link
🔨 Latest commit 7f21821
🔍 Latest deploy log https://app.netlify.com/projects/docs-projectcapsule/deploys/69cfb851d107690008416d15
😎 Deploy Preview https://deploy-preview-82--docs-projectcapsule.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.

@sandert-k8s sandert-k8s marked this pull request as ready for review April 3, 2026 12:36
Copilot AI review requested due to automatic review settings April 3, 2026 12:36
Copy link
Copy Markdown

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 documentation examples related to replication template context usage and Capsule Proxy global proxy settings, aiming to provide clearer, more accurate cluster-scoped examples.

Changes:

  • Adds additional “Context” examples for TenantResource and GlobalTenantResource templates (including a full context dump snippet).
  • Updates the GlobalProxySettings example to use a cluster-scoped Capsule resource instead of Pods.
  • Minor formatting/name fixes in “What’s New” markdown.

Reviewed changes

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

File Description
content/en/docs/whats-new.md Minor markdown/name formatting adjustments in the updates/events sections.
content/en/docs/replications/tenant.md Expands “Context” section with context-dump guidance and an imagePullSecrets use case.
content/en/docs/replications/global.md Mirrors the “Context” expansion for global replications with a dependsOn-based example.
content/en/docs/proxy/proxysettings.md Updates GlobalProxySettings cluster-scoped resource example and formatting.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: sandert-k8s <sandert98@gmail.com>
Copy link
Copy Markdown

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

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


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 49 to 61
clusterResources:
- apiGroups:
- ""
- "capsule.clastix.io/v1beta2"
resources:
- "pods"
- "globalcustomquotas"
operations:
- List
selector:
matchLabels:
app.kubernetes.io/type: dev
capsule.clastix.io/tenant: green
- apiGroups:
- "kyverno.io/v1"
resources:
Copy link

Copilot AI Apr 3, 2026

Choose a reason for hiding this comment

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

In this example apiGroups values include the API version (e.g. capsule.clastix.io/v1beta2 and kyverno.io/v1). The text above states these fields work like Kubernetes ClusterRoles, where apiGroups must be the group only (e.g. capsule.clastix.io / kyverno.io), with the version omitted. Using group/version here may mislead users and won’t match Kubernetes RBAC semantics.

Copilot uses AI. Check for mistakes.
Signed-off-by: sandert-k8s <sandert98@gmail.com>
Signed-off-by: sandert-k8s <sandert98@gmail.com>
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