Moti Asayag has posted comments on this change.

Change subject: engine: Add error message for ifup failure
......................................................................


Patch Set 2:

In reply to:
> Mike Kolesnik         Sep 11 10:19 AM
>
> Patch Set 1: Code-Review+2
> 
> On a general note, perhaps it's worthwhile to write something like "Please 
> check the server log for more details" in the UI when this happens, since the 
> log is more detailed in most errors than the static translation that we 
> have.. Maybe there's even a way to print the correlation ID to look for

IMO if an error message is retrieved from VDSM it should be added as part of 
the text of the audit log as the reason, e.g. in brackets after the resolved 
audit log message.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I07adde3368c31319bed48839d089ba3ac48cd424
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkole...@redhat.com>
Gerrit-Reviewer: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to