Change:
Immediately after this test was reenabled in we got huge numbers of parallel flakes in . This might be a coincidence. I'll make a PR to move it to flaky. If things haven't improved after it's moved then I'll revert, else I'll assign this issue to !
Do you have any logs? references , but that has nothing to do with NFS nor PersistentVolumes.
. It's pretty clear this test caused the flakes we were seeing.
do you have any logs?
It hasn't flaked since your fix went in, but we should keep the issue open until it's moved out of the flaky suite.
The fix Joe Finney developed was merged on 5/30. The signature for the problem in the log file referenced here is: 6 18:56:07.895: Expected PersistentVolume to be bound, but got nil ClaimRef: kind:"" apiVersion:""` It looks like from this dashboard the last instance of this error log signature for the test PersistentVolumes NFS test to the flaky suite. was on 5/30 right around the time the code was merged. I found the off the dashboard at https://k8s- Since then the error has not occurred with that signature since I am tempted to close this bug, but keep the the test suite as marked "Flaky". The merge bot should start flagging the new bugs for this test going forward.
If it's no longer flaky we should move it out of the flaky suite. While it's in the flaky suite it won't block merges if it breaks and mergebot won't open bugs for it. I'm worried that if we just close this issue it'll sit in the flaky suite and break and noone will notice for a long time.
This bug has not been reproduced for past 2 months, can we close it?
It looks like the test is still in the flaky suite, although with a different name. Once it's moved out of there then this can be closed.
[FLAKE-PING] This flaky-test issue would love to have more attention...
[FLAKE-PING] This flaky-test issue would love to have more attention...
Not reproduced for three months, closing.

Discussion:
NodeSyncPeriod:                    unversioned.Duration{Duration: 10 * time.Second}, ResourceQuotaSyncPeriod:           unversioned.Duration{Duration: 5 * time.Minute}, NamespaceSyncPeriod:               unversioned.Duration{Duration: 5 * time.Minute}, <del> PVClaimBinderSyncPeriod:           unversioned.Duration{Duration: 10 * time.Minute}, </del> <ins> PVClaimBinderSyncPeriod:           unversioned.Duration{Duration: 15 * time.Second}, </ins> HorizontalPodAutoscalerSyncPeriod: unversioned.Duration{Duration: 30 * time.Second}, DeploymentControllerSyncPeriod:    unversioned.Duration{Duration: 30 * time.Second}, MinResyncPeriod:                   unversioned.Duration{Duration: 12 * time.Hour},

--pv-recycler-pod-template-filepath-hostpath="": The file path to a pod definition used as a template for HostPath persistent volume recycling. This is for development and testing only and will not work in a multi-node cluster. --pv-recycler-pod-template-filepath-nfs="": The file path to a pod definition used as a template for NFS persistent volume recycling --pv-recycler-timeout-increment-hostpath=30: the increment of time added per Gi to ActiveDeadlineSeconds for a HostPath scrubber pod.  This is for development and testing only and will not work in a multi-node cluster. <del> --pvclaimbinder-sync-period=10m0s: The period for syncing persistent volumes and persistent volume claims </del> <ins> --pvclaimbinder-sync-period=15s: The period for syncing persistent volumes and persistent volume claims </ins> --replicaset-lookup-cache-size=4096: The the size of lookup cache for replicatsets. Larger number = more responsive replica management, but more MEM load. --replication-controller-lookup-cache-size=4096: The the size of lookup cache for replication controllers. Larger number = more responsive replica management, but more MEM load. --resource-quota-sync-period=5m0s: The period for syncing quota usage status in the system

--terminated-pod-gc-threshold=12500: Number of terminated pods that can exist before the terminated pod garbage collector starts deleting terminated pods. If <= 0, the terminated pod garbage collector is disabled. ``` <del> ###### Auto generated by spf13/cobra on 26-May-2016 </del> <ins> ###### Auto generated by spf13/cobra on 27-May-2016 </ins> <!-- BEGIN MUNGE: GENERATED_ANALYTICS -->

func NewPersistentVolumeControllerOptions() PersistentVolumeControllerOptions { return PersistentVolumeControllerOptions{ <del> PVClaimBinderSyncPeriod: 10 * time.Minute, </del> <ins> PVClaimBinderSyncPeriod: 15 * time.Second, </ins> VolumeConfigFlags: VolumeConfigFlags{ // default values here PersistentVolumeRecyclerMaximumRetry:             3,