Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add a flag allowing contention profiling of the API server#36756

Merged
k8s-github-robot merged 1 commit into
kubernetes:masterfrom
gmarek:contention
Nov 15, 2016
Merged

Add a flag allowing contention profiling of the API server#36756
k8s-github-robot merged 1 commit into
kubernetes:masterfrom
gmarek:contention

Conversation

@gmarek
Copy link
Copy Markdown
Contributor

@gmarek gmarek commented Nov 14, 2016

Useful for performance debugging.

cc @smarterclayton @timothysc @lavalamp

Add a flag allowing contention profiling of the API server

This change is Reviewable

@gmarek gmarek added area/apiserver release-note Denotes a PR that will be considered when it comes time to generate release notes. labels Nov 14, 2016
@k8s-github-robot k8s-github-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Nov 14, 2016
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

Jenkins GCE e2e failed for commit c97633b. Full PR test history.

The magic incantation to run this job again is @k8s-bot cvm gce e2e test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@smarterclayton
Copy link
Copy Markdown
Contributor

lgtm

1 similar comment
@wojtek-t
Copy link
Copy Markdown
Member

lgtm

@wojtek-t wojtek-t added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Nov 15, 2016
@wojtek-t wojtek-t added this to the v1.5 milestone Nov 15, 2016
@gmarek
Copy link
Copy Markdown
Contributor Author

gmarek commented Nov 15, 2016

@saad-ali - it's a very safe change that is needed for performance debugging.

@gmarek
Copy link
Copy Markdown
Contributor Author

gmarek commented Nov 15, 2016

@k8s-bot gce test this issue: #IGNORE
timeout

@k8s-github-robot
Copy link
Copy Markdown

@k8s-bot test this [submit-queue is verifying that this PR is safe to merge]

@k8s-github-robot
Copy link
Copy Markdown

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit b2cd67c into kubernetes:master Nov 15, 2016
@timothysc
Copy link
Copy Markdown
Contributor

/cc @jeremyeder

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

Labels

area/apiserver lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants