Skip to content

IBperf logging improvement#259

Open
sarachoi-amd wants to merge 2 commits into
mainfrom
feature/ibperf_logging
Open

IBperf logging improvement#259
sarachoi-amd wants to merge 2 commits into
mainfrom
feature/ibperf_logging

Conversation

@sarachoi-amd

Copy link
Copy Markdown

Motivation

Goals: Make ibperf test logs easier to read and grep, with less SSH noise, while keeping the result and verification output

Technical Details

Files changed:

  • cvs/lib/ibperf_lib.py
  • cvs/tests/ibperf/ib_perf_bw_test.py
  • cvs/tests/ibperf/install_ibperf_tools.py

What improved:

  • Logging infrastructure: switched to the standard logging module and added structured helpers for bandwidth/latency summaries and run-completion messages
  • Less console noise: suppressed remote command output, kept concise INFO summaries and moved verbose details to DEBUG
  • Code cleanup / deduplication: consolidated ROCm path detection and added caching to avoid repeated SSH lookups
  • Test script improvements: improved test logging across install and perf scripts, reduced verbosity, and added install skip notification
  • Minor fixes: added empty-result guards and removed redundant log lines

Test Plan

ran ib_perf_bw_test

Test Result

The test passed, with the log file size was reduced by half.

Submission Checklist

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