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

CI Hardware Tests additions #32

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

szszszsz
Copy link
Member

@szszszsz szszszsz commented Nov 6, 2023

This PR adds a couple of changes related to the Hardware Tests project. Please see if these make sense to you:

  • html and xml reports
  • USBIP Sim version reporting
  • SSH flow correction
  • USBIP Sim setup correction
  • SSH test refactoring (to avoid import cycle among others)

@szszszsz szszszsz added the enhancement New feature or request label Nov 6, 2023
@robin-nitrokey
Copy link
Member

Currently it’s hard to review the changes because there are conflicts and because they are mostly in one commit. Please consider splitting it up into multiple PRs or commits and/or adding more context to the commits. I’ll try to extract the most obvious changes into separate PRs once I have the time.

@daringer
Copy link
Contributor

the ssh parts don't work for me and also don't inside the hil, I will extract the reports thing into a separate PR so we can use master for the hil

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

Successfully merging this pull request may close these issues.

3 participants