Skip to content

Support block profile by pprof webserver#7122

Merged
openshift-bot merged 1 commit intoopenshift:masterfrom
nak3:support-block-by-pprofile
Feb 9, 2016
Merged

Support block profile by pprof webserver#7122
openshift-bot merged 1 commit intoopenshift:masterfrom
nak3:support-block-by-pprofile

Conversation

@nak3
Copy link
Copy Markdown
Contributor

@nak3 nak3 commented Feb 8, 2016

No description provided.

@nak3
Copy link
Copy Markdown
Contributor Author

nak3 commented Feb 8, 2016

Without runtime.SetBlockProfileRate(1), block will show 0 as below.

profiles:
0   block
1358    goroutine
1178    heap
13  threadcreate

@smarterclayton
Copy link
Copy Markdown
Contributor

Lgtm [merge]

@openshift-bot
Copy link
Copy Markdown
Contributor

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link
Copy Markdown
Contributor

Evaluated for origin test up to cd77ecb

@nak3
Copy link
Copy Markdown
Contributor Author

nak3 commented Feb 9, 2016

hack/verify-generated-conversions.sh failed. It looks like this PR fix it.

I rebased this PR.
Could you please run merge again @smarterclayton ?

@openshift-bot
Copy link
Copy Markdown
Contributor

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/4905/) (Image: devenv-rhel7_3374)

@openshift-bot
Copy link
Copy Markdown
Contributor

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/942/)

@smarterclayton
Copy link
Copy Markdown
Contributor

[merge]

@openshift-bot
Copy link
Copy Markdown
Contributor

Evaluated for origin merge up to cd77ecb

openshift-bot pushed a commit that referenced this pull request Feb 9, 2016
@openshift-bot openshift-bot merged commit 48fa43f into openshift:master Feb 9, 2016
@nak3 nak3 deleted the support-block-by-pprofile branch December 6, 2016 00:52
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.

3 participants