Skip to content

Uv migration - #600

Open
chombourger wants to merge 4 commits into
siemens:nextfrom
chombourger:uv-migration
Open

Uv migration#600
chombourger wants to merge 4 commits into
siemens:nextfrom
chombourger:uv-migration

Conversation

@chombourger

Copy link
Copy Markdown
Collaborator

No description provided.

Signed-off-by: Cedric Hombourger <cedric.hombourger@siemens.com>
Centralize exception definitions by moving MissingCowDeviceError from
mtda/storage/helpers/image.py to mtda/exceptions.py.

Signed-off-by: Cedric Hombourger <cedric.hombourger@siemens.com>
Migrate from from setup.py to uv + pyproject.toml to modernize
the build and packaging infrastructure.

Signed-off-by: Cedric Hombourger <cedric.hombourger@siemens.com>
commit():
- Missing self.lock acquisition (race with to_host/to_target)
- Returned None on success; callers expect True
- No guard against committing while storage is ON_TARGET,
  which risks corrupting the backing image

rollback():
- COW size passed as raw bytes with 'M' suffix, producing a
  disk 1024x larger than configured (e.g. 64 MiB → 64 TiB);
  fix by dividing by 1024^2 to match configure()

Signed-off-by: Cedric Hombourger <cedric.hombourger@siemens.com>
@fmoessbauer

Copy link
Copy Markdown
Member

To be frank, I'm not a fan of the uv migration: The uv tool itself is not packaged in Debian (there is an ITP and also partial packagings, but the command itself is still missing).

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants