Skip to content

Conversation

@Menglths
Copy link
Contributor

@Menglths Menglths commented Dec 9, 2025

Purpose

To avoid manual errors, an automated pre-check for the initial setup is needed. This helps detect issues early, improve inference stability, and prevent faults from configuration errors.

Modifications

Added configurations for SSH password-free login, health status checks of hardware devices, network connectivity between nodes, TLS verification between nodes, model weight file integrity and consistency, and storage mount point rate checks.

  • Added test/common/envprecheck/run_env_precheck.py

  • Added test/suites/E2E/test_environment_precheck.py

  • Modified test/config.yaml

  • Modified test/suites/E2E/test_uc_performance.py

Test

Verified each pre-check function runs correctly. Assertions for overall status are preserved and working.
image

@Menglths Menglths force-pushed the environment-preCheck branch 5 times, most recently from 4c8a646 to e5dd2a6 Compare December 9, 2025 09:21
# ========= Environment Setup =========
@pytest.mark.stage(0)
@pytest.mark.feature("test_ssh_login")
@export_vars
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this decorator is only for internal usage and we may not need to update external database while doing precheck things

@@ -0,0 +1,238 @@
import os
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A guide doc is also needed to let users run this test fixture easily

@Menglths Menglths force-pushed the environment-preCheck branch 3 times, most recently from 275022a to b178efd Compare December 9, 2025 13:38
@Menglths Menglths force-pushed the environment-preCheck branch 2 times, most recently from cceae72 to df1b0b2 Compare December 9, 2025 13:54
@Menglths Menglths force-pushed the environment-preCheck branch 3 times, most recently from 3152df0 to 56b1416 Compare December 9, 2025 14:22
@Menglths Menglths closed this Dec 9, 2025
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.

2 participants