CLOUDP-141985: Fix the GCP PE comparison (#746)

fabritsius · web-flow · commit 74f9564c4150 · 2022-11-01T09:45:38.000+03:00
diff --git a/pkg/controller/atlasproject/private_endpoint.go b/pkg/controller/atlasproject/private_endpoint.go
@@ -268,7 +268,7 @@ func endpointNeedsUpdating(specPeService mdbv1.PrivateEndpoint, atlasPeService a
 		case provider.ProviderAWS, provider.ProviderAzure:
 			return specPeService.ID != atlasPeService.InterfaceEndpointID()
 		case provider.ProviderGCP:
-			return specPeService.EndpointGroupName != atlasPeService.InterfaceEndpointID() && len(atlasPeService.EndpointServiceName) != len(specPeService.Endpoints)
+			return specPeService.EndpointGroupName != atlasPeService.InterfaceEndpointID() || len(atlasPeService.ServiceAttachmentNames) != len(specPeService.Endpoints)
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -268,7 +268,7 @@ func endpointNeedsUpdating(specPeService mdbv1.PrivateEndpoint, atlasPeService a`
`268`	`268`	`case provider.ProviderAWS, provider.ProviderAzure:`
`269`	`269`	`return specPeService.ID != atlasPeService.InterfaceEndpointID()`
`270`	`270`	`case provider.ProviderGCP:`
`271`		`- return specPeService.EndpointGroupName != atlasPeService.InterfaceEndpointID() && len(atlasPeService.EndpointServiceName) != len(specPeService.Endpoints)`
	`271`	`+ return specPeService.EndpointGroupName != atlasPeService.InterfaceEndpointID() \|\| len(atlasPeService.ServiceAttachmentNames) != len(specPeService.Endpoints)`
`272`	`272`	`}`
`273`	`273`	`}`
`274`	`274`