Skip to content

Conversation

@l8556
Copy link
Member

@l8556 l8556 commented Jan 20, 2026

No description provided.

Nikolai Lambrov added 7 commits January 20, 2026 09:28
- Removed the call to `_remove_useless_dvd_images` from the VM unpacking process to streamline the workflow.
- Reintroduced the call to `_remove_useless_dvd_images` during VM registration, ensuring unnecessary DVD images are cleaned up at the appropriate stage.

These changes enhance the clarity and efficiency of the VM update process.
… cleanup in VmUpdater

- Changed the VBoxWrapper dependency in `pyproject.toml` and `uv.lock` to point to the `add_config_path` branch for improved configuration handling.
- Modified the `_remove_useless_dvd_images` method in the `VmUpdater` class to accept a `config_path` parameter, allowing for more flexible VM management during DVD image cleanup.

These changes improve the functionality and clarity of VM updates and dependency management.
… check

- Updated the VmUpdater class to ensure that the `_remove_useless_dvd_images` method is only called if the VBox file exists and is a valid file. This change improves the robustness of the VM registration process by preventing unnecessary calls when the file is not found.

These modifications enhance the clarity and reliability of VM management during the registration phase.
… to streamline supported operating systems for testing environments.
…mprove clarity. This change focuses on enhancing the readability of the module description without altering functionality.
…ion files

- Changed the VBoxWrapper dependency in `pyproject.toml` and `uv.lock` to reference the `master` branch instead of the `add_config_path` branch. This update ensures that the latest stable version of the VBoxWrapper is used for improved functionality and compatibility.
@l8556 l8556 added this pull request to the merge queue Jan 20, 2026
Merged via the queue into master with commit 62d13e8 Jan 20, 2026
5 checks passed
@l8556 l8556 deleted the fix_updater branch January 20, 2026 11:56
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