Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

v2v - Enhanced Error Reporting in UI from virt-v2v log #734

Closed
kedark3 opened this issue Oct 25, 2018 · 7 comments
Closed

v2v - Enhanced Error Reporting in UI from virt-v2v log #734

kedark3 opened this issue Oct 25, 2018 · 7 comments

Comments

@kedark3
Copy link

kedark3 commented Oct 25, 2018

This is a RFE BZ, Fabien and Brett will need to collaborate with UI team to discuss this.
https://bugzilla.redhat.com/show_bug.cgi?id=1595365

@bthurber
Copy link

@vconzola do you mind reviewing? I think this may require some discussion to see what we can expose in the UI.

@vconzola
Copy link

@bthurber Yeah, I think this will require a discussion. We can provide more detailed error information in the UI, but we can't generate that information. It will have to come from the backend.

@mturley
Copy link
Contributor

mturley commented Mar 18, 2019

After discussion with @fdupont-redhat, he is going to make a PR to add this enhanced error message to the task objects at task.options.progress.states[task.options.progress.current_state].message, which we'll need to expose in the UI. @vconzola I'm thinking we can just add a "Message:" line to the existing popover on a migration task, here:

Screenshot 2019-03-18 12 52 48

Does that work for you?

@mturley mturley self-assigned this Mar 18, 2019
@mturley
Copy link
Contributor

mturley commented Mar 21, 2019

Note to self: this will depend on ManageIQ/manageiq-content#514.
We decided to call the new field "Status Detail" in the popover, and it should have space above and below it.

@ghost
Copy link

ghost commented Mar 22, 2019

Backend and automate PRs are merged. Have fun exposing the message in the UI :)

@mturley
Copy link
Contributor

mturley commented Mar 22, 2019

Thanks @fdupont-redhat! This is near the top of my list, I'll let you know when I have a PR ready.

@mturley
Copy link
Contributor

mturley commented Mar 25, 2019

@fdupont-redhat here's the PR for this one: #918

mzazrivec added a commit that referenced this issue Mar 26, 2019
[#734] Add 'Status Detail' enhanced error message to info popover on a migration task
simaishi pushed a commit that referenced this issue Apr 5, 2019
[#734] Add 'Status Detail' enhanced error message to info popover on a migration task

(cherry picked from commit 9db5954)

https://bugzilla.redhat.com/show_bug.cgi?id=1693747
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants