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

Use async DiagnosticsClient APIs #790

Merged

Conversation

jander-msft
Copy link
Member

These changes rely on the following PR from the diagnostics repo: dotnet/diagnostics#2350

These changes only switch to using the async APIs on the DiagnosticsClient class. Subsequent changes to timeout on querying the process list will be made in a different PR.

@jander-msft jander-msft requested a review from a team as a code owner August 27, 2021 16:39
@jander-msft jander-msft force-pushed the dev/jander/diagnostics-client-async branch from 5b53c6d to 8d06a15 Compare September 2, 2021 03:55
@jander-msft
Copy link
Member Author

jander-msft commented Sep 2, 2021

/azp run dotnet-monitor-ci

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jander-msft jander-msft merged commit 534983c into dotnet:main Sep 3, 2021
@jander-msft jander-msft deleted the dev/jander/diagnostics-client-async branch September 3, 2021 01:13
jander-msft added a commit that referenced this pull request Sep 3, 2021
* Add live metrics api (#750)

* Add live metrics api

* Fix invalid metric value

* PR Feedback

* Rename livemetrics to collectmetrics

* Use async DiagnosticsClient APIs. (#790)

* Update dependencies from https://github.com/dotnet/diagnostics build 20210902.1 (#819)

[main] Update dependencies from dotnet/diagnostics

* Update dependencies from https://github.com/dotnet/aspnetcore build 20210903.2 (#822)

VS.Redist.Common.AspNetCore.SharedFramework.x64.6.0
 From Version 6.0.0-rc.2.21452.1 -> To Version 6.0.0-rc.2.21453.2

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>

* Update dependencies from https://github.com/dotnet/runtime build 20210902.24 (#821)

VS.Redist.Common.NetCore.SharedFramework.x64.6.0
 From Version 6.0.0-rc.2.21452.2 -> To Version 6.0.0-rc.2.21452.24

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>

* Update dependencies from https://github.com/dotnet/arcade build 20210902.4 (#820)

Microsoft.DotNet.XUnitExtensions , Microsoft.DotNet.Arcade.Sdk
 From Version 6.0.0-beta.21451.3 -> To Version 7.0.0-beta.21452.4

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>

* Consolidate task and completion source cancellation methods. (#818)

Add TaskCompletionSourceExtensions to dotnet-monitor.
Move WithCancellation extension method to dotnet-monitor.

* Documentation for metrics (#815)

Also fixup some examples for Bearer

Co-authored-by: Wiktor Kopec <[email protected]>
Co-authored-by: dotnet-maestro[bot] <42748379+dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants