dedup in updatePodStatus (#830)

Co-authored-by: Brian Goff <cpuguy83@gmail.com>
This commit is contained in:
Weidong Cai
2020-06-16 05:35:14 +08:00
committed by GitHub
parent 05fc1a43be
commit 2398504d08

View File

@@ -209,6 +209,9 @@ func (pc *PodController) updatePodStatus(ctx context.Context, podFromKubernetes
kPod.Lock()
podFromProvider := kPod.lastPodStatusReceivedFromProvider.DeepCopy()
kPod.Unlock()
if cmp.Equal(podFromKubernetes.Status, podFromProvider.Status) {
return nil
}
// We need to do this because the other parts of the pod can be updated elsewhere. Since we're only updating
// the pod status, and we should be the sole writers of the pod status, we can blind overwrite it. Therefore
// we need to copy the pod and set ResourceVersion to 0.