Bug 1873549

Verify this bug with the official build:
virt-viewer-9.0-4.el8.x86_64

Steps:

Steps:
RHV env setup:  Prepare ISO images in ISO storage domains.
iso_domain: 
RHEL-8.2.0-20200404.0-BaseOS-x86_64-boot.iso


1. Open the VM in RHV and check the "Change CD" sbu-menu in File menu of remote-viewer

Result:
1.1 All ISO images in ISO domain are listed(Here is: RHEL-8.2.0-20200404.0-BaseOS-x86_64-boot.iso)
1.2 Select on ISO image and check it, then the ISO image is attached to the vm.
1.3 Uncheck the current ISO image, then the image is detached from the vm.
1.4 Add a new iso image in ISO_DOMAIN of RHV, then click the "Refresh" button, the new image can be listed successfully and can be attached to vm and detached from vm successfully.
1.5 Remove all iso images from ISO_DOMAIN Of RHV, then click the "Refresh" button, one window pops and shows "Failed to fetch CD names", then click 'Close' button, and then click "Close" button on the main window, exits from "Change CD" window.(Details please see Screenshot-1 and Screensho-2).
It also happens when there is no iso images attached to ISO_DOMAIN, then directly check the "Change CD" sbu-menu in File menu of remote-viewer.


2. Test with Ovirt URI scheme to connect to RHV VMs:

$ remote-viewer --ovirt-ca-file=ca.crt ovirt://rhv.example.com/esx6.7-rhel8.2-x86_64

Result:
2.1 We can connect to vm on rhv successfully after input correct username and password.
2.2 Click the "Change CD" menu item, get the same result with step-1.

@Etrunko, the basic function of 'Change CD' from ISO_DOMAIN works well, but please pay attention to the test result of Step1.5, I think there is no need to have the dual windows to show "Failed to fetch CD names", I suggest to file a new bug to improve it, thanks.

Sorry for ignore this test scenario yesterday.

Created attachment 1713754 [details]
Screenshot for the dual windows

Created attachment 1713757 [details]
Screenshot for the main window

(In reply to zhoujunqin from comment #11)
> 
> @Etrunko, the basic function of 'Change CD' from ISO_DOMAIN works well, but
> please pay attention to the test result of Step1.5, I think there is no need
> to have the dual windows to show "Failed to fetch CD names", I suggest to
> file a new bug to improve it, thanks.
> 

Noted, thank you.

(In reply to Eduardo Lima (Etrunko) from comment #14)
> (In reply to zhoujunqin from comment #11)
> > 
> > @Etrunko, the basic function of 'Change CD' from ISO_DOMAIN works well, but
> > please pay attention to the test result of Step1.5, I think there is no need
> > to have the dual windows to show "Failed to fetch CD names", I suggest to
> > file a new bug to improve it, thanks.
> > 
> 
> Noted, thank you.

Thanks for your reply, Etrunko.
I filed a new bug 1876290 to track the Step1.5 issue, so I move this bug from ON_QA to VERIFIED status, thanks.

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (virt-viewer bug fix and enhancement update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2020:4788