Change in ovirt-engine[master]: core: Print HE's domain name if it's not found

amureini at redhat.com amureini at redhat.com
Tue Jul 14 12:39:59 UTC 2015


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 at 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 at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list