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

Add support for wildcard, *, for SMB Server Shares in Windows module, Perfmon Metricset #31516

Open
sachin-frayne opened this issue May 4, 2022 · 7 comments
Labels
Metricbeat Metricbeat Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team

Comments

@sachin-frayne
Copy link
Contributor

Describe the enhancement: Add support for wildcard, *, for SMB Server Shares in Windows module, Perfmon Metricset

Describe a specific use case for the enhancement or feature: When you have multiple SMB Server Shares, and you want to collect data from all of them one would assume that you can use wildcard like with other systems but and error is thrown:

ERROR module/wrapper.go:259 Error fetching data for metricset windows.perfmon: failed reading counters: failed collecting counter values: No data to return.
# This config works when providing only one instance "_Total"
- module: windows
  period: 60s
  metricsets:
  - perfmon
  ...
  perfmon.queries:
  - object: "SMB Server Shares"
  instance: "_Total"
  ...

# But when we change it to get all the instances, we get the errors explained above:
- module: windows
  period: 60s
  metricsets:
  - perfmon
  ...
    perfmon.queries:
    - object: "SMB Server Shares"
    instance: "*"
    ...

instance: "*" works for something like "LogicalDisk" but not for "SMB Server Shares" it seems.
I think that the main reason for this is that windows has a strange notation for server shares: \\*\C$ for example.
The instances contain * and \, so I suppose this feature is currently not supported.

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label May 4, 2022
@ruflin ruflin added the Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team label May 4, 2022
@elasticmachine
Copy link
Collaborator

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

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label May 4, 2022
@willemdh
Copy link

willemdh commented Oct 4, 2022

Can someone tackle this issue please. Perfmon monitoirng is such a basic functionality and it currrently does NOT work for months already....

@willemdh
Copy link

The silence at Elastic is deafening lately. Some feedback after all these months would have been nice.

@andrewkroh andrewkroh added the Metricbeat Metricbeat label Jan 9, 2023
@willemdh
Copy link

15 months later and still no way to index perfmon counters containing wildcards... :(

@MakoWish
Copy link
Contributor

I'll see your 15 months and raise you another three months.

@nimarezainia
Copy link
Contributor

We are currently unable to commit to a time frame for addressing this issue due to other higher priority development activities. However if anyone has a PR for this issue that would allow you to move forward, we would certainly prioritize its review and have it merged.

@jlind23 @pierrehilbert

@willem-dhaese
Copy link

willem-dhaese commented May 15, 2024

2 years later and we still cannot index perfmon metrics in a stable way. I guess the same issue exists in Agent?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Metricbeat Metricbeat Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team
Projects
None yet
Development

No branches or pull requests

8 participants