[INFRA-455] feat: add agent consumer#270
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
WalkthroughAdds an optional agent consumer service to the enterprise Helm chart, including configuration values, setup questions, documentation, a runtime ConfigMap, and a Kubernetes Deployment with resource, scheduling, probe, secret, and environment settings. ChangesAgent Consumer Deployment
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant HelmValues
participant AgentConsumerConfigMap
participant AgentConsumerDeployment
participant AgentConsumerProcess
HelmValues->>AgentConsumerConfigMap: Render queue_name and prefetch_count
HelmValues->>AgentConsumerDeployment: Render enabled workload configuration
AgentConsumerConfigMap->>AgentConsumerDeployment: Provide runtime configuration
AgentConsumerDeployment->>AgentConsumerProcess: Start agent consumer container
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Linked to Plane Work Item(s) References This comment was auto-generated by Plane |
What
Adds a new opt-in Agent Consumer deployment to the
plane-enterprisechart:templates/workloads/agent-consumer.deployment.yaml,templates/config-secrets/agent-consumer.yaml,services.agent_consumer/env.agent_consumer_envsblocks invalues.yaml, aquestions.ymlinstaller group, README docs, and a chart version bump (3.0.0→3.1.0).Why
Companion to
helm-charts-private@5cfc2d8(INFRA-432, "add support for new agent consumer", #677), which added this RabbitMQ-backed agent consumer to theplane-cloudchart. This brings the same consumer to the enterprise self-hosted chart.Scope / behavior
.Values.services.agent_consumer.enabled(defaultfalse) in both new templates — no rendered resources and no behavior change for existing installs unless explicitly enabled.webhook_consumerservice pattern exactly (labels/annotations helper, pod scheduling, pod/container security context, doc-store + opensearch secrets, conditional silo secret whenservices.silo.enabled) — kept consistent with every other consumer in this chart rather than a literal port of theplane-cloudversion, which lacks those helpers/secrets.<release>-agent-consumer-varsexposesAGENT_QUEUE_NAME(defaultplane.agent) andAGENT_PREFETCH_COUNT(default10).plane-cechart — this consumer doesn't exist there, matchingwebhook_consumer/automation_consumer, which are enterprise-only.Testing
No
helmbinary was available in this environment, sohelm template/helm lintwere not run.values.yaml,questions.yml, andChart.yamlwere confirmed to be parseable YAML. The new templates were hand-verified line-by-line againstwebhook-consumer.deployment.yaml/webhook-consumer.yamlfor structural parity. Recommend runninghelm template charts/plane-enterprise --set services.agent_consumer.enabled=truebefore merge.Related
Companion to helm-charts-private commit
5cfc2d8(INFRA-432 / #677).Summary by CodeRabbit
New Features
Documentation
Chores