Uploaded image for project: 'OpenShift Virtualization'
  1. OpenShift Virtualization
  2. CNV-30572

The checkbox shouldn't be automatically checked when creating an SSH

    • Icon: Bug Bug
    • Resolution: Done-Errata
    • Icon: Undefined Undefined
    • CNV v4.14.0
    • None
    • CNV User Interface
    • None
    • False
    • Hide

      None

      Show
      None
    • False
    • ---
    • ---
    • None

      When the user creates a VirtualMachine from a template, and customizes the Authorized SSH key, currently the checkbox is automatically checked.

      To fix this, the check box should be unchecked by default under the "Use existing" and "Add new" radio buttons.

      Current:

      How it should look:

            [CNV-30572] The checkbox shouldn't be automatically checked when creating an SSH

            Errata Tool added a comment -

            Since the problem described in this issue should be resolved in a recent advisory, it has been closed.

            For information on the advisory (Important: OpenShift Virtualization 4.14.0 Images security and bug fix 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/RHSA-2023:6817

            Errata Tool added a comment - Since the problem described in this issue should be resolved in a recent advisory, it has been closed. For information on the advisory (Important: OpenShift Virtualization 4.14.0 Images security and bug fix 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/RHSA-2023:6817

            CPaaS Service Account mentioned this issue in a merge request of cpaas-midstream / openshift-virtualization / kubevirt-plugin on branch cnv-4.15-rhel-9_upstream_ad48df6ba7e8b491fa7e2d1b90ee7d6d:

            Updated US source to: f36902f Merge pull request #1405 from avivtur/fix-centos-vm-name

            GitLab CEE Bot added a comment - CPaaS Service Account mentioned this issue in a merge request of cpaas-midstream / openshift-virtualization / kubevirt-plugin on branch cnv-4.15-rhel-9_ upstream _ad48df6ba7e8b491fa7e2d1b90ee7d6d : Updated US source to: f36902f Merge pull request #1405 from avivtur/fix-centos-vm-name

            CPaaS Service Account mentioned this issue in a merge request of cpaas-midstream / openshift-virtualization / kubevirt-plugin on branch cnv-4.14-rhel-9_upstream_ad48df6ba7e8b491fa7e2d1b90ee7d6d:

            Updated US source to: f36902f Merge pull request #1405 from avivtur/fix-centos-vm-name

            GitLab CEE Bot added a comment - CPaaS Service Account mentioned this issue in a merge request of cpaas-midstream / openshift-virtualization / kubevirt-plugin on branch cnv-4.14-rhel-9_ upstream _ad48df6ba7e8b491fa7e2d1b90ee7d6d : Updated US source to: f36902f Merge pull request #1405 from avivtur/fix-centos-vm-name

            Thanks Tal. So then the bug is also for creating VMs from InstanceTypes, too, in 4.14, where the checkbox is enabled but checked by default.

            Hilda Stastna (Inactive) added a comment - - edited Thanks Tal. So then the bug is also for creating VMs from InstanceTypes, too, in 4.14, where the checkbox is enabled but checked by default.

            Tal Tobias added a comment -

            hstastna@redhat.com 
            The checkbox should be enabled but not checked by default.

            i.e - empty clickable square

            Tal Tobias added a comment - hstastna@redhat.com   The checkbox should be enabled but not checked by default. i.e - empty clickable square

            Hi rh-ee-ttobias,

            This applies anywhere we create new VMs (template and instancetype), but creating with instancetypes is in the correct state, so the only fix that should be done is in creating a VM from the template.

            Does this mean that when creating VM from InstanceType the checkbox should be enabled?

            Hilda Stastna (Inactive) added a comment - Hi rh-ee-ttobias , This applies anywhere we create new VMs (template and instancetype), but creating with instancetypes is in the correct state, so the only fix that should be done is in creating a VM from the template. Does this mean that when creating VM from InstanceType the checkbox should be enabled?

            Tal Tobias added a comment -

            hstastna@redhat.com 
            You're right.

            Tal Tobias added a comment - hstastna@redhat.com   You're right.

            Another bug related to the checkbox:
            when unchecking the checkbox and saving the change, the change is not saved at all:
            when opening the checkbox the second time, it remains checked!
            This is true for both "Use existing" and "Add new" radio buttons.

            rh-ee-ttobias am I right and is this a bug? Or is that an intentional behavior? Thanks!

            Hilda Stastna (Inactive) added a comment - - edited Another bug related to the checkbox: when unchecking the checkbox and saving the change, the change is not saved at all: when opening the checkbox the second time, it remains checked! This is true for both "Use existing" and "Add new" radio buttons. rh-ee-ttobias am I right and is this a bug? Or is that an intentional behavior? Thanks!

            Tal Tobias added a comment -

            Hi hstastna@redhat.com 
            This applies anywhere we create new VMs (template and instancetype), but creating with instancetypes is in the correct state, so the only fix that should be done is in creating a VM from the template.

            Thank you!

            Tal Tobias added a comment - Hi hstastna@redhat.com   This applies anywhere we create new VMs (template and instancetype), but creating with instancetypes is in the correct state, so the only fix that should be done is in creating a VM from the template. Thank you!

            Hi rh-ee-ttobias, just to make sure: does this apply ONLY on the scenario of creating a VM from template (in Catalog)? We have more ways of creating a VM. Thanks in advance!

            Hilda Stastna (Inactive) added a comment - Hi rh-ee-ttobias , just to make sure: does this apply ONLY on the scenario of creating a VM from template (in Catalog )? We have more ways of creating a VM. Thanks in advance!

              aturgema Aviv Turgeman
              rh-ee-ttobias Tal Tobias
              Shijie Xue Shijie Xue (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved: