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

adjust Odata web 2.0 expand command #5

Merged
merged 2 commits into from
Mar 5, 2025
Merged

adjust Odata web 2.0 expand command #5

merged 2 commits into from
Mar 5, 2025

Conversation

degricar
Copy link
Contributor

@degricar degricar commented Mar 4, 2025

does not work with HPE currently, tested for gen10 and gen11
e.g.

2025-03-04 11:15:20,850 1 redfish_collector.py:229 ERROR Target node001r-bb563.cc.eu-de-1.cloud.sap: Unable to connect to URL https://10.245.236.4/redfish/v1/Systems?$expand=*
2025-03-04 11:15:20,850 1 redfish_collector.py:299 ERROR Target node001r-bb563.cc.eu-de-1.cloud.sap: No Systems Members found!

@BerndKue
Copy link
Contributor

BerndKue commented Mar 5, 2025

I'm not sure if that works. /1 is not for all vendors working as far as I remember. I think you need to remove the /1.
Latest the Superdome will have an issue with this because it has several Partitions listed under system. But that would be ok to break for the tume being to unblock the other servers.

@degricar
Copy link
Contributor Author

degricar commented Mar 5, 2025

I'm not sure if that works. /1 is not for all vendors working as far as I remember. I think you need to remove the /1. Latest the Superdome will have an issue with this because it has several Partitions listed under system. But that would be ok to break for the tume being to unblock the other servers.

You are right
for the HPE Superdome it is "@odata.id": "/redfish/v1/Systems/Partition0"
only one entry there though, under /Chassis there are more
for Dell "@odata.id": "/redfish/v1/Systems/System.Embedded.1"
But in the meanwhile I found that it works with a dot instead of the asterisk for all vendors, adjusting it accordingly

@degricar degricar changed the title remove Odata web 2.0 expand command adjust Odata web 2.0 expand command Mar 5, 2025
@BerndKue BerndKue merged commit 3bb894f into main Mar 5, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants