AAP-58539 Move to dispatcherd#16209
Conversation
|
Looking like it's making good progress. Next I would like to try to focus on the decorator transition awx/awx/main/scheduler/tasks.py Line 10 in d7eb714 Should be something that can be replaced now with the |
04536cf to
0bc3434
Compare
|
from checks The |
|
#16209 (comment) I think this was coming from the old heartbeat function and is removed now. |
d4e8e8c to
814475b
Compare
this feels stale because that import no longer exists |
|
If you have problems with the linters you might need to rebase to get #16232 also likely related to the code conflicts. |
* started removing feature flags and adjusting logic * WIP
* moved to dispatcher decorator * updated as many as I could find
* remove any code that is not used by the call back receiver
* added back get_auto_max_workers into common utils * formatting edits
* moved status out and deleted control as no longer needed * removed hazmat
* removed unused imports * adjusted test import to pull correct method
* fixed imports and addressed clusternode heartbeat test * took a chainsaw to task.py as well
* added back hazmat per @AlanCoding feedback around config * removed baseworker completely and refactored it into the callback worker
cde2e73 to
bb8ecc5
Compare
|
Because the publish.py module was removed, it looks like |
* added back some code that was lost in the merge conflict * remove dispatcher mock publish setting
* fixed the call to cluster_node_heartbeat missing binder * formatting/linter fixes
* add back more missing things
|
Next up, we see in server logs, via artifacts of the test run: The pid method was removed, and I suggest just putting in |
* on a secret hunt to find the hidden attribute error in the server logs
* updated the operator timeout to near healthy run time
* Fully remove amd migrate away from celery scheduler * Also put in ttry/except for receptor config on start up for debugging
|
Blocked by, and is follow-on stuff to ansible/awx#16209 Will have its hashes ruffled up by #915 But that's just normal merge order of operations. Otherwise I think this looks to be in good shape.


SUMMARY
Removing the dispatcher feature flag as well as migrating anything that might be needed to new dispatcherd library.
PR is still a WIP and not finished. If you see anything or think of anything please note it.
ISSUE TYPE
COMPONENT NAME
ADDITIONAL INFORMATION