feat: add componentConfig and annotations API types (EP#1898)#126
feat: add componentConfig and annotations API types (EP#1898)#126swghosh wants to merge 1 commit intoopenshift:ai-staging-release-1.0from
Conversation
…sConfig Extend the ExternalSecretsConfig API with componentConfig and annotations fields as specified in enhancement proposal #1898 (ESO-266). New types added: - ComponentConfig: per-component deployment overrides and env vars - DeploymentConfig: deployment-level settings (revisionHistoryLimit) - Annotation: custom annotation key-value pairs New ComponentName enum values: Webhook, CertController Includes CEL validation for reserved annotation prefixes and env var names, integration test suite coverage, regenerated deepcopy and CRD manifests. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Important Review skippedAuto reviews are limited based on label configuration. 🚫 Review skipped — only excluded labels are configured. (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: swghosh The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Summary
ExternalSecretsConfigAPI withannotationsandcomponentConfigsfields onControllerConfigas specified in Enhancement Proposal #1898ComponentConfig,DeploymentConfig,AnnotationComponentNameenum values:Webhook,CertControllerkubernetes.io/,app.kubernetes.io/,openshift.io/,k8s.io/) and reserved environment variable prefixes (HOSTNAME,KUBERNETES_,EXTERNAL_SECRETS_)Test plan
🤖 Generated with Claude Code