Omer Frenkel has posted comments on this change.

Change subject: core: Add warning when running VM as stateless with unsnapable 
disks (#834891)
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File 
backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties
Line 63: USER_RUN_VM_AS_STATELESS_WITH_DISKS_NOT_ALLOWING_SNAPSHOT=User trying 
to run VM ${VmName} as stateless with one or more of disks that do not allow 
snapshots. 
sounds better to me:
"VM ${VmName} was run as stateless with one or more of disks that do not allow 
snapshots."
if want to add the user you should add it at the end as in other commands:
"VM ${VmName} was run as stateless with one or more of disks that do not allow 
snapshots (User:${UserName})."

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I44eec85962a5edf69e5977894de2c3518d4e1dc0
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tni...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Asaf Shakarchi <a...@redhat.com>
Gerrit-Reviewer: Ayal Baron <aba...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Liron Aravot <lara...@redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com>
Gerrit-Reviewer: Michael Kublin <mkub...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to