Change in ovirt-node-plugin-vdsm[master]: engine_page: Add validation for None type

dougsland at redhat.com dougsland at redhat.com
Tue Nov 10 16:35:38 UTC 2015


Douglas Schilling Landgraf has submitted this change and it was merged.

Change subject: engine_page: Add validation for None type
......................................................................


engine_page: Add validation for None type

Currently there is no validation which might break when
users try to set the root password using TUI. This
patch adds the validation.

Change-Id: Ieb20d1563fa9d13f1d266942a8fb8486da1882ca
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1279329
Signed-off-by: Douglas Schilling Landgraf <dougsland at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/48319
Continuous-Integration: Jenkins CI
Reviewed-by: Fabian Deutsch <fabiand at redhat.com>
---
M src/engine_page.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Douglas Schilling Landgraf: Verified; Looks good to me, approved; Passed CI tests
  Fabian Deutsch: Looks good to me, approved
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ieb20d1563fa9d13f1d266942a8fb8486da1882ca
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-node-plugin-vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Anatoly Litovsky <tlitovsk at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list