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

neonvm-controller: expand the tests with QMP mocked #943

Closed
wants to merge 14 commits into from

Conversation

Omrigan
Copy link
Contributor

@Omrigan Omrigan commented May 24, 2024

Bases on top of the #935

Part of #763, resolves #716

Omrigan added 6 commits May 23, 2024 12:40
Signed-off-by: Oleg Vasilev <[email protected]>
Signed-off-by: Oleg Vasilev <[email protected]>
Signed-off-by: Oleg Vasilev <[email protected]>
Signed-off-by: Oleg Vasilev <[email protected]>
Signed-off-by: Oleg Vasilev <[email protected]>
Signed-off-by: Oleg Vasilev <[email protected]>
@sharnoff
Copy link
Member

I see this PR moves the qmp files into a separate package. That would resolve #716, right? (maybe that can be a separate PR? up to you)

Omrigan added 4 commits May 30, 2024 17:09
Signed-off-by: Oleg Vasilev <[email protected]>
Signed-off-by: Oleg Vasilev <[email protected]>
Signed-off-by: Oleg Vasilev <[email protected]>
Omrigan added 4 commits May 30, 2024 17:25
Signed-off-by: Oleg Vasilev <[email protected]>
Signed-off-by: Oleg Vasilev <[email protected]>

complete test

Signed-off-by: Oleg Vasilev <[email protected]>

add new files

Signed-off-by: Oleg Vasilev <[email protected]>
@Omrigan Omrigan force-pushed the oleg/neonvm-qmp-tests branch from 6506a93 to c827e71 Compare June 17, 2024 13:16
@Omrigan Omrigan mentioned this pull request Jun 21, 2024
5 tasks
Base automatically changed from oleg/neonvm-tests to main June 21, 2024 16:49
@Omrigan
Copy link
Contributor Author

Omrigan commented Aug 11, 2024

Superseded by #1032

@Omrigan Omrigan closed this Aug 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

move neonvm controller qmp logic into separate package
2 participants