Skip to content

Conversation

@bhagyapathak
Copy link
Contributor

@bhagyapathak bhagyapathak commented Dec 12, 2025

Fixes verify_reload_hyperv_modules, stress_tlb_basic, stress_tlb_stressng test cases

@LiliDeng LiliDeng merged commit 4273419 into microsoft:main Feb 2, 2026
27 checks passed
@bhagyapathak bhagyapathak requested a review from LiliDeng February 2, 2026 06:19
@kanchansenlaskar
Copy link
Collaborator

@bhagyapathak, As discussed offline, this change in lisa/tools/modprobe.py is causing a regression as the chown command is being run without waiting for the vm connectivity to come up or waiting for the reload process to finish. Kindly either:

  1. pass sudo=True in this line: cat.read(loop_process_pid_file_name, force_run=True)
    or
  2. move the chown command inside the try section of the while loop.

I prefer the 1st option, and then please test it.
cc: @LiliDeng

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.

4 participants