• k8s-merge-robot's avatar
    Merge pull request #25263 from jsafrane/devel/adopt-recycle-pod · 62a8394e
    k8s-merge-robot authored
    Automatic merge from submit-queue
    
    volume recycler: Don't start a new recycler pod if one already exists.
    
    Recycling is a long duration process and when the recycler controller is restarted in the meantime, it should not start a new recycler pod if there is one already running.
    
    This means that the recycler pod must have deterministic name based on name of the recycled PV, we then get name conflicts when creating the pod.
    
    Two things need to be changed:
    
    - recycler controller and recycler plugins must pass the PV.Name to place, where the pod is created. This is most of the patch and it should be pretty straightforward.
    
    - create recycler pod with deterministic name and check "already exists" error.
    
    When at it, remove useless 'resourceVersion' argument and make log messages starting with lowercase.
    
    There is an unit test to check the behavior + there is an e2e test that checks that regular recycling is not broken (it does not try to run two recycler pods in parallel as the recycler is single-threaded now).
    62a8394e
Name
Last commit
Last update
..
daemon Loading commit data...
deployment Loading commit data...
endpoint Loading commit data...
framework Loading commit data...
garbagecollector Loading commit data...
gc Loading commit data...
job Loading commit data...
namespace Loading commit data...
node Loading commit data...
persistentvolume Loading commit data...
petset Loading commit data...
podautoscaler Loading commit data...
replicaset Loading commit data...
replication Loading commit data...
resourcequota Loading commit data...
route Loading commit data...
service Loading commit data...
serviceaccount Loading commit data...
volume Loading commit data...
OWNERS Loading commit data...
controller_utils.go Loading commit data...
controller_utils_test.go Loading commit data...
doc.go Loading commit data...
lookup_cache.go Loading commit data...