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
Get Windows Process time correctly #97491
Get Windows Process time correctly #97491
Conversation
/sig windows |
/test pull-kubernetes-e2e-azure-disk-windows |
@jsturtevant since this targets |
from that repository (https://github.com/kubernetes/component-base#where-does-it-come-from):
|
did something change? |
You are right! I had the auto publishing flow reversed. So please ignore the above comment. |
/lgtm |
/test pull-kubernetes-e2e-aks-engine-azure-windows |
/milestone v1.21 |
/retest |
/remove-sig api-machinery |
041cf06
to
c874228
Compare
c874228
to
27a8912
Compare
something went wrong with the rebase and removal of bazel. Got a ci error that said to run |
27a8912
to
2c9e440
Compare
/retest |
1 similar comment
/retest |
Hello! I am from the Bug Triage team! Friendly reminder that code freeze is starting March 9th, 2021 (less than 1 week from now). We want to ensure that each PR has a chance to be merged before code freeze. As the PR is tagged for 1.21, is it still planned for this release? |
Thanks for this cleanup /lgtm |
/test pull-kubernetes-e2e-azure-disk-windows |
/approve I'd like to see a test but my ideas for testing this aren't very good |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jsturtevant, lavalamp 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 cleanup
What this PR does / why we need it:
The
process_start_time_seconds
gauge falls back to time.now() but we get an error on Windows. This information is available via the windows golang.org/x/sys/windows package we can get an accurate time.Which issue(s) this PR fixes:
Fixes #97490
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: