Skip to content

feat(api): add reconcile_rsource Celery task skeleton - #280

Open
eLRuLL wants to merge 1 commit into
adr001/resource-dual-writefrom
adr001/celery-reconciliation
Open

feat(api): add reconcile_rsource Celery task skeleton#280
eLRuLL wants to merge 1 commit into
adr001/resource-dual-writefrom
adr001/celery-reconciliation

Conversation

@eLRuLL

@eLRuLL eLRuLL commented Apr 29, 2026

Copy link
Copy Markdown
Member

Description

Adds a skeleton Celery reconciler for the resource control-plane.

  • New task core.tasks.reconcile_resource(resource_id) (name="core.tasks.reconcile_resource").
  • Gated by RESOURCE_RECONCILER_ENABLED (default False). When disabled, returns immediately without touching the database.
  • When enabled: loads Resource, exits early if missing or terminal phase; otherwise no-op (no Kubernetes, no state writes yet).
  • Settings + .env.example document the flag.
  • tests/core/test_reconcile_resource.py: disabled path, terminal phase, idempotent double-run on non-terminal, missing id.

Issue

  • _N/A

Checklist before requesting a review

  • I have performed a self-review of my code.
  • My code follows the style guidelines of this project.
  • I have made corresponding changes to the documentation. (.env.example only)
  • New and existing tests pass locally with my changes. (run python manage.py test tests.core.test_reconcile_resource)
  • If this change is a core feature, I have added thorough tests.
  • If this change affects or depends on the behavior of other estela repositories, I have created pull requests with the relevant changes in the affected repositories. (not applicable)
  • I understand that my pull request may be closed if it becomes obvious or I did not perform all of the steps above.

@eLRuLL
eLRuLL requested review from erick-GeGe and joaquingx April 29, 2026 16:51
@github-actions

Copy link
Copy Markdown

Docs preview

New docs deployment: [72b09d4f0c917c506425f9a1bff82ad73ed9c62a]!
Check out the docs preview here 👈

@joaquingx joaquingx left a comment

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.

Schedule of the task isn't on the PR.

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