Skip to content

Commit 6bb4457

Browse files
authored
Merge pull request #388 from arangodb/bug-fix/remove-server-no-coord
Ignore connection errors when removing server.
2 parents fad7c29 + 04b7cf3 commit 6bb4457

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

pkg/deployment/reconcile/action_remove_member.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,8 @@ func (a *actionRemoveMember) Start(ctx context.Context) (bool, error) {
6969
}
7070
if err := arangod.RemoveServerFromCluster(ctx, client.Connection(), driver.ServerID(m.ID)); err != nil {
7171
if !driver.IsNotFound(err) && !driver.IsPreconditionFailed(err) {
72-
return false, maskAny(errors.Wrapf(err, "Failed to remove server from cluster: %#v", err))
72+
a.log.Err(err).Str("member-id", m.ID).Msgf("Failed to remove server from cluster")
73+
// ignore this error, maybe all coordinators are failed and no connction to cluster is possible
7374
} else if driver.IsPreconditionFailed(err) {
7475
cluster, err := client.Cluster(ctx)
7576
if err != nil {

0 commit comments

Comments
 (0)