-
Notifications
You must be signed in to change notification settings - Fork 81
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
add return_fields parmeter to the configuration_item_info module #208
add return_fields parmeter to the configuration_item_info module #208
Conversation
Build succeeded. ✔️ build-ansible-collection SUCCESS in 3m 01s |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please add tests and create a changelog fragment. See this fragment as an example?
62a799f
to
36f0eca
Compare
* Allow user to specify return fields for the specifed configuration item info Signed-off-by: Toni Moreno Giménez <[email protected]> Signed-off-by: Abhijeet Kasurde <[email protected]>
36f0eca
to
ce1aacf
Compare
/lgtm |
This commit adds integration tests for PR#208.
This commit adds integration tests for PR#208.
This commit adds integration tests for PR #208.
SUMMARY
add return_fields parmeter to the configuration_item_info module, needed for choose only needed information It will speed considerably the response. and the returned data to process. It will be needed to build data maintentance proceses with heavily loaded classes.
Implements #100
ISSUE TYPE
COMPONENT NAME
configuration_item_info
ADDITIONAL INFORMATION
Consider a data process to update info only some ci's with operational_status="opeational" , you need first query the complete class and after do something in a loop with the sys_id of each CI.
Time taken to execute this playbook with "return_fields" : 4.405 seconds
Time taken to execute this playbook without "return_fields" : 11.94 seconds ( only with 6 ci's !!!)
we can guess what will happend when exected this query on a class with 100K registers.
result with return_fields
result without return_fields