Skip to content

Commit 05b4772

Browse files
committed
moved check outside static if else
1 parent 0eec9f7 commit 05b4772

File tree

1 file changed

+8
-6
lines changed

1 file changed

+8
-6
lines changed

nodescraper/plugins/inband/amdsmi/amdsmi_analyzer.py

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -441,12 +441,7 @@ def analyze_data(
441441
self.check_expected_max_power(data.static, args.expected_max_power)
442442
if args.expected_driver_version:
443443
self.check_expected_driver_version(data.static, args.expected_driver_version)
444-
if args.expected_memory_partition_mode or args.expected_compute_partition_mode:
445-
self.check_expected_memory_partition_mode(
446-
data.partition,
447-
args.expected_memory_partition_mode,
448-
args.expected_compute_partition_mode,
449-
)
444+
450445
self.static_consistancy_check(data.static)
451446
if (
452447
self.system_info.sku
@@ -464,6 +459,13 @@ def analyze_data(
464459
sku_name=args.sku_name,
465460
)
466461

462+
if args.expected_memory_partition_mode or args.expected_compute_partition_mode:
463+
self.check_expected_memory_partition_mode(
464+
data.partition,
465+
args.expected_memory_partition_mode,
466+
args.expected_compute_partition_mode,
467+
)
468+
467469
if args.expected_pldm_version:
468470
self.check_pldm_version(data.firmware, args.expected_pldm_version)
469471

0 commit comments

Comments
 (0)