Add missing namespace to Get function (#896)

h0tbird · k8s-ci-robot · commit e325dfa6ecd8 · 2019-04-15T11:00:16.000-07:00
diff --git a/pkg/controller/machineset/status.go b/pkg/controller/machineset/status.go
@@ -112,7 +112,7 @@ func updateMachineSetStatus(c client.Client, ms *v1alpha1.MachineSet, newStatus
 			break
 		}
 		// Update the MachineSet with the latest resource version for the next poll
-		if getErr = c.Get(context.Background(), client.ObjectKey{Name: ms.Name}, ms); getErr != nil {
+		if getErr = c.Get(context.Background(), client.ObjectKey{Namespace: ms.Namespace, Name: ms.Name}, ms); getErr != nil {
 			// If the GET fails we can't trust status.Replicas anymore. This error
 			// is bound to be more interesting than the update failure.
 			return nil, getErr

Original file line number	Diff line number	Diff line change
`@@ -112,7 +112,7 @@ func updateMachineSetStatus(c client.Client, ms *v1alpha1.MachineSet, newStatus`
`112`	`112`	`break`
`113`	`113`	`}`
`114`	`114`	`// Update the MachineSet with the latest resource version for the next poll`
`115`		`- if getErr = c.Get(context.Background(), client.ObjectKey{Name: ms.Name}, ms); getErr != nil {`
	`115`	`+ if getErr = c.Get(context.Background(), client.ObjectKey{Namespace: ms.Namespace, Name: ms.Name}, ms); getErr != nil {`
`116`	`116`	`// If the GET fails we can't trust status.Replicas anymore. This error`
`117`	`117`	`// is bound to be more interesting than the update failure.`
`118`	`118`	`return nil, getErr`