Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/rmsv2 #111

Open
wants to merge 13 commits into
base: features/m2m
Choose a base branch
from
Prev Previous commit
Next Next commit
added check for multiple rms collector enabled
Omri Avraham committed Sep 30, 2024
commit 9dff053a0b9ee2169e2a808b0bc35c6465b95bef
8 changes: 8 additions & 0 deletions omnistat/monitor.py
Original file line number Diff line number Diff line change
@@ -65,6 +65,14 @@ def __init__(self, config):
logging.error('Please choose either "enable_rocm_smi" or "enable_amd_smi" in runtime config')
sys.exit(1)

# verify only one RMS collector is enabled
if self.runtimeConfig["collector_enable_rms"] and self.runtimeConfig["collector_enable_rms_v2"]:
logging.error("")
logging.error("[ERROR]: Only one RMS data collector may be configured at a time.")
logging.error("")
logging.error('Please choose either "collector_enable_rms" or "collector_enable_rms_v2" in runtime config')
sys.exit(1)

self.runtimeConfig["collector_enable_amd_smi_process"] = config["omnistat.collectors"].getboolean(
"enable_amd_smi_process", False
)