Moti Asayag has posted comments on this change.

Change subject: webadmin & engine: Add validation for snapshot description 
length
......................................................................


Patch Set 3: (2 inline comments)

....................................................
Commit Message
Line 3: AuthorDate: 2012-11-03 23:48:43 +0200
Line 4: Commit:     Arik Hadas <aha...@redhat.com>
Line 5: CommitDate: 2012-11-05 16:59:19 +0200
Line 6: 
Line 7: webadmin & engine: Add validation for snapshot description length
as Allon commented, please modify to "webadmin, engine:"
(replace & with ,)
Line 8: 
Line 9: Add validation which checks that the length of the snapshot
Line 10: description does not exceed 4000 (GENERAL_MAX_SIZE) characters
Line 11: at the webadmin dialog for creating snapshot


Line 14: snapshot description does not exceed GENERAL_MAX_SIZE
Line 15: for CreateAllSnapshhotsFromVmCommand in the engine
Line 16: 
Line 17: Change-Id: I780eec76b6eb576d28b3650f4d34384594a8044d
Line 18: Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=871029
please replace the url with: http://bugzilla.redhat.com/871029


--
To view, visit http://gerrit.ovirt.org/9011
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I780eec76b6eb576d28b3650f4d34384594a8044d
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: Vered Volansky <vvola...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to