augur icon indicating copy to clipboard operation
augur copied to clipboard

Use configured Celery concurrency values instead of hardcoded values (#3403)

Open iGufrankhan opened this issue 1 month ago • 3 comments

Fixes #3403

Description

This PR replaces the hardcoded Celery worker concurrency values in
augur/application/cli/tasks.py with the configurable values defined in the Celery section of the Augur config.

The following settings are now read dynamically through AugurConfig:

  • core_worker_count
  • secondary_worker_count

The scheduling worker intentionally remains at --concurrency=1, since only one scheduler should run at a time.

Notes for Reviewers

  • No functional behavior is changed besides making concurrency configuration-driven.

Signed commits

  • [x] Yes, I signed my commits.

iGufrankhan avatar Nov 15 '25 18:11 iGufrankhan

The process names is separate. this PR updates the CLI to use the values from the config (augur.json or DB) for concurrency values when starting aufur from CLI

MoralCode avatar Dec 09 '25 22:12 MoralCode

The process names is separate. this PR updates the CLI to use the values from the config (augur.json or DB) for concurrency values when starting aufur from CLI

ok ... with that and the redundant tag I'm unsure what to do with this one ...

sgoggins avatar Dec 09 '25 22:12 sgoggins

The process names is separate. this PR updates the CLI to use the values from the config (augur.json or DB) for concurrency values when starting aufur from CLI

ok ... with that and the redundant tag I'm unsure what to do with this one ...

Redundant tag just serves as a flag to say "hey, there are duplicates of this thing" (i.e. if two newcomers submit PRs for the same issue). it just means we have to make sure we pick the PR with the best implementation and, once it merges, the other one gets automatically closed for being a duplicate.

MoralCode avatar Dec 11 '25 14:12 MoralCode