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
fix: log info or error print #3410
base: master
Are you sure you want to change the base?
Conversation
googs1025
commented
Apr 13, 2024
- found some typo or code style err
- fix log print level
d3b8de4
to
0c42e04
Compare
/restart |
f444538
to
3888c02
Compare
8318f99
to
f763733
Compare
/lgtm Please rebase and squash commits to only one. |
@lowang-bh done |
/lgtm |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: lowang-bh The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
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
New changes are detected. LGTM label has been removed. |
Signed-off-by: googs1025 <googs1025@gmail.com> fix: log level Signed-off-by: googs1025 <googs1025@gmail.com>
@@ -181,7 +181,7 @@ func (alloc *Action) allocateResourcesForTasks(tasks *util.PriorityQueue, job *a | |||
klog.V(3).Infof("There are <%d> nodes for Job <%v/%v>", len(ssn.Nodes), job.Namespace, job.Name) | |||
|
|||
if err := ssn.PrePredicateFn(task); err != nil { | |||
klog.V(3).Infof("PrePredicate for task %s/%s failed for: %v", task.Namespace, task.Name, err) | |||
klog.V(4).Infof("PrePredicate for task %s/%s failed for: %v", task.Namespace, task.Name, err) |
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.
log levels are different in this file, we'd better unify them to the same, but whether to 3 or 4 need to be discussed, so does other file like recalim: )