Skip to content

chore: sync deployment-operator CRDs - #801

Open
github-actions[bot] wants to merge 1 commit into
mainfrom
crd-sync-2026-08-12
Open

chore: sync deployment-operator CRDs#801
github-actions[bot] wants to merge 1 commit into
mainfrom
crd-sync-2026-08-12

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Automated pull request to sync deployment-operator CRDs from console.

@github-actions
github-actions Bot requested a review from a team as a code owner August 12, 2026 00:29
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Aug 12, 2026
@greptile-apps

greptile-apps Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This automated CRD synchronization exposes component SHA cache TTL configuration and adds PI as a supported agent runtime.

  • Adds componentShaCacheTTL to AgentConfiguration.spec.
  • Adds PI runtime configuration fields and admits PI in the AgentRuntime type enum.
  • Leaves PI's AI-proxy model-prefix behavior undocumented.

Confidence Score: 4/5

The PR appears safe to merge, with a non-blocking documentation gap around PI model routing through the AI proxy.

The synchronized schemas consistently add the new fields, but PI users enabling aiProxy are not told how bare model identifiers are prefixed or whether provider/name form is required.

Files Needing Attention: charts/deployment-operator/crds/deployments.plural.sh_agentruntimes.yaml

Important Files Changed

Filename Overview
charts/deployment-operator/crds/deployments.plural.sh_agentconfigurations.yaml Adds an optional component SHA cache TTL string consistent with the CRD's existing duration-field schemas.
charts/deployment-operator/crds/deployments.plural.sh_agentruntimes.yaml Adds PI runtime configuration and enum support, but the existing AI-proxy documentation does not explain PI's bare-model provider mapping.

Reviews (1): Last reviewed commit: "chore: sync deployment-operator CRDs" | Re-trigger Greptile

- OPENCODE
- GEMINI
- CODEX
- PI

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.

P2 Document PI proxy model routing

The new PI runtime supports aiProxy, but the proxy documentation enumerates bare-model prefix behavior for every runtime except PI. Users therefore cannot determine whether PI model values are automatically prefixed or must use explicit provider/name form, leading to ambiguous or incorrectly routed configurations.

Knowledge Base Used: Deployment Operator Helm Chart

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant