Unverified Commit a44c2b9e authored by k8s-ci-robot's avatar k8s-ci-robot Committed by GitHub

Merge pull request #70035 from liggitt/retain-quota-metadata

Avoid dropping resourcequota metadata in controller
parents 79dab474 fdd87a88
...@@ -347,24 +347,17 @@ func (rq *ResourceQuotaController) syncResourceQuota(resourceQuota *v1.ResourceQ ...@@ -347,24 +347,17 @@ func (rq *ResourceQuotaController) syncResourceQuota(resourceQuota *v1.ResourceQ
// Create a usage object that is based on the quota resource version that will handle updates // Create a usage object that is based on the quota resource version that will handle updates
// by default, we preserve the past usage observation, and set hard to the current spec // by default, we preserve the past usage observation, and set hard to the current spec
usage := v1.ResourceQuota{ usage := resourceQuota.DeepCopy()
ObjectMeta: metav1.ObjectMeta{ usage.Status = v1.ResourceQuotaStatus{
Name: resourceQuota.Name, Hard: hardLimits,
Namespace: resourceQuota.Namespace, Used: used,
ResourceVersion: resourceQuota.ResourceVersion,
Labels: resourceQuota.Labels,
Annotations: resourceQuota.Annotations},
Status: v1.ResourceQuotaStatus{
Hard: hardLimits,
Used: used,
},
} }
dirty = dirty || !quota.Equals(usage.Status.Used, resourceQuota.Status.Used) dirty = dirty || !quota.Equals(usage.Status.Used, resourceQuota.Status.Used)
// there was a change observed by this controller that requires we update quota // there was a change observed by this controller that requires we update quota
if dirty { if dirty {
_, err = rq.rqClient.ResourceQuotas(usage.Namespace).UpdateStatus(&usage) _, err = rq.rqClient.ResourceQuotas(usage.Namespace).UpdateStatus(usage)
return err return err
} }
return nil return nil
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment