Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Sending ctrl-alt-delete does not have any reaction on a windows VM #4535

Closed
Martin-Weiss opened this issue Sep 15, 2023 · 8 comments · Fixed by harvester/dashboard#925
Assignees
Labels
backport-needed/1.2.2 kind/bug Issues that are defects reported by users or that we know have reached a real release not-require/test-plan Skip to create a e2e automation test issue reproduce/needed Reminder to add a reproduce label and to remove this one severity/needed Reminder to add a severity label and to remove this one
Milestone

Comments

@Martin-Weiss
Copy link

Describe the bug
Here we do not have any reaction:

grafik

To Reproduce
Have a windows 2022 VM and try to send ctrl-alt-del

Expected behavior
ctrl-alt-del is received and we can login

Environment

  • Harvester ISO version: 1.2.0
  • Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): not relevant
  • Browser: Firefox 117.0 on Mac (M1)

Additional context
Workaround on Mac: use control+option+delete on the keyboard

@Martin-Weiss Martin-Weiss added kind/bug Issues that are defects reported by users or that we know have reached a real release reproduce/needed Reminder to add a reproduce label and to remove this one severity/needed Reminder to add a severity label and to remove this one labels Sep 15, 2023
@joshfranzen
Copy link

joshfranzen commented Oct 2, 2023

Same issue here running on Harvester 1.2.0

here is the VM config:

spec:
  runStrategy: RerunOnFailure
  template:
    metadata:
      annotations:
        harvesterhci.io/sshNames: '[]'
        harvesterhci.io/waitForLeaseInterfaceNames: '[]'
      creationTimestamp: null
      labels:
        harvesterhci.io/creator: terraform-provider-harvester
        harvesterhci.io/vmName: d-n-win-image
    spec:
      affinity:
        podAntiAffinity:
          preferredDuringSchedulingIgnoredDuringExecution:
            - podAffinityTerm:
                labelSelector:
                  matchExpressions:
                    - key: harvesterhci.io/creator
                      operator: Exists
                topologyKey: kubernetes.io/hostname
              weight: 100
      domain:
        cpu:
          cores: 4
        devices:
          disks:
            - name: cdrom-disk
              cdrom:
                bus: sata
              bootOrder: 1
            - name: osdisk
              disk:
                bus: sata
              bootOrder: 2
          inputs:
            - bus: usb
              name: tablet
              type: tablet
          interfaces:
            - masquerade: {}
              macAddress: 52:54:00:e1:5c:be
              model: e1000
              name: nic-1
        features:
          acpi: {}
          smm:
            enabled: true
        firmware:
          bootloader:
            efi:
              secureBoot: true
        machine:
          type: q35
        memory:
          guest: 1948Mi
        resources:
          limits:
            cpu: '4'
            memory: 2Gi
          requests:
            cpu: 250m
            memory: 1365Mi
      evictionStrategy: LiveMigrate
      hostname: d-n-win-image
      networks:
        - name: nic-1
          pod: {}
      terminationGracePeriodSeconds: 120
      volumes:
        - name: cdrom-disk
          persistentVolumeClaim:
            claimName: d-n-win-image-cdrom-disk-cpfhh
        - name: osdisk
          persistentVolumeClaim:
            claimName: d-n-win-image-osdisk-bphjg

@alexey-rawyan
Copy link

alexey-rawyan commented Oct 22, 2023

as for now you can create management shortcut key (record new one with ctrl then alt then del, and save it) - it works.
image

@W0n9
Copy link

W0n9 commented Oct 23, 2023

The same problem here, and custom shortcut works.

@gaktive
Copy link

gaktive commented Jan 2, 2024

@bk201 I see the PR was merged back on that day but this ticket was reopened. Is there something left to do here?

Internal reference: SURE-7210

@bk201 bk201 added this to the v1.3.0 milestone Jan 3, 2024
@bk201 bk201 added the not-require/test-plan Skip to create a e2e automation test issue label Jan 3, 2024
@harvesterhci-io-github-bot

Pre Ready-For-Testing Checklist

  • If labeled: require/HEP Has the Harvester Enhancement Proposal PR submitted?
    The HEP PR is at:

  • Where is the reproduce steps/test steps documented?
    The reproduce steps/test steps are at:

  • Is there a workaround for the issue? If so, where is it documented?
    The workaround is at:

  • Have the backend code been merged (harvester, harvester-installer, etc) (including backport-needed/*)?
    The PR is at:

    • Does the PR include the explanation for the fix or the feature?

    • Does the PR include deployment change (YAML/Chart)? If so, where are the PRs for both YAML file and Chart?
      The PR for the YAML change is at:
      The PR for the chart change is at:

  • If labeled: area/ui Has the UI issue filed or ready to be merged?
    The UI issue/PR is at:

  • If labeled: require/doc, require/knowledge-base Has the necessary document PR submitted or merged?
    The documentation/KB PR is at:

  • If NOT labeled: not-require/test-plan Has the e2e test plan been merged? Have QAs agreed on the automation test case? If only test case skeleton w/o implementation, have you created an implementation issue?

    • The automation skeleton PR is at:
    • The automation test case PR is at:
  • If the fix introduces the code for backward compatibility Has a separate issue been filed with the label release/obsolete-compatibility?
    The compatibility issue is filed at:

@bk201
Copy link
Member

bk201 commented Jan 3, 2024

@torchiaf Can you update the test plan in #4535 (comment) and move the issue to ready for testing? And we also need to backport this to v1.2.2.

@bk201
Copy link
Member

bk201 commented Jan 3, 2024

@bk201 I see the PR was merged back on that day but this ticket was reopened. Is there something left to do here?

Internal reference: SURE-7210

@gaktive The issue was reopened because the issue was accidentally closed by keywords in the issue's PR. The internal ticket will be marked as resolved once QA verifies the issue.

@TachunLin TachunLin self-assigned this Jan 29, 2024
@TachunLin
Copy link

TachunLin commented Jan 29, 2024

Verified fixed on v1.3.0-rc1 (01/26). Close this issue.

Result

$\color{green}{\textsf{PASS}}$ Click ctrl+alt+delete key combination works on windows login page $~~$
  • Click the ctrl+alt+delete key combination on the windows server 2022 login page works as expected
vokoscreenNG-2024-01-29_14-46-44.mp4
$\color{green}{\textsf{PASS}}$ Click ctrl+alt+delete key combination works inside windows $~~$
  • Click the ctrl+alt+delete key combination after login to windows server 2022 desktop works as expected
vokoscreenNG-2024-01-29_14-47-57.mp4

Test Information

  • Test Environment: Single nodes harvester on local kvm machine
  • Harvester version: v1.3.0-rc1 (01/26)
  • Windows ISO: windows_server_2022_en-us

Verify Steps

On Windows server 2022
  1. Create windows server 2022 iso image
  2. Create a VM, add the second SATA disk
  3. Install windows server 2002 with desktop
  4. After installation complete
  5. Open the web console
  6. In the windows login page, click the Shortcut Keys
  7. Click Ctrl , Alt, Del in sequence
  8. After login to desktop
  9. Click Ctrl , Alt, Del in sequence again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-needed/1.2.2 kind/bug Issues that are defects reported by users or that we know have reached a real release not-require/test-plan Skip to create a e2e automation test issue reproduce/needed Reminder to add a reproduce label and to remove this one severity/needed Reminder to add a severity label and to remove this one
Projects
Community Issue Review
Resolved/Scheduled
Development

Successfully merging a pull request may close this issue.

10 participants