[Bugfix] Prevent Member Maintenance Error log (#1136)

Author: ajanikow

CHANGELOG.md

@@ -4,6 +4,7 @@
 - (Feature) Add action progress
 - (Feature) Ensure consistency during replication cancellation
 - (Feature) Add annotation to change architecture of a member
+- (Bugfix) Prevent Member Maintenance Error log
 
 ## [1.2.19](https://github.com/arangodb/kube-arangodb/tree/1.2.19) (2022-10-05)
 - (Bugfix) Prevent changes when UID is wrong

pkg/deployment/features/features.go

@@ -20,7 +20,11 @@
 
 package features
 
-import "github.com/arangodb/go-driver"
+import (
+	"github.com/arangodb/go-driver"
+
+	api "github.com/arangodb/kube-arangodb/pkg/apis/deployment/v1"
+)
 
 const (
 	Enabled  = "true"
@@ -40,6 +44,7 @@ type Feature interface {
 	Deprecated() (bool, string)
 	Hidden() bool
 	Supported(v driver.Version, enterprise bool) bool
+	ImageSupported(i *api.ImageInfo) bool
 }
 
 type feature struct {
@@ -51,6 +56,14 @@ type feature struct {
 	hidden                                        bool
 }
 
+func (f feature) ImageSupported(i *api.ImageInfo) bool {
+	if i == nil {
+		return false
+	}
+
+	return f.Supported(i.ArangoDBVersion, i.Enterprise)
+}
+
 func (f feature) Hidden() bool {
 	return f.hidden
 }

pkg/deployment/reconcile/helper_wrap.go

@@ -51,7 +51,7 @@ func withMemberMaintenance(group api.ServerGroup, member api.MemberStatus, reaso
 		return plan
 	}
 
-	if !features.Version310().Enabled() {
+	if !features.Version310().ImageSupported(member.Image) {
 		return plan
 	}
 

pkg/deployment/resources/pod_creator_probes.go

@@ -513,5 +513,5 @@ func getProbeRetries(group api.ServerGroup) (int32, int32) {
 // IsServerProgressAvailable returns true if server progress is available.
 func IsServerProgressAvailable(group api.ServerGroup, imageInfo api.ImageInfo) bool {
 	return group == api.ServerGroupDBServers &&
-		features.Version310().Supported(imageInfo.ArangoDBVersion, imageInfo.Enterprise)
+		features.Version310().ImageSupported(&imageInfo)
 }