Skip to content
This repository was archived by the owner on Mar 17, 2021. It is now read-only.

Initial commit of perfcollect script - #1

Merged
brianrob merged 1 commit into
dotnet:masterfrom
brianrob:perfcollect
Oct 28, 2015
Merged

Initial commit of perfcollect script#1
brianrob merged 1 commit into
dotnet:masterfrom
brianrob:perfcollect

Conversation

@brianrob

Copy link
Copy Markdown
Member

Contains the initial commit for the perfcollect script, which is aimed at making .NET performance data and collection with perf_event easier.

@brianrob

Copy link
Copy Markdown
Member Author

@karelz, can you please review this?

Comment thread src/performance/perfcollect/perfcollect Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: ... you WILL be looking at ...

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, thanks. Fixed.

@karelz

karelz commented Oct 28, 2015

Copy link
Copy Markdown
Member

LGTM (with just minor comments above - feel free to discard them if you disagree).

@brianrob

Copy link
Copy Markdown
Member Author

Thanks for the review @karelz.

brianrob added a commit that referenced this pull request Oct 28, 2015
Initial commit of perfcollect script
@brianrob
brianrob merged commit e6d3e53 into dotnet:master Oct 28, 2015
@brianrob
brianrob deleted the perfcollect branch October 28, 2015 19:10
schaabs pushed a commit that referenced this pull request Oct 29, 2015
Adding initial port of reliability tooling.  Some tooling is not port…
mleenhardt added a commit to mleenhardt/corefx-tools that referenced this pull request Apr 4, 2018
`uname -a` doesn't contain "Debian" on Jessie or Stretch.

Output from a microsoft/dotnet:2.0.6-sdk-2.1.104 container using stretch as its base image:
`uname -a` -> "Linux 1a46aee325a3 4.9.87-linuxkit-aufs dotnet#1 SMP Wed Mar 14 15:12:16 UTC 2018 x86_64 GNU/Linux"
`cat /etc/debian_version` -> "8.10"
`cat /etc/issues` -> "Debian GNU/Linux 8 \n \l"

Output from a microsoft/dotnet:2.0.6-sdk-2.1.104-jessie container using jessie as its base image:
`uname -a` -> "Linux ae834cf8bd36 4.9.87-linuxkit-aufs dotnet#1 SMP Wed Mar 14 15:12:16 UTC 2018 x86_64 GNU/Linux"
`cat /etc/debian_version` -> "9.4"
`cat /etc/issues` -> "Debian GNU/Linux 9 \n \l"
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants