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
Enable profiling by default in the scheduler #84835
Enable profiling by default in the scheduler #84835
Conversation
@denkensk: You must be a member of the kubernetes/milestone-maintainers GitHub team to set the milestone. If you believe you should be able to issue the /milestone command, please contact your and have them propose you as an additional delegate for this responsibility. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@@ -111,4 +111,10 @@ func SetDefaults_KubeSchedulerConfiguration(obj *kubeschedulerconfigv1alpha1.Kub | |||
defaultPodMaxBackoffSeconds := int64(10) | |||
obj.PodMaxBackoffSeconds = &defaultPodMaxBackoffSeconds | |||
} | |||
|
|||
// Enable profiling by default in the scheduler. | |||
enableProfiling := true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move it inside the if statement like the others.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks ~ Done
enableProfiling := true | ||
if obj.EnableProfiling == nil { | ||
obj.EnableProfiling = &enableProfiling | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
set EnableContentionProfiling
to true by default if EnableProfiling
is true:
if *obj.EnableProfiling && obj.EnableContentionProfiling == nil {
enableContentionProfiling := true
obj.EnableContentionProfiling = &enableContentionProfiling
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks ~ Done
ae63bd5
to
2aed1fd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Looks good, thanks
/test pull-kubernetes-kubemark-e2e-gce-big |
@@ -111,4 +111,16 @@ func SetDefaults_KubeSchedulerConfiguration(obj *kubeschedulerconfigv1alpha1.Kub | |||
defaultPodMaxBackoffSeconds := int64(10) | |||
obj.PodMaxBackoffSeconds = &defaultPodMaxBackoffSeconds | |||
} | |||
|
|||
// Enable profiling to true by default in the scheduler. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: Enable profiling by default in the scheduler
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks~~ Done
obj.EnableProfiling = &enableProfiling | ||
} | ||
|
||
// Enable ContentionProfiling to true by default if EnableProfiling is true. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: Enable contention profiling by default if profiling is enabled.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks~~ Done
2aed1fd
to
06842bf
Compare
06842bf
to
e85ebcc
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ahg-g, denkensk The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind feature
/sig scheduling
/priority important-soon
/milestone v1.17
What this PR does / why we need it:
It is useful when debugging the performance of a running cluster. API server already enables profiling by default.
Which issue(s) this PR fixes:
Fixes #84799
Does this PR introduce a user-facing change?:
/assign @ahg-g