Skip to content

[iis][windows] Adds cpu number information to IIS module and Windows perfmon dataset#48637

Open
giorgi-imerlishvili-elastic wants to merge 6 commits intoelastic:mainfrom
giorgi-imerlishvili-elastic:iis-show-cpu-number2
Open

[iis][windows] Adds cpu number information to IIS module and Windows perfmon dataset#48637
giorgi-imerlishvili-elastic wants to merge 6 commits intoelastic:mainfrom
giorgi-imerlishvili-elastic:iis-show-cpu-number2

Conversation

@giorgi-imerlishvili-elastic
Copy link
Contributor

@giorgi-imerlishvili-elastic giorgi-imerlishvili-elastic commented Feb 1, 2026

Proposed commit message

Adds cpu number information to IIS module and Windows perfmon dataset
Purpose of this change is to help customers have better view on CPU performance. For example with IIS module customer is able to see CPU load but currently it doesn't have ability to show how many CPU are there in total. This sometimes might lead to confusion because currently CPU load is provided with percents where maximum percent cap is calculated via this formula: from 0 through 100 * ProcessorCount (source: https://learn.microsoft.com/en-us/windows/win32/perfctrs/collecting-performance-data). So when someone sees for example CPU load: 155% without knowing CPU number, they won't be able to calculate cap.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works. Where relevant, I have used the stresstest.sh script to run them under stress conditions and race detector to verify their stability.
  • I have added an entry in ./changelog/fragments using the changelog tool.

Disruptive User Impact

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Use cases

Screenshots

Logs

@giorgi-imerlishvili-elastic giorgi-imerlishvili-elastic added enhancement Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team labels Feb 1, 2026
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Feb 1, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@mergify
Copy link
Contributor

mergify bot commented Feb 1, 2026

This pull request does not have a backport label.
If this is a bug or security fix, could you label this PR @giorgi-imerlishvili-elastic? 🙏.
For such, you'll need to label your PR with:

  • The upcoming major version of the Elastic Stack
  • The upcoming minor version of the Elastic Stack (if you're not pushing a breaking change)

To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2026

@elasticmachine
Copy link
Contributor

Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)

@VihasMakwana
Copy link
Contributor

@giorgi-imerlishvili-elastic Is there an open issue for this? If not, could you update PR description with "Why this change is required and what does it fix/enhance?"

@giorgi-imerlishvili-elastic
Copy link
Contributor Author

@giorgi-imerlishvili-elastic Is there an open issue for this? If not, could you update PR description with "Why this change is required and what does it fix/enhance?"

Just added description and related issues

Copy link
Contributor

@VihasMakwana VihasMakwana left a comment

Choose a reason for hiding this comment

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

just a nit. LGTM otherwise

Copy link
Contributor

@jmikell821 jmikell821 left a comment

Choose a reason for hiding this comment

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

Just a couple of slight comments, thanks!

Webserver related metrics.

**`iis.webserver.host.cpu.count`**
: Number of CPUs host have
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
: Number of CPUs host have
: Number of CPUs the host has

- name: host.cpu.count
type: long
description: >
Number of CPUs host have
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Number of CPUs host have
Number of CPUs the host has

@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2026

✅ Vale Linting Results

No issues found on modified lines!


The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants