(Feature) Add ACSDeploymentSynced condition type and fix comparison of SecretHashes method (#944)

CHANGELOG.md

@@ -8,6 +8,7 @@
 - (Feature) Allow raw json value for license token-v2
 - (Update) Replace `beta.kubernetes.io/arch` to `kubernetes.io/arch` in Operator Chart
 - (Feature) Add operator shutdown handler for graceful termination
+- (Feature) Add `ACSDeploymentSynced` condition type and fix comparison of `SecretHashes` method
 
 ## [1.2.12](https://github.com/arangodb/kube-arangodb/tree/1.2.12) (2022-05-10)
 - (Feature) Add CoreV1 Endpoints Inspector

pkg/apis/deployment/v1/secret_hashes.go

@@ -38,10 +38,10 @@ type SecretHashes struct {
 
 // Equal compares two SecretHashes
 func (sh *SecretHashes) Equal(other *SecretHashes) bool {
-	if sh == nil || other == nil {
+	if sh == nil && other == nil {
-		return false
-	} else if sh == other {
 		return true
+	} else if sh == nil || other == nil {
+		return false
 	}
 
 	return sh.AuthJWT == other.AuthJWT &&

pkg/apis/deployment/v2alpha1/secret_hashes.go

@@ -38,10 +38,10 @@ type SecretHashes struct {
 
 // Equal compares two SecretHashes
 func (sh *SecretHashes) Equal(other *SecretHashes) bool {
-	if sh == nil || other == nil {
+	if sh == nil && other == nil {
-		return false
-	} else if sh == other {
 		return true
+	} else if sh == nil || other == nil {
+		return false
 	}
 
 	return sh.AuthJWT == other.AuthJWT &&

pkg/deployment/acs/sutil/conditions.go

@@ -28,4 +28,5 @@ const (
 	RemoteDeploymentReadyCondition api.ConditionType = "RemoteDeploymentReadyCondition"
 	RemoteCacheReadyCondition      api.ConditionType = "RemoteCacheReady"
 	ConnectionReadyCondition       api.ConditionType = "ConnectionReady"
+	ACSDeploymentSyncedCondition   api.ConditionType = "ACSDeploymentSynced"
 )