/pcp

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

pcp-pidstat: initial commit of PCP implementation of pidstat #97

Merged
merged 1 commit into from Jul 6, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@sitaramshelke @natoscott
@sitaramshelke
Contributor

sitaramshelke commented Jul 6, 2016

No description provided.

@natoscott
Contributor

natoscott commented Jul 6, 2016

Nice work Sitaram, looks good!

I'm seeing a failure in the new test qa/1078 - looks like two small things:

  • We need to use -z to provide deterministic timestamps in the test output (I have a fix for this one)
  • The individual processes are output in a different order for me - I guess different python version or different dictionary/hashing algorithm, or something like that. Is there a way to sort the pcp-pidstat output? (e.g. by pid or by top cpu user, or something like that - just to get that deterministic output). I'm not seeing it from a quick man page scan.

cheers.

@natoscott natoscott merged commit 4a102ce into performancecopilot:master Jul 6, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment