FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

AD fixes (windows.plugin) by thiagoftsm · Pull Request #21454 · netdata/netdata · GitHub

AD fixes (windows.plugin) - #21454

Merged
thiagoftsm merged 4 commits into
netdata:masterfrom
thiagoftsm:perflib_ad
Dec 12, 2025
Merged

AD fixes (windows.plugin)#21454
thiagoftsm merged 4 commits into
netdata:masterfrom
thiagoftsm:perflib_ad

Conversation

thiagoftsm commented Dec 12, 2025
edited by cubic-dev-ai Bot
Loading

Copy link
Copy Markdown
Contributor
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 Bot added area/collectors Everything related to data collection collectors/windows labels Dec 12, 2025
thiagoftsm requested a review from stelfrag December 12, 2025 01:58

cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

No issues found across 3 files

thiagoftsm merged commit 22fbd2c into netdata:master Dec 12, 2025
119 checks passed
thiagoftsm deleted the perflib_ad branch December 12, 2025 12:27
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
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL