Martin Peřina has posted comments on this change.

Change subject: core: webadmin: Fix NPE in VdsArchitectureHelper
......................................................................


Patch Set 3:

AFAIK restapi changes are not needed, because this query is called only from UI 
using these steps:

1. Click on New Host
2. Fill in values, try to test Fence Agent

The reason for this code is that we try to test fence agent of the host which 
is not yet saved in db, so we need to provide required attributes as a 
parameters of the query.

-- 
To view, visit https://gerrit.ovirt.org/38963
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I361c930d01a17da8bc1085286fdb845af0b5c237
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Peřina <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Martin Peřina <[email protected]>
Gerrit-Reviewer: Ori Liel <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to