Skip to content

Commit

Permalink
Don't install python3-eventlet for worker
Browse files Browse the repository at this point in the history
It was there so that we could switch between gevent & eventlet
for a greenlets library (for concurrency pool), but never actually tried it.
We've been using gevent for several months now.
  • Loading branch information
jpopelka committed Dec 8, 2022
1 parent bcf2c5d commit 813f7e9
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
4 changes: 1 addition & 3 deletions files/install-deps-worker.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,7 @@
- postgresql # pg_dump
- python3-boto3 # AWS (S3)
- python3-fasjson-client
# remove if we decide not to use concurrency
- python3-eventlet
- python3-gevent
- python3-gevent # concurrency pool, see run_worker.sh
# v6 = bodhi-client, v5 = python3-bodhi{,-client}
- bodhi-client
# needed when installing from koji - there are no GPG-signed packages
Expand Down
2 changes: 1 addition & 1 deletion files/run_worker.sh
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ elif [[ "${CELERY_COMMAND}" == "worker" ]]; then
CONCURRENCY="${CONCURRENCY:-$DEFAULT_CONCURRENCY}"
export CONCURRENCY

# Options: prefork | eventlet | gevent | solo
# Options: solo | gevent
# https://www.distributedpython.com/2018/10/26/celery-execution-pool/
DEFAULT_POOL="solo"
DEFAULT_CONCURRENCY_POOL="gevent"
Expand Down

0 comments on commit 813f7e9

Please sign in to comment.