Skip to content

Commit ecd1393

Browse files
committed
Fix bug in ComputeDomain object being written to mutation cache
Signed-off-by: Kevin Klues <[email protected]>
1 parent 11606fd commit ecd1393

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

cmd/compute-domain-daemon/computedomain.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -411,7 +411,8 @@ func (m *ComputeDomainManager) removeNodeFromComputeDomain(ctx context.Context)
411411
// Update status and (upon success) store the latest version of the object
412412
// (as returned by the API server) in the mutation cache.
413413
newCD.Status.Nodes = updatedNodes
414-
if _, err := m.config.clientsets.Nvidia.ResourceV1beta1().ComputeDomains(newCD.Namespace).UpdateStatus(ctx, newCD, metav1.UpdateOptions{}); err != nil {
414+
newCD, err = m.config.clientsets.Nvidia.ResourceV1beta1().ComputeDomains(newCD.Namespace).UpdateStatus(ctx, newCD, metav1.UpdateOptions{})
415+
if err != nil {
415416
return fmt.Errorf("error removing node from ComputeDomain status: %w", err)
416417
}
417418
m.mutationCache.Mutation(newCD)

0 commit comments

Comments
 (0)