Allon Mureinik has submitted this change and it was merged.
Change subject: core: Print HE's domain name if it's not found
......................................................................
core: Print HE's domain name if it's not found
If the Hosted Engine's domain cannot be found in the system it will be
null, and cannot be logged effectively.
Instead, we should log the name we used to search for it.
This patch fixes the FindBugs warning NP_LOAD_OF_KNOWN_NULL_VALUE on
this line.
Change-Id: I6ea3980419d63e7ac1d85da9be0b3233ea7ca1ba
Signed-off-by: Allon Mureinik <amureini(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/HostedEngineImporter.java
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Tal Nisan: Looks good to me, approved
Allon Mureinik: Verified; Passed CI tests
--
To view, visit
https://gerrit.ovirt.org/43593
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6ea3980419d63e7ac1d85da9be0b3233ea7ca1ba
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org