Improve the node package list updates#17704
Conversation
8117d4c to
fa387db
Compare
|
/hold |
|
/hold |
a635d2e to
f85467e
Compare
Signed-off-by: Ciprian Hacman <ciprian@hakman.dev>
f85467e to
26f7013
Compare
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hakman The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
| // If it is newer than 10 minutes, we skip apt-get update. | ||
| stat, err := os.Stat("/var/lib/dpkg/status") | ||
| // If the package manager update was less than 10 minutes ago, skip updating the package list. | ||
| if d.IsDebianFamily() && time.Since(packageManagerLastUpdated) > 10*time.Minute { |
There was a problem hiding this comment.
Why 10 min ? I think apt update take less than 5 min to run.
There was a problem hiding this comment.
This is meant to run apt-get update every 10 min (arbitrary number). I expect bootstrap to finish much faster, but seems reasonable.
|
/lgtm |
|
/override pull-kops-e2e-k8s-gce-cilium |
|
@hakman: Overrode contexts on behalf of hakman: pull-kops-e2e-k8s-gce-cilium DetailsIn 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-sigs/prow repository. |
Addressing code review comments from #17690.
/cc @justinsb @ameukam @rifelpet