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

Unified worker #996

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from
Draft

Unified worker #996

wants to merge 9 commits into from

Conversation

jakogut
Copy link
Contributor

@jakogut jakogut commented May 17, 2023

This PR moves the QEMU worker into the core module, removing the IPC mechanism at the same time.

A CLI script becomes the entrypoint for testing.

This PR requires the unified-leviathan branch from meta-balena to run the unmanaged OS suite.

jakogut added 9 commits May 16, 2023 15:06
Change-type: minor
Signed-off-by: Joseph Kogut <[email protected]>
Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Remove catch block that unconditionally returns undefined, swallowing
any exceptions. Instead, allow them to be raised, increasing visibility
and debuggability.

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Worker becomes BaseWorker, the interface for workers. Remove server
code, instead allowing worker implementations to directly inherit from
and override base worker functionality.

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Previously, images used for testing were gzipped, and would be extracted
to a temporary location before flashing and executing tests. This
necessitated gzipping the image before using it, and introduced
unnecessary complexity and runtime costs. Remove it in favor of a single
input image path to an uncompressed image.

Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
Change-type: patch
Signed-off-by: Joseph Kogut <[email protected]>
@jakogut jakogut requested review from klutchell and removed request for klutchell May 17, 2023 01:46
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.

2 participants