tree 569a4996d559ea622881a96a1b0aa7e818ea7a83
parent 36ea1cefb542a70be3e567c4b256e7b103c70b2a
author Ed Tanous <edtanous@google.com> 1632274100 -0700
committer Ed Tanous <ed@tanous.net> 1632934021 +0000

Make services not required

The recent change to NetworkProtocols has now made particular services
"required" to exist in an image.  This corrects it to properly check for
the "process doesn't exist" error code, and bail out early without
setting an internal error.

As pointed out in one of the early reviews, this logic can also be
simplified and moved, so this code also moves the early property filling
code.

This allows deploying systems without IPMI, and have them function
correctly.

Tested:
Loaded in qemu without IPMI present, did not receive 500 on:
curl -vvvv --insecure --user root:0penBmc
"https://192.168.7.2/redfish/v1/Managers/bmc/NetworkProtocol"

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I0a8eb687826d055b4eb43ca53120f39c21934b36