Skip to content

Remove in-process mode from hyperlight-guest and remaining clean up in hyperlight-host #496

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

Merged

Conversation

simongdavies
Copy link
Contributor

@simongdavies simongdavies commented May 15, 2025

Completes implementation of #472

One final simplification we can make is to get rid of OutBHandlerWrapper since we now only have one implementation of outb handler function now.

Added #495 to track this

@simongdavies simongdavies added the kind/enhancement For PRs adding features, improving functionality, docs, tests, etc. label May 15, 2025
@simongdavies simongdavies force-pushed the remove-in-process-guest branch from 9284c53 to d90df64 Compare May 15, 2025 19:33
@simongdavies simongdavies changed the title Remove in-process mode from hyperlight-guest and remaining clean up im hyperlight-host Remove in-process mode from hyperlight-guest and remaining clean up in hyperlight-host May 15, 2025
@simongdavies simongdavies force-pushed the remove-in-process-guest branch from d90df64 to 51317d7 Compare May 15, 2025 19:44
Copy link
Contributor

@ludfjig ludfjig left a comment

Choose a reason for hiding this comment

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

Nice :)

Copy link
Contributor

@danbugs danbugs left a comment

Choose a reason for hiding this comment

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

LGTM

@simongdavies simongdavies merged commit e63a0cc into hyperlight-dev:main May 15, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement For PRs adding features, improving functionality, docs, tests, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants