Skip to content
This repository was archived by the owner on Oct 6, 2023. It is now read-only.

Conversation

@yanggangtony
Copy link
Contributor

/kind cleanup

Hey @everpeace .
I unintentionally search this repo from the k/k main repos.
I recently are learn and research the k8s code .

Just do some clean code for kube-throttler.
If this pr is make something mess , please feel free to told me..
Thanks.

Signed-off-by: yanggang <[email protected]>
Copy link
Owner

@everpeace everpeace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yanggangtony Thank you very much for cleaning up my old code!

)

scheduleInformerFactory.Start(ctx.Done())
scheduleInformerFactory.WaitForCacheSync(ctx.Done())
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

if err != nil {
utilruntime.HandleError(errors.Wrapf(err, "Failed to unreserve pod %s/%s in ThrottleController", pod.Namespace, pod.Name))
}
pod.GetName()
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@everpeace everpeace merged commit 397216b into everpeace:master May 10, 2023
@github-actions github-actions bot mentioned this pull request May 10, 2023
@yanggangtony
Copy link
Contributor Author

Thanks

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants