Skip to content

[chores] Update Celery beat schedule logic for user deactivation  #587

@pandafy

Description

@pandafy

Context:

  • The user expiration logic will change in openwisp-users#499:
    • openwisp-users introduces a periodic task that deactivates users with a past expiration date and sends notification emails (idempotent, timezone-aware, configurable reminders, etc).
  • openwisp-radius#706 will remove the local deactivation logic in favor of unified handling in openwisp-users.

Current logic in docker-openwisp:

  • The Celery beat schedule for deactivating users is only enabled and executed when openwisp-radius is present/enabled. This check will become obsolete since the expiration and deactivation logic will fully reside in openwisp-users.

Required changes:

  • Update how the Celery beat job for user deactivation is scheduled:
    • The periodic task should not depend on openwisp-radius being enabled.
    • The task for user deactivation (as per new openwisp-users implementation) should always be scheduled if openwisp-users is present.
  • Review orchestration, template logic, and docs as needed to ensure the correct and always-on scheduling of this job.
  • Remove any outdated conditions/checks that couple task scheduling to openwisp-radius.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No fields configured for Task.

    Projects

    Status
    To do
    Status
    To do

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions