----- Original Message -----
From: "Greg Sheremeta" <gshereme(a)redhat.com>
To: "Einav Cohen" <ecohen(a)redhat.com>
Cc: "engine-devel" <engine-devel(a)ovirt.org>, "users"
<users(a)ovirt.org>
Sent: Tuesday, January 14, 2014 12:35:53 PM
Subject: Re: [Engine-devel] Showing stacktrace info in GUI error dialog
----- Original Message -----
> From: "Einav Cohen" <ecohen(a)redhat.com>
> To: "engine-devel" <engine-devel(a)ovirt.org>, "users"
<users(a)ovirt.org>,
> "Martin Perina" <mperina(a)redhat.com>
> Sent: Tuesday, January 14, 2014 11:23:34 AM
> Subject: [Engine-devel] Showing stacktrace info in GUI error dialog
>
> Hi, this is about patch [1] - showing server-side-exception
> stacktrace info in GUI error dialog.
>
> a couple of notes about this patch:
>
> (1) I find it a little strange that server-side exceptions
> are not logged in the server side and are / will be displayed
> only on the client-side - this needs to be fixed first.
>
+1.
> (2) once everything is logged on the server side, we can
> debate whether it makes sense to put the full exception
> stack-trace *also* in the GUI error pop-up.
> I think that it is not completely necessary (since if there
> is a server side exception, the user is likely to go to the
> server side log anyway to get the full picture / context), but
> if it is (and I'd like to hear your opinion about that) - we
> shouldn't show the entire exception in the dialog once the dialog
> is displayed - that's too much information to absorb.
> We should show whatever we are showing today, and also something
> like a "More Details" collapsible section (collapsed by default)
> that, when expanded, displays the full exception details.
>
> comments are welcome.
I personally appreciate web apps that allow me to see an error (or even
expose an entire log file), but do so gracefully -- hide it under a option,
That should read "under a menu option"
use a "More Details" option like you mention, etc.
I think saving a person the step of ssh'ing to a server and digging for a log
file is a nice-to-have.
Greg
>
> ----
> Thanks,
> Einav
>
> [1]
http://gerrit.ovirt.org/#/c/23096/
> _______________________________________________
> Engine-devel mailing list
> Engine-devel(a)ovirt.org
>
http://lists.ovirt.org/mailman/listinfo/engine-devel
>
_______________________________________________
Engine-devel mailing list
Engine-devel(a)ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-devel