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

CMS: probe checking the number and volume of rules per rse, activity and state #151

Merged
merged 1 commit into from
Dec 3, 2024

Conversation

eachristgr
Copy link
Contributor

@eachristgr eachristgr commented Nov 8, 2024

Related to dmwm/CMSRucio#747

The script acts on rules with the following states:

  • OK
  • REPLICATING
  • STUCK

The script pushes the following metrics by using PrometheusPusher:

  • rule_count_per_rse_activity_state: The number of rules per rse, activity and state
  • rule_volume_per_rse_activity_state: The total size (in bytes) of rules per rse, activity and state

Tested it on Integration and it works as far as I can see.
More details about the implementation are commented in the script.

FYI @haozturk

eachristgr added a commit to eachristgr/CMSRucio that referenced this pull request Nov 11, 2024
eachristgr added a commit to eachristgr/CMSRucio that referenced this pull request Nov 11, 2024
ericvaandering added a commit to dmwm/CMSRucio that referenced this pull request Nov 27, 2024
@eachristgr eachristgr added the CMS label Dec 2, 2024
@eachristgr
Copy link
Contributor Author

Changes were applied in favor of readability and commits were squashed to ensure a cleaner merge.

Copy link
Contributor

@ericvaandering ericvaandering left a comment

Choose a reason for hiding this comment

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

Sorry, two more changes.

  1. Please put a newline at the very end
  2. The commit has to be renamed to designate CMS: "The subject of each commit must be prefixed by a label. This can be omitted for rare changes across the entire repository. If there is an issue related to the commit, then its number should be appended. For example, a commit may resemble this: ATLAS: Port probes to Python 3 #1234" per the Contributing.md file

@eachristgr
Copy link
Contributor Author

Sorry, two more changes.

1. Please put a newline at the very end

2. The commit has to be renamed to designate CMS: "The subject of each commit must be prefixed by a label. This can be omitted for rare changes across the entire repository. If there is an issue related to the commit, then its number should be appended. For example, a commit may resemble this: ATLAS: Port probes to Python 3 #1234" per the Contributing.md file

Changes applied. Hope the commit message looks good as well.

@ericvaandering
Copy link
Contributor

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants