-
Notifications
You must be signed in to change notification settings - Fork 7.6k
Add back Get-Counter
cmdlet for Windows
#10933
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
Conversation
@sdwheeler new fwlink for Get-Counter? |
@SteveL-MSFT https://go.microsoft.com/fwlink/?linkid=2109647
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found https://github.com/microsoft/Tx. It seems work with blg.
@iSazonov nice find, seems that Tx is using a different set of APIs to read the log, not sure if there is a different public api to write the binary log. I suggest we defer this for now. |
Maybe open a tracking issue if you plan to investigate this? |
@SteveL-MSFT Can you run a coordinated package build and send me a Direct Message with the build information? Let's verify this doesn't cause any new compliance issues. |
src/Microsoft.PowerShell.Commands.Diagnostics/Microsoft.PowerShell.Commands.Diagnostics.csproj
Show resolved
Hide resolved
🎉 Handy links: |
PR Summary
Reviewed the APIs being used.
Get-Counter
can work with just public APIs published in pdh.h.Import-Counter
andExport-Counter
rely on undocumented APIs so those are removed. The PDH apis are really old so I don't believe there's any chance to get those APIs or the binary log format documented so removed that code.Get-Counter -ListSet
has Description property as empty because it isn't supported by PDH apis.PR Checklist
.h
,.cpp
,.cs
,.ps1
and.psm1
files have the correct copyright headerWIP:
or[ WIP ]
to the beginning of the title (theWIP
bot will keep its status check atPending
while the prefix is present) and remove the prefix when the PR is ready.Get-Counter
cmdlet for PS7 MicrosoftDocs/PowerShell-Docs#5020