Skip to content

Comments

Opensvc daemon status and warning based on treshold#1237

Merged
caffeinated92 merged 2 commits intodevelopfrom
opensvc-daemon-status
Oct 7, 2025
Merged

Opensvc daemon status and warning based on treshold#1237
caffeinated92 merged 2 commits intodevelopfrom
opensvc-daemon-status

Conversation

@caffeinated92
Copy link
Collaborator

This pull request introduces new monitoring and warning mechanisms for OpenSVC orchestrator nodes, focusing on memory and swap thresholds, and improves the reliability and performance of OpenSVC stats collection. It also includes minor fixes and enhancements to the dashboard display and error messaging.

OpenSVC node health monitoring:

  • Added the CheckOpenSVCTresholds method to Cluster to monitor nodes for minimum available memory and swap thresholds, setting new warning states (WARN0150, WARN0151) when thresholds are breached.
  • Introduced new error messages in config/error.go for memory and swap threshold breaches (WARN0150, WARN0151).

OpenSVC stats handling improvements:

  • Updated DaemonNodeStats struct to include MinAvailMem and MinAvailSwap fields, and modified the stats extraction logic to retrieve these values. [1] [2]
  • Changed the behavior of GetOpenSVCStats to return an empty slice instead of an error when not using OpenSVC, improving downstream logic.

Concurrency and performance enhancements:

  • Made OpenSVC stats reload in ReplicationManager concurrent and protected with a flag to prevent overlapping fetches, improving performance and reliability. [1] [2] [3]

Dashboard calculation fix:

  • Corrected the calculation of used memory and swap in the OpenSVC dashboard component to use percentages, ensuring accurate display.

…improve stats fetching logic in ReplicationManager; round memory and swap usage calculations in OpenSVCNodeCard for accuracy
@caffeinated92 caffeinated92 merged commit ca4bb6c into develop Oct 7, 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.

1 participant