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

Rework usb-hid-and-msc-support.robot #216

Open
1 of 4 tasks
Psotas opened this issue Mar 8, 2024 · 3 comments
Open
1 of 4 tasks

Rework usb-hid-and-msc-support.robot #216

Psotas opened this issue Mar 8, 2024 · 3 comments
Assignees
Labels
enhancement New feature or request usb

Comments

@Psotas
Copy link
Contributor

Psotas commented Mar 8, 2024

The problem you're addressing (if any)

Describe the solution you'd like

No response

Where is the value to a user, and who might that user be?

No response

Describe alternatives you've considered

No response

Additional context

No response

@m-iwanicki
Copy link
Contributor

USB001.001 should work when this PR will be merged: #351

Currently:

 robot -v snipeit:no -L TRACE -v rte_ip:192.168.10.233 -v config:protectli-vp2410  dasharo-compatibility/usb-hid-and-msc-support.robot
==============================================================================
Usb-Hid-And-Msc-Support
==============================================================================
USB001.001 USB devices detected in FW :: Check whether USB devices... | FAIL |
Variable '${PIKVM_IP}' not found.
------------------------------------------------------------------------------
USB001.002 USB devices detected by OS (Ubuntu) :: Check whether th... ......
Checking if usbutils is installed...

Package usbutils is installed
USB001.002 USB devices detected by OS (Ubuntu) :: Check whether th... | FAIL |
'      bInterfaceClass         8 Mass Storage
      bInterfaceClass         9 Hub
      bInterfaceClass       224 Wireless
      bInterfaceClass       224 Wireless
      bInterfaceClass       224 Wireless
      bInterfaceClass       224 Wireless
      bInterfaceClass       224 Wireless
      bInterfaceClass       224 Wireless
      bInterfaceClass       224 Wireless
      bInterfaceClass       224 Wireless
      bInterfaceClass         9 Hub' does not contain 'Human Interface Device'
------------------------------------------------------------------------------
USB001.003 USB devices detected by OS (Windows) :: Check whether t... | SKIP |
USB001.003 not supported
------------------------------------------------------------------------------
USB002.001 USB keyboard detected in FW :: Check whether the extern... | PASS |
------------------------------------------------------------------------------
USB002.002 USB keyboard in OS (Ubuntu) :: Check whether the extern... | PASS |
------------------------------------------------------------------------------
USB002.003 USB keyboard in OS (Windows) :: Check whether the exter... | SKIP |
USB002.003 not supported
------------------------------------------------------------------------------
USB003.001 Upload 1GB file on USB storage (Ubuntu) :: Check whethe... | SKIP |
USB003.001 not supported
------------------------------------------------------------------------------
USB003.002 Upload 1GB file on USB storage (Windows) :: Check wheth... | SKIP |
USB003.002 not supported
------------------------------------------------------------------------------
Usb-Hid-And-Msc-Support                                               | FAIL |
8 tests, 2 passed, 2 failed, 4 skipped
==============================================================================

With my PR:

 robot -v snipeit:no -L TRACE -v rte_ip:192.168.10.233 -v config:protectli-vp2410  dasharo-compatibility/usb-hid-and-msc-support.robot
==============================================================================
Usb-Hid-And-Msc-Support
==============================================================================
USB001.001 USB devices detected in FW :: Check whether USB devices... | PASS |
------------------------------------------------------------------------------
USB001.002 USB devices detected by OS (Ubuntu) :: Check whether th... ......
Checking if usbutils is installed...

Package usbutils is installed
USB001.002 USB devices detected by OS (Ubuntu) :: Check whether th... | PASS |
------------------------------------------------------------------------------
USB001.003 USB devices detected by OS (Windows) :: Check whether t... | SKIP |
USB001.003 not supported
------------------------------------------------------------------------------
USB002.001 USB keyboard detected in FW :: Check whether the extern... | SKIP |
USB002.001 not supported
------------------------------------------------------------------------------
USB002.002 USB keyboard in OS (Ubuntu) :: Check whether the extern... | SKIP |
USB002.002 not supported
------------------------------------------------------------------------------
USB002.003 USB keyboard in OS (Windows) :: Check whether the exter... | SKIP |
USB002.003 not supported
------------------------------------------------------------------------------
USB003.001 Upload 1GB file on USB storage (Ubuntu) :: Check whethe... | SKIP |
USB003.001 not supported
------------------------------------------------------------------------------
USB003.002 Upload 1GB file on USB storage (Windows) :: Check wheth... | SKIP |
USB003.002 not supported
------------------------------------------------------------------------------
Usb-Hid-And-Msc-Support                                               | PASS |
8 tests, 2 passed, 0 failed, 6 skipped
==============================================================================

2 failing tests now succeed and 2 succeeding tests are now skipped as on tested machine there was no keyboard.

@m-iwanicki
Copy link
Contributor

Both should be now fixed in #351:

From what I've tested

Should also be fixed (after merge)

@m-iwanicki
Copy link
Contributor

usb-hid-and-msc-support.robot should still be reworked somehow, because currently I think for tests to complete successfully USB drive should have preferably one FAT32 partition that's a little over 2 GB in size (maybe a little over 1 GB would work but I'm not sure). This partition should also be "bootable" so USB is detected in Boot Menu
In my tests I prepared USB with gdisk, with partition set as EFI (type EF00) and then formatted with mkfs.fat -F 32 /dev/sdc1

ubuntu@3mdeb:~$ lsblk /dev/sdc -o +fstype
NAME   MAJ:MIN RM  SIZE RO TYPE MOUNTPOINTS FSTYPE
sdc      8:32   1 28,7G  0 disk
└─sdc1   8:33   1   10G  0 part             vfat

And then copied EFI directory from random /boot directory (left only Boot directory removed rest)

.
├── EFI
│   └── Boot
│       ├── bootx64.efi
│       ├── fbx64.efi
│       └── mmx64.efi
└── System Volume Information
    ├── IndexerVolumeGuid
    └── WPSettings.dat

3 directories, 5 files

I also had to set disk letter first time manually in Windows.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request usb
Projects
None yet
Development

No branches or pull requests

3 participants