Skip to content

Commit b6b10cb

Browse files
authored
Merge pull request #740 from NVIDIA/backport-721-to-release-25.8
[release-25.8] Fix bug in ComputeDomain object being written to mutation cache
2 parents 2ef1f2a + affb6ef commit b6b10cb

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)