Skip to content
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

Open
AlanCoding opened this issue Jan 18, 2022 · 11 comments
Assignees

Comments

@AlanCoding
Copy link
Member

ISSUE TYPE
  • Feature Idea
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.

@AlanCoding
Copy link
Member Author

I'm putting blocked label on, because I would like to hold off until we get a disposition on ansible/ansible-runner#957

@shanemcd
Copy link
Member

shanemcd commented Feb 7, 2022

@AlanCoding Can this move out of Blocked now?

@AlanCoding
Copy link
Member Author

Yes, agreed, moved.

@AlanCoding
Copy link
Member Author

This should be a performance improvement on execution nodes, similar in scope to the performance improvement we get for controller nodes from #11356

@ikhan2010
Copy link

@jbradberry @rebeccahhh Do you have capacity to handle this in this iteration?

@jbradberry
Copy link
Contributor

@ikhan2010 probably

@rebeccahhh
Copy link
Member

I do

@rebeccahhh rebeccahhh self-assigned this Apr 20, 2022
@ikhan2010
Copy link

@rebeccahhh thanks for stepping up. Kindly let me know how much effort is this.

@rebeccahhh
Copy link
Member

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).

@AlanCoding
Copy link
Member Author

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

@rebeccahhh
Copy link
Member

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants