Skip to content

[INFRA-443] - feat(plane-enterprise): add external-api and worker-importers workloads#264

Open
akshat5302 wants to merge 1 commit into
masterfrom
feat/plane-enterprise-external-api-worker-importers
Open

[INFRA-443] - feat(plane-enterprise): add external-api and worker-importers workloads#264
akshat5302 wants to merge 1 commit into
masterfrom
feat/plane-enterprise-external-api-worker-importers

Conversation

@akshat5302

@akshat5302 akshat5302 commented Jul 10, 2026

Copy link
Copy Markdown
Member
  • Add new importer worker deployment,
  • Add new external API deployment
  • Add new Jira-related envs in silo configmap
  • Documentation and questions have been updated to reflect these changes.

…orters workloads

Port the external-api and worker-importers workloads from
makeplane/commercial-deployments#31 to the plane-enterprise chart as
opt-in services (both disabled by default), and add Jira Server import
tuning envs (JIRA_SERVER_ISSUES_PAGE_SIZE, JIRA_SERVER_ISSUES_PARALLEL_PAGES)
to the silo ConfigMap.

Co-Authored-By: WOZCODE <contact@withwoz.com>
@coderabbitai

coderabbitai Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

The enterprise Helm chart adds optional External API and importer worker deployments, exposes their configuration through values and installer questions, documents the settings, and adds Jira Server importer pagination variables to the Silo ConfigMap.

Changes

Enterprise workload expansion

Layer / File(s) Summary
Configuration contracts and Jira settings
charts/plane-enterprise/Chart.yaml, charts/plane-enterprise/values.yaml, charts/plane-enterprise/questions.yml, charts/plane-enterprise/templates/config-secrets/silo.yaml, charts/plane-enterprise/README.md
Adds configuration and documentation for both optional workloads, adds Jira pagination variables with defaults to the Silo ConfigMap, and bumps the chart version to 2.8.0.
External API workload
charts/plane-enterprise/templates/workloads/external-api.deployment.yaml
Conditionally creates an External API Service and Deployment with configurable replicas, resources, scheduling, secrets, S3 CA support, and HTTP readiness probing.
Importer worker workload
charts/plane-enterprise/templates/workloads/worker-importers.deployment.yaml
Conditionally creates a Celery importer Deployment with configurable resources, secrets, S3 CA support, startup command, service account, and process readiness probing.

Estimated code review effort: 4 (Complex) | ~45 minutes

Sequence Diagram(s)

sequenceDiagram
  participant HelmValues
  participant Kubernetes
  participant ExternalAPIContainer
  HelmValues->>Kubernetes: Render external_api Service and Deployment when enabled
  Kubernetes->>ExternalAPIContainer: Start configured External API container
  ExternalAPIContainer-->>Kubernetes: Return HTTP readiness on port 8000
Loading
sequenceDiagram
  participant HelmValues
  participant Kubernetes
  participant ImporterWorker
  HelmValues->>Kubernetes: Render worker_importers Deployment when enabled
  Kubernetes->>ImporterWorker: Start celery.importer
  ImporterWorker-->>Kubernetes: Report readiness from worker process
Loading

Suggested reviewers: mguptahub, sriramveeraghanta, pratapalakshmi

Poem

A rabbit hops through charts so neat,
New workers march on tiny feet.
Jira pages flow in rows,
External APIs bloom and grow.
Helm lights every workload bright!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: adding the external-api and worker-importers workloads to plane-enterprise.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/plane-enterprise-external-api-worker-importers

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@makeplane

makeplane Bot commented Jul 10, 2026

Copy link
Copy Markdown

Linked to Plane Work Item(s)

This comment was auto-generated by Plane

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.

2 participants