-
Notifications
You must be signed in to change notification settings - Fork 109
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Launch Multiple Worker Instances using CeleryExecutor #4542
Comments
btylerburton
moved this from 📔 Product Backlog
to 🏗 In Progress [8]
in data.gov team board
Dec 5, 2023
Some learnings...
|
Given what we've learned above, I'm going to close this ticket until we're focused on optimization. |
github-project-automation
bot
moved this from 🏗 In Progress [8]
to ✔ Done
in data.gov team board
Dec 12, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
User Story
In order to achieve true horizontal scaling, datagovteam wants to be able to launch multiple instances of the airflow-worker application and see them pick up queued work.
Acceptance Criteria
[ACs should be clearly demoable/verifiable whenever possible. Try specifying them using BDD.]
cf push --vars-file my_vars_file
AND I have configured the datagov-harvester manifest to launch multiple instances of the worker application
WHEN I look at the "Cluster Activity" tab in Airflow UI
THEN I will see that it is queuing up new work and that the new queued work is getting picked up and run by a worker node
Background
Currently, launching more than one instance of the airflow-worker application causes the worker instances not to pick up work, whereas a single instance has no issues.
Considering the Celery documentation, this may be mitigated by launching the worker instances with a hostname:
Determine if we can launch the worker instances using the
.profile
and supply them with a unique start command, similar toairflow celery worker -n worker-{CF_INSTANCE_INDEX}
using CF Env varsSecurity Considerations (required)
[Any security concerns that might be implicated in the change. "None" is OK, just be explicit here!]
Sketch
.profile
and supply them with a unique start commandThe text was updated successfully, but these errors were encountered: