- 
                Notifications
    
You must be signed in to change notification settings  - Fork 8.1k
 
          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-Countercan work with just public APIs published in pdh.h.Import-CounterandExport-Counterrely 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 -ListSethas Description property as empty because it isn't supported by PDH apis.PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:or[ WIP ]to the beginning of the title (theWIPbot will keep its status check atPendingwhile the prefix is present) and remove the prefix when the PR is ready.Get-Countercmdlet for PS7 MicrosoftDocs/PowerShell-Docs#5020