-
Notifications
You must be signed in to change notification settings - Fork 75
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
Proposal: Introduce Custom Field for BMC Power Status in Netbox-sync #379
Comments
Hi, This sounds reasonable, will try to change this behavior as described. |
Hi @sh0x, really sorry, it's almost a year. But I finally pushed a change which implements exactly this behaviour. Would be happy if you could try the |
It works great, thanks for implementing it! |
Are you actually working at VMware? |
Yes I'm still there :) |
I just read that more then half of the employees have been booted sincethe takeover, which is crazy. So you're telling me that VMware itself is using this project? |
I'm using check_redfish and netbox-sync to sync redfish data for ~1,500 servers we manage (Dell, HP, Lenovo), works great! |
Wow, great to hear. Didn't anticipate this when I started this project 😅. And these days, both projects are just a hobby of mine as I shifted my work focus. |
I've heard from multiple customers/friends using your project in their companies; it's a really valuable addition to netbox. I'm personally using it for my HomeLab and helps me to keep an overview (let's say, the lab "it escalated quickly"...) But also interesting to hear that VMware/Broadcom is using it too. Definitely an achievement - but honestly well deserved! |
Thank you for the kind words. It's great to here that people appreciate this work. |
Issue
Currently, in Netbox-sync, the BMC power status is synchronized with the management status in Netbox. While Netbox's management status is designed for inventory provisioning purposes (Offline, Active, Planned, Staged, Failed, Inventory), it's being utilized by netbox-sync to represent the server power status.
https://github.com/bb-Ricardo/netbox-sync/blob/main/module/sources/check_redfish/import_inventory.py#L217
Proposal
I suggest implementing a custom field specifically for the BMC power status instead of repurposing the device management status field. This separation would maintain clarity and prevent the BMC power status from conflating with device management statuses.
Reasoning
By introducing a dedicated field for BMC power status, we ensure that both the server's operational state and its management status remain distinct and accurately represented within Netbox. This separation enhances clarity and avoids potential confusion or misinterpretation of device statuses.
Benefits
Conclusion
By implementing a dedicated field for BMC power status, we can enhance the clarity and maintain the integrity of device management within Netbox, ensuring a more intuitive and efficient experience for users.
The text was updated successfully, but these errors were encountered: