Skip to content

Conversation

@thiagoftsm
Copy link
Contributor

@thiagoftsm thiagoftsm commented Dec 12, 2025

Summary

The Pull Request (PR) introduces adjustments to the metrics and standards for the AD files.

Test Plan
  1. Check CI and possible typos.
Additional Information
For users: How does this change affect me?

Summary by cubic

Standardizes and fixes Windows AD, ADCS, and ADFS perflib charts so they collect the right counters and use consistent names. This prevents bad data and makes the plugin more reliable.

  • Bug Fixes

    • Guard against missing counters in AD charts (early return) and initialize chart/dim only once.
    • ADCS: fix retrieval processing chart and labels; use ADCSRequestCryptoSigningTime for signing chart; rename crypto_singing to crypto_signing; remove duplicate callback.
    • ADFS: correct metric keys for OAuth Basic vs Post flows; fix chart IDs/titles for logon certificate and password grant requests; call rrdset_done on the correct federation chart.
  • Refactors

    • Unify control flow and naming across perflib-ad.c for consistent chart creation and updates.

Written for commit dc33483. Summary will update automatically on new commits.

@github-actions github-actions bot added area/collectors Everything related to data collection collectors/windows labels Dec 12, 2025
@thiagoftsm thiagoftsm requested a review from stelfrag December 12, 2025 01:58
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 3 files

@thiagoftsm thiagoftsm merged commit 22fbd2c into netdata:master Dec 12, 2025
119 checks passed
@thiagoftsm thiagoftsm deleted the perflib_ad branch December 12, 2025 12:27
@stelfrag stelfrag mentioned this pull request Dec 12, 2025
stelfrag pushed a commit to stelfrag/netdata that referenced this pull request Dec 12, 2025
Ferroin pushed a commit that referenced this pull request Dec 15, 2025
(cherry picked from commit 22fbd2c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/collectors Everything related to data collection collectors/windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants