• Kubernetes Submit Queue's avatar
    Merge pull request #33146 from MrHohn/kubedns-readiness · 9e4ba186
    Kubernetes Submit Queue authored
    Automatic merge from submit-queue
    
    Tune down initialDelaySeconds for readinessProbe.
    
    Fixed #33053.
    
    Tuned down the `initialDelaySeconds`(original 30s) for readiness probe to 3 seconds and `periodSeconds`(default 10s) to 5 seconds to shorten the initial time before a dns server pod being exposed. This configuration passed DNS e2e tests and did not even hit any readiness failure(for kube-dns) with a GCE cluster with 4 nodes during the experiments.
    
    For scaling out kube-dns servers, it took less than 10s for servers being exposed after they appeared as running, which is much faster than 30+s(the original cost).
    
    `failureThreshold` is left as default(3) and it would not lead to restart because the status of readiness probe would only affect whether endpoints being exposed in service or not(in the dns service point of view). According to the implementation of [prober](https://github.com/kubernetes/kubernetes/blob/master/pkg/kubelet/prober/worker.go), the number of retries for readiness probe is unbounded. Hence there is no obvious effect if the readiness probe fail several times in the beginning. 
    
    The state machine of prober could be illustrated with below figure:
    
    ![drawing](https://cloud.githubusercontent.com/assets/8681801/18693503/fb4466dc-7f56-11e6-8671-0a14c4835d24.jpeg)
    
    I want to see the e2e result of this PR for further evaluation.
    
    @thockin @bprashanth
    9e4ba186
Name
Last commit
Last update
..
coreos Loading commit data...
debian Loading commit data...
gci Loading commit data...
trusty Loading commit data...
config-common.sh Loading commit data...
config-default.sh Loading commit data...
config-test.sh Loading commit data...
configure-vm.sh Loading commit data...
delete-stranded-load-balancers.sh Loading commit data...
list-resources.sh Loading commit data...
upgrade.sh Loading commit data...
util.sh Loading commit data...