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

inventory plugin is only resturning 50 results per group #547

Closed
tallyoh opened this issue Sep 4, 2023 · 6 comments · Fixed by #575
Closed

inventory plugin is only resturning 50 results per group #547

tallyoh opened this issue Sep 4, 2023 · 6 comments · Fixed by #575
Labels
area/ome ome modules type/bug Something isn't working

Comments

@tallyoh
Copy link

tallyoh commented Sep 4, 2023

Hello,

I am using the ome_inventory plugin for ansible and I am only able to receive 50 results, and only if I specify a group. Without a group created in ome, I actually dont get any results at all. Has anyone encountered this issue before? Any solutiuon?

Thanks in advance.

@tallyoh tallyoh added the type/question Further information is requested label Sep 4, 2023
@anupamaloke
Copy link
Collaborator

@tallyoh, could you please let us know the version of the openmanage collection that you are using?

@anupamaloke anupamaloke added the area/ome ome modules label Sep 25, 2023
@sachin-apa sachin-apa added type/bug Something isn't working and removed type/question Further information is requested labels Sep 26, 2023
@sachin-apa
Copy link
Collaborator

@tallyoh I have a fix for this attached please replace and let me know if it is good, please feel free to submit a PR for the fix.
ome_inventory.zip

@DellDavidMcKay
Copy link

I gave this a try, it did not return anymore results than the old version. I still get the top 50. I have over 400 blades in OME.

@DellDavidMcKay
Copy link

DellDavidMcKay commented Oct 10, 2023

If I access the direct RESTful API which returns the same result as the playbook, it will work if I specify $skip=0&$top=2000
If I do not set skip=0, I only get the top 50 results from OME. This is under data pagination in the OME RESTful API guide

I did have to change queries to var.json instead of var.device_info but otherwise the direct rest call was fine

@sachin-apa
Copy link
Collaborator

sachin-apa commented Nov 2, 2023

@DellDavidMcKay Thanks for checking.. i tried here and it seems to be working good, we are paginating the values and fetching all the devices.
Anyways i will create a PR with the changes.

@tallyoh
Copy link
Author

tallyoh commented Nov 29, 2023

Thank you for fixing this. I can confirm that the recently released 8.5.0 version does indeed fix the inventory plugin. I am now able to get all results (not just the first 50). Thanks again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ome ome modules type/bug Something isn't working
Projects
4 participants