Tal Nisan has submitted this change and it was merged. Change subject: engine: NPE in GLUSTERFSStorageHelper ...................................................................... engine: NPE in GLUSTERFSStorageHelper Fixing the ConnectHostToStoragePoolServersParameters constructor, to include 'StoragePool'. In the flow of refreshing storage connections and pool metada due to Vdsm reports, the ConnectHostToStoragePoolServersParameters constructor was used without 'StoragePool'. Several classes like CINDERStorageHelper and GLUSTERFSStorageHelper are expecting the parameter to have a StoragePool populated. Change-Id: I3b80bcc9606677912e88dc265d36c8c9f8ba8138 Bug-Url: https://bugzilla.redhat.com/1277570 Signed-off-by: Fred Rolland <frolland@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/ConnectHostToStoragePoolServersParameters.java 2 files changed, 9 insertions(+), 3 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Freddy Rolland: Verified -- To view, visit https://gerrit.ovirt.org/48378 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3b80bcc9606677912e88dc265d36c8c9f8ba8138 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Freddy Rolland <frolland@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: automation@ovirt.org