Comment 12 for bug 1318317

Right, that is because "you" (slash the default) have specified in the openmpi configuration file (/etc/default/openmpi) that IPMI_SI is needed on your system. Thus, when it fails to load successfully, it is still flagged as:

# no bits set = no errors
# bit 0 set = minimum modules aren't loaded
# bit 1 set = requested feature module isn't loaded
# bit 2 set = /dev/ipmi0 (or /dev/imb if using that instead) doesn't exist
# bit 3 set = /dev/watchdog doesn't exist
# bit 4 set = lockfile doesn't exist
# bit 5 set = modules are loaded even when asked to be unloaded

bit 0 being set (RETVAL | 1).

I guess we could change the logic in a different way such that bit 1 should be set in this case, rather than bit 0. But that would redefine what the "minimum" modules are in OpenIPMI, which per the init-script are ipmi_msghandler and ipmi_{si,smb} if those options are set in the config file.

So I think what I provided is probably the best combination for the configuration by default -- Robie did point out that my patch has a logical issue in that the last added line is really a no-op. So I'll spin up a new version and push it to my PPA. If you can test that one as well, just to be sure it's functionally the same, then we'll work on getting that into -proposed (once I learn how to do that :)

I'll provide another update once the new packages are present.

Thanks, Jeremy!