pr feedback

added intermittent mem spike endpoint

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update README.md

Update README.md

Update README.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

Update intermittent_memory_spike.md

initial add

[dotnet-counters] Make refresh-interval optional (#253)

* --refresh-interval is now optional

* Make the default refresh-interval 1

Restore commit information to native images (#250)

Terminate dotnet-trace when the runtime closes the eventpipe

Fix a small typo in memory leak tutorial (#249)

Overall cleanup for build system - part 1 (#247)

* Remove dotnet-analyze from build and CreateVersionFile
* Rename NuGet config file to be compliant with casing
* Use msbuild command in place of hardcoded paths
* Add details file
* Consolidate versions into a single file
* Over all cleanup of project files, properties, and targets
* Little workaround for older version of Arcade around license
* Change format of versioning resources for readability
* Update XUnit versions

Merge pull request #245 from MarioHewardt/master

Initial scenario based tutorial

fix pr feedback

Moving counters doc to where all other documentations live (#246)

Initial scenario based tutorial

Update installation instruction for dotnet-counters for P5 (#244)

Dotnet counters doc (#219)

* some documentation updates to dotnet-counters

* Readme under dotnet-counters directory

* some cleanup

* Adding example for filtering'

* fix list cmd

* Adding some more description in example usage per PR feedback

* PR feedback

* Give some example counters to intro section

* Some syntax fix

* Add dotnet-counters doc to main README

Fix missing metadata in Linux core dumps. (#229)

* Fix missing metadata in Linux core dumps.

Issue #https://github.com/dotnet/diagnostics/issues/56

Both Windows and Linux minidumps don't save all the assemblies' metadata which can cause
stack traces to display !Unknown. On Windows most debuggers like Windbg and VS can load
the module from file for this kind of dumps. On Linux lldb/SOS doesn't load the native
modules and really doesn't know anything about the managed assemblies.

Add metadata callback to datatarget and it's implementation in SOS.NETCore.

Add IsMetadataMemory() and the metadata region list. This is because lldb on core dumps
returns 0's on missing metadata memory reads instead of an error.

Merge pull request #236 from dotnet/juhoyosa/rollforward-major

Add runtime config template for major rollforward

Update docs for preview 5. (#238)

Update the tools version to 1.0.3-preview5.19251.2.

Add runtime config template for major rollforward

Ignore any provider specified via --profile option if they are already specified with --providers option (#224)

Preview5 doc update. (#225)

Add Release notes for dotnet-dump.

Remove --add-source in installation instructions.

Fix issue with dotnet-counters not being able to target user-defined EventSource (#222)

Fixing list command (#218)

Merge pull request #214 from mikem8361/fromrelease

Commits from release/3.0.

Merge pull request #213 from mikem8361/updoc

Update tool installation docs to 1.0.3-preview5.19228.1

Merge pull request #211 from mikem8361/fixazdo

Fix artifacts.

Merge pull request #209 from mikem8361/fixpush

Fix artifact publishing.

Fix artifact publishing.

Merge pull request #208 from mikem8361/fixart

Fix naming of artifacts.

Fix naming of artifacts.

Add Convert command (#196)

* Add Convert command:
* make default format for all platforms 'netperf'
* update spec to refelct impl

* Simplify try/catch to using statements in converter code

* Account for scenario when someone doesn't specify an output filename and the input trace file doesn't have the default name

* Grammar nit

* Update enum value to use Pascal case
* remove -f shorthand for format option

* * remove try/catch
* use ExistingOnly argument extension
* remove unnecessary Arity qualifications

Add command handler to dotnet-counters (#200)

Bringing dotnet counters up to spec [part 1] (#199)

* fix build warnings

* For some reason we dont have CounterType field inside the payload...

* support multiple providers

* Filtering

* Fix broken alignment on list command

* spaces

* support filtering for an entire provider

* Removing -r option

* Add filter for System.Runtime for default monitoring session

Make dotnet-counter stream values in real time (#197)

* Streaming counters

* DisplayName stuff

* fix build

* few demo changes

* parsing command-line provided counter list works now

* No more string parsing!!!

* some more fixes in writer

* update license header, display rates

* Removing useless files

* Use known displaynames for known providers

* cleanup

* spaces

* more cleanup

* more cleanup

* PR feedback

Renaming non-spec options on dotnet-trace (#192)

* Rename the "profiles" verb to "list-profiles"
* Rename the "endpoints" verb to "list-processes"
* Move pack option/verb to future.
* Move the 'g' command (capture GC heap snapshot) to future.
* Updating docs with missing functionality.

Not ready on Non-Windows platform. (#178)

Not ready on Non-Windows platform for **.NET 3.0 Preview5**. This is related to this bug fix: https://github.com/dotnet/coreclr/pull/24198, which will be included in **Preview6**.
The issue manifests when EventPipe is enabled without LTTng. In this case, GC events are disabled after the first GC because LTTng probes are disabled.

Merge pull request #187 from mikem8361/fixlldbdoc

Fix lldb build instructions and scripts.

Fix lldb build instructions and scripts.

For CentOS 7, Fedora 24 and OpenSUSE.

Typos (#177)

* comamnd -> command
* commited -> committed
* convinient -> convenient
* currnet -> current
* deafualts -> defaults
* defualts -> defaults
* dianostics -> diagnostics
* interogates -> interrogates
* precendent -> precedent
* Redit -> Redis
* if -> is

Merge pull request #179 from mikem8361/fixcentos7

Fix the build-install-lldb.sh network failure.

Fix the build-install-lldb.sh network failure.

Changed downloading binutils from sourceware.org to http://ftp.gnu.org/gnu/binutils.

Merge pull request #176 from mikem8361/vs2019fix

Fix building with vs2019 only.

Fix building with vs2019 only.

Updating gc/gc-collect options to map PerfView flags. (#175)

Merge pull request #164 from github-john-doe/typos

Typos

[dotnet-trace] Small changes (#173)

- Replace hardcoded numbers with existing/equivalent enum.
- Move non-common options to Collect, and started plumbing pack options.
- Remove comment.
- Remove pack changes

Speedscope format (#171)

* Incorporate code from closed PR #114 to enable speedscope output:
* adds format option

* Actually add the converter...

* update dotnet-trace instructions

* Expanded on output format section of instructions

* Change protection level of TraceFileFormatConverter

* Updates in response to feedback

* Update src/Tools/dotnet-trace/TraceFileFormatConverter.cs

Co-Authored-By: josalem <josalem@microsoft.com>
* Update src/Tools/dotnet-trace/CommandLine/Options/CommonOptions.cs

Co-Authored-By: josalem <josalem@microsoft.com>
* Fix typos

[dotnet-trace] Adding --profile options and minor fixes. (#172)

* OnStop: rename pid -> processId
* Adding a Versions.props
* Update Collect output format. Added the process main module path.
* Adding profiles verb
* Adding --profile option.
* up-to-spec
* Renaming few ETW instances to EventPipe.

Merge pull request #169 from mikem8361/gcheap

Add a destructor for sos::GCHeap

Add a destructor for sos::GCHeap

Integer Conversion Issues from GCC

From PR https://github.com/dotnet/coreclr/pull/23934 in coreclr.

dotnet-trace instructions draft and misc. (#161)

* Move instructions under documentation.
* Update docs with current help string, new option, and repo readme.
* Making dotnet-trace a packable tool
* Updating default CircularBufferSizeMB to 256MB.
* Making RuntimeClient packable.
* Enable `ports` as `endpoints` for end-users
* processId must be greater than zero.

persistance -> persistence

overriden -> overridden

migth -> might

existant -> existent

debugge -> debuggee

comparsion -> comparison

commnd -> command

Commited -> Committed

Attrbute -> Attribute

avaliable -> available

Merge pull request #162 from mikem8361/docup

Update global tool versions in docs.

Update global tool versions in docs.

Change ResetCurrentConsoleLine() to use explicit ANSI escape codes  (#159)

* Change ResetCurrentConsoleLine() to use explicit ANSI escape codes to rewrite line

* Add System.CommandLine.Rendering and use it to determine when we can use ANSI escape codes.

* Add fallback if vterm is not supported

* Changes in resposne to feedback:
* actually assign prevBufferWidth
* reset cursor to specific row and column instead of just column

Merge pull request #160 from mikem8361/preview5

Misc doc and version updates for preview5

Misc doc and version updates for preview5.

Reconcile dotnet-dump spec/implementation. Issue #126.

Add  ctrl c handler and modify enter key handler to be non-blocking (#155)

* Add  ctrl c handler and modify enter key handler to be non-blocking

[dotnet-trace] More changes to make tool ready for preview5 (#156)

- Updating tests.
- Restore test project to run as netcoreapp3.0
  - EventPipe with IPC is a 3.0 feature, it does not work in older releases.
- Make dotnet-trace up to spec.
- Fix some markdown syntaxt errors
- Fix typo
- `--process-id` maps to `processId`
- Reduce named pipe connect timeout from 20 sec to 3 sec.
- Update --providers description.

[dotnet-trace] Making tool up to spec. (#154)

* Making dotnet-trace more up to spec.
* Move the start/stop commands under debug.
* Basic data sanitazation.
* Properly clear the current console line.
* Remove debug code, and some renaming to match functionality and runtime.
* Update tests.

[dotnet-trace] Minor updates to match recent CoreClr changes, error handling, and tests (#153)

- Move common csproj properties to a common props file
- Add more tracing file extensions to .gitignore
- Update runtime client to enable streaming on non-Windows platforms.
- Adding helper bash scripts
- Update RunTimeClient
  - Updated `DiagnosticMessageType` private -> public
  - Ignore `sessionId == 0` on the "Disable" command
  - Added a `SessionLoggingType` to be added to the command protocol
  - Removed `multi file sec` related functionality
  - Ignore buffer size of zero
  - Added some initial tests that need to be moved to CoreClr.
- Update dotnet-counters with changes in RuntimeClient library
  - dotnet-counters.csproj : error NU1605: Detected package downgrade: Microsoft.Diagnostics.Tracing.TraceEvent from 2.0.40 to 2.0.38.
  - Use new helper interface.

Remove call to Bind() from EventPipeClient (#149)

* Remove Bind from EventPipeClient

* cleanup

Make dotnet-counters use IPC to enable/disable counters (#148)

* Modify dotnet-counters to use IPC to enable/disable counters

* Add parsing for counterList command line arg

* remove unused using

* move scope of defaultProvider

* Update RuntimeClient namespace

Make dotnet-counters use UseDefaults() with System.CommandLine (#146)

[dotnet-trace] Update namespaces based on previous PR feedback. (#145)

[dotnet-trace] Updated dotnet-trace to use IPC to communicate with the runtime (#143)

* Microsoft.Diagnostics.Tools.Collect -> Microsoft.Diagnostics.Tools.Trace
* Updated dotnet-trace with the following options:
  start - Starts tracing
  stop - Stops tracing
* Ignore netperf files.
* Naive stream.
* Helper scripts.
* Update the `diagnostics.sln` file.
* Small changes in API

Merge pull request #144 from mikem8361/docverup

Update tool install version to 1.0.2-preview3.19170.2

Update tool install version to 1.0.2-preview3.19170.2

Merge pull request #142 from mikem8361/miscstuff

Misc changes