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