Arik Hadas has posted comments on this change.

Change subject: core: migration logs
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.ovirt.org/#/c/28928/1/backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties
File 
backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties:

Line 241: USER_SUSPEND_VM_OK=VM ${VmName} on Host ${VdsName} is suspended.
Line 242: VM_FAILURE=VM ${VmName} cannot be found on Host ${VdsName}.
Line 243: VM_MIGRATION_ABORT=Migration failed: ${MigrationError} (VM: 
${VmName}, Source: ${VdsName}, Destination: ${DestinationVdsName}).
Line 244: VM_MIGRATION_DONE=Migration completed (VM: ${VmName}, Source: 
${VdsName}, Destination: ${DestinationVdsName}, Duration: ${Duration} sec).
Line 245: VM_MIGRATION_FAILED=Migration failed (VM: ${VmName}, Source: 
${VdsName}).
> why no migration error here?
the plan was that for regular migrations (i.e not to specific server) we'll 
have the VM_MIGRATION_ABORT log for each attempt that failed (which includes 
the error) and when we reach the maximum number of tries we'll also have this 
one to notify that the whole migration failed
Line 246: VM_MIGRATION_TO_SERVER_FAILED=Migration failed${DueToMigrationError} 
(VM: ${VmName}, Source: ${VdsName}, Destination: ${DestinationVdsName}).
Line 247: VM_MIGRATION_FAILED_NO_VDS_TO_RUN_ON=Migration failed, No available 
host found (VM: ${VmName}, Source: ${VdsName}).
Line 248: VM_MIGRATION_FAILED_DURING_MOVE_TO_MAINTENANCE=Migration 
failed${DueToMigrationError} while Host is in 'preparing for maintenance' 
state.\n  Consider manual intervention\: stopping/migrating Vms as Host's state 
will not\n  turn to maintenance while VMs are still running on it.(VM: 
${VmName}, Source: ${VdsName}, Destination: ${DestinationVdsName}).
Line 249: VM_SET_TO_UNKNOWN_STATUS=VM ${VmName} was set to the Unknown status.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I074836f713cc708c9b680bb2d6b3e99af17f66c3
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <[email protected]>
Gerrit-Reviewer: Arik Hadas <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to