ipmitool default interface
I have seen several requests for help on this list that could have been avoided if the BMC internal ipmitool used dbus as its default interface.
I do not think we are quite ready to upstream this commit or the actual interface, since the dbus interface will be changing as part of the refactor of ipmid's architecture. But for now, it makes sense to set the default interface as dbus.
After I have finished working on the refactor, I will push the changes upstream and then we should be able to just run with a standard cross-compiled build of ipmitool.
--Vernon
+1
-Ed
Quoted 14 lines
I have seen several requests for help on this list that could have been avoided if the BMC internal ipmitool used dbus as its default interface.
I do not think we are quite ready to upstream this commit or the actual interface, since the dbus interface will be changing as part of the refactor of ipmid's architecture. But for now, it makes sense to set the default interface as dbus.
After I have finished working on the refactor, I will push the changes upstream and then we should be able to just run with a standard cross- compiled build of ipmitool.
--Vernon
Quoted 3 lines · Vernon Mauery
After I have finished working on the refactor, I will push the changes upstream and then we should be able to just run with a standard cross-compiled build of ipmitool.
Great! Just make sure the code meets
https://github.com/ipmitool/ipmitool/wiki/Coding-Standards
WBR, Alexander.
P.S. I haven't forgotten about your UDP packet size pull request there, just need to thoroughly review it and that takes time.
Hi Vernon,
Quoted 2 lines
I have seen several requests for help on this list that could have been avoided if the BMC internal ipmitool used dbus as its default interface.
I do not think we are quite ready to upstream this commit or the actual interface, since the dbus interface will be changing as part of the refactor of ipmid's architecture. But for now, it makes sense to set the default interface as dbus.
OK, makes sense. I've merged your change for this and pushed to the openbmc ipmitool repo.
Cheers,
Jeremy