Skip to content

Conversation

@zhiweiv
Copy link
Contributor

@zhiweiv zhiweiv commented Mar 17, 2020

What this PR does / why we need it:
Current implementation usually takes about 500ms to finish, the new one costs almost zero. It comes from https://github.com/kubernetes/kubernetes/blob/master/pkg/kubelet/util/util_windows.go#L119.

I have verified they generate same time value.

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged):

@jaer-tsun
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jaer-tsun jaer-tsun requested a review from ashvindeodhar March 18, 2020 18:44
@matmerr
Copy link
Member

matmerr commented Apr 2, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

4 participants