-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Improve event processing timings by eliminating -partial.json
event data file writes
#11557
Comments
I'm putting blocked label on, because I would like to hold off until we get a disposition on ansible/ansible-runner#957 |
@AlanCoding Can this move out of Blocked now? |
Yes, agreed, moved. |
This should be a performance improvement on execution nodes, similar in scope to the performance improvement we get for controller nodes from #11356 |
@jbradberry @rebeccahhh Do you have capacity to handle this in this iteration? |
@ikhan2010 probably |
I do |
@rebeccahhh thanks for stepping up. Kindly let me know how much effort is this. |
I am on PTO for a week so @jbradberry if you want to get progress on this don't wait for me (no pressure, I'll be back after PyCon). |
I'm going to put this in Unplanned for now (because planning is starting and it will be at home with other perf/scale issues). Please put it back into In Progress if you pick it back up again @rebeccahhh |
I put a draft PR up last week but I am working on other things so this is kind of on my back burner: ansible/ansible-runner#1072 |
ISSUE TYPE
SUMMARY
Re-do the pull request here:
ansible/ansible-runner#269
Make all tests in ansible-runner pass, verify functionality with AWX, and hand over to be performance tested.
The text was updated successfully, but these errors were encountered: