Created attachment 1713064 [details]
Description of problem:
Please remove the "Access Mode" from RHV to CNV VM import wizard, in the "Edit Disk" window, under the "Advanced" section.
Currently "Access Mode" is not active, and displays "RWO" value, that is not honored.
The actual "Access Mode" in the VM import request is set by the source VM in the VM import backend code.
Version-Release number of selected component (if applicable):
This bug is driven from bug 1864577
This looks similar to https://bugzilla.redhat.com/show_bug.cgi?id=1850574
In this case we have a fields that the Importer calculates by it's self (we can't overwrite it), but since we use the vm-wizard for import, we can see it as ab option.
Do you think of a design ( like in https://bugzilla.redhat.com/show_bug.cgi?id=1850574 ) where we can tell the user that this fields are not relevant for the import flow without actually removing them ?
another option is not to use the vm-wizard for import, since it needs only the imported vm name, unlike the many inputs that we use to creating a vm, making most of the wizard irrelevant ?
This is slightly different. It sounds like we could not show the access mode if it doesn’t apply.
at today's meeting we concluded it would be more beneficial to gather info from backend and show the right value which would be resolved by vm-import-operator
This bug is about that Access mode shows a false fixed data "RWO".
Is it possible to target it to 4.6 please?
> Is it possible to target it to 4.6 please?
Sure, moving to 4.6
we also see the "Access Mode" in the review tab, when removing the "Access Mode" input, we also need to remove it from the review tab,
@Matt, maybe remove the network and disks completely from the review tab in the case of imports because until we get the information from the operator https://bugzilla.redhat.com/show_bug.cgi?id=1873779#c4 they will show wrong information ?
I don't think that is necessary to remove it as we try to show the correct information and the review tab is quite useful - especially when you do clean import and you see only import tab and review tab
plus we decided to supply the backend value, and not to remove the access mode input
> plus we decided to supply the backend value, and not to remove the access mode input
Filip hi, this BZ is now on 4.6, do we have an API-call/CRD-status to give us this info in 4.6 ?
Can we deliver this in 4.6 ?
yes we do, merged today: https://github.com/ManageIQ/manageiq-v2v-conversion_host/pull/87