Skip to content

Commit a230d70

Browse files
Manciukicroypat
authored andcommitted
chore: remove sleep from wrmsr test
Remove the sleep as they are now passing successfully on AMD as well. Closes: #3453 Signed-off-by: Riccardo Mancini <[email protected]>
1 parent faaea96 commit a230d70

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

tests/integration_tests/functional/test_cpu_features_x86_64.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import re
1212
import shutil
1313
import sys
14-
import time
1514
from difflib import unified_diff
1615
from pathlib import Path
1716

@@ -421,11 +420,6 @@ def test_cpu_wrmsr_snapshot(
421420
msrs_before_fname = snapshot_artifacts_dir / shared_names["msrs_before_fname"]
422421

423422
dump_msr_state_to_file(msr_reader_bin, msrs_before_fname, vm.ssh)
424-
# On T2A, the restore test fails with error "cannot allocate memory" so,
425-
# adding delay below as a workaround to unblock the tests for now.
426-
# TODO: Debug the issue and remove this delay. Create below issue to track this:
427-
# https://github.com/firecracker-microvm/firecracker/issues/3453
428-
time.sleep(0.25)
429423

430424
# Take a snapshot
431425
snapshot = vm.snapshot_diff()

0 commit comments

Comments
 (0)