Skip to content

ci: load kernel module for passkey testing with vfido#8489

Merged
ikerexxe merged 1 commit into
SSSD:masterfrom
ikerexxe:test-vhci-hcd
Mar 6, 2026
Merged

ci: load kernel module for passkey testing with vfido#8489
ikerexxe merged 1 commit into
SSSD:masterfrom
ikerexxe:test-vhci-hcd

Conversation

@ikerexxe

@ikerexxe ikerexxe commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

virtual-fido requires vhci-hcd kernel module to be loaded on the host to work

@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@ikerexxe
ikerexxe force-pushed the test-vhci-hcd branch 2 times, most recently from a626f26 to 7c4ab91 Compare March 3, 2026 15:05
@ikerexxe ikerexxe changed the title ci: load kernel module for passkey testing ci: load kernel module for passkey testing with vfido Mar 3, 2026
@ikerexxe
ikerexxe marked this pull request as ready for review March 4, 2026 15:48

@spoore1 spoore1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

virtual-fido requires `vhci-hcd` kernel module to be loaded to work

Signed-off-by: Iker Pedrosa <[email protected]>
Reviewed-by: Justin Stephenson <[email protected]>
Reviewed-by: Scott Poore <[email protected]>
@sssd-bot

sssd-bot commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

The pull request was accepted by @spoore1 with the following PR CI status:


🟢 CodeQL (success)
🟢 osh-diff-scan:fedora-rawhide-x86_64:upstream (success)
🟢 rpm-build:centos-stream-10-x86_64:upstream (success)
🟢 rpm-build:fedora-42-x86_64:upstream (success)
🟢 rpm-build:fedora-43-x86_64:upstream (success)
🟢 rpm-build:fedora-44-x86_64:upstream (success)
🟢 rpm-build:fedora-rawhide-x86_64:upstream (success)
🟢 Analyze (target) / cppcheck (success)
🟢 Build / freebsd (success)
🟢 Build / make-distcheck (success)
🟢 ci / intgcheck (centos-10) (success)
🟢 ci / intgcheck (fedora-42) (success)
🟢 ci / intgcheck (fedora-43) (success)
🟢 ci / intgcheck (fedora-44) (success)
🟢 ci / intgcheck (fedora-45) (success)
🟢 ci / prepare (success)
🟢 ci / system (centos-10) (success)
🟢 ci / system (fedora-42) (success)
🟢 ci / system (fedora-43) (success)
🟢 ci / system (fedora-44) (success)
🟢 ci / system (fedora-45) (success)
➖ Coverity scan / coverity (skipped)
🟢 Static code analysis / codeql (success)
🟢 Static code analysis / pre-commit (success)
🟢 Static code analysis / python-system-tests (success)


There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants