cclohmar/vm-bench
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: remove unused out_dir from submit_job + Copy flow uses legacy fallback

Browse source 
1. submit_job() was creating empty tmp/out/{vmid}/ dirs via the old
   flat STAGING_OUT path. Removed — _process_job handles this via
   _staging_out() in the session-aware path.

2. Copy (re-convert) flow was hardcoding boot_type=uefi. Now uses
   auto_detect_boot=true with boot_type=legacy as fallback, matching
   the behavior of the original analyze→convert flow.
This commit is contained in:
Claus Lohmar 2026-07-23 10:52:20 +00:00
parent 6617aca341
commit 90bf8e4842
2 changed files with 1 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
backend/provisioner.py
View file

@ -55,9 +55,6 @@ def submit_job(req: JobSubmissionRequest) -> JobStatusResponse:
"""Queue a conversion job and start it in a background thread."""
job_id = f"job_{req.vmid}_{int(time.time())}"
out_dir = STAGING_OUT / str(req.vmid)
out_dir.mkdir(parents=True, exist_ok=True)
with _jobs_lock:
_jobs[job_id] = {
"vmid": req.vmid,

2
frontend/templates/index.html
View file

@ -418,7 +418,7 @@ async function copyVM(jobId, vmid, sourceFilename) {
const diskGb = document.getElementById('copy-disk').value;
if (diskGb) payload.append('target_disk_size_gb', diskGb);
payload.append('auto_detect_boot', 'true');
payload.append('boot_type', 'uefi');
payload.append('boot_type', 'legacy');
payload.append('session_id', window.VM_BENCH_SID || '');
try {
const resp = await fetch('/session/confirm', { method: 'POST', body: payload });