chore: sync deployment-operator CRDs - #801
Conversation
Greptile SummaryThis automated CRD synchronization exposes component SHA cache TTL configuration and adds PI as a supported agent runtime.
Confidence Score: 4/5The 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
|
| 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 |
There was a problem hiding this comment.
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!
Automated pull request to sync deployment-operator CRDs from console.