From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by lists.proxmox.com (Postfix) with ESMTPS id E07F960D38 for ; Wed, 18 Nov 2020 15:53:47 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id D822B1D40B for ; Wed, 18 Nov 2020 15:53:17 +0100 (CET) Received: from proxmox-new.maurer-it.com (proxmox-new.maurer-it.com [212.186.127.180]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by firstgate.proxmox.com (Proxmox) with ESMTPS id 28CB01D3F2 for ; Wed, 18 Nov 2020 15:53:17 +0100 (CET) Received: from proxmox-new.maurer-it.com (localhost.localdomain [127.0.0.1]) by proxmox-new.maurer-it.com (Proxmox) with ESMTP id EFF6E439C8 for ; Wed, 18 Nov 2020 15:53:16 +0100 (CET) From: Stoiko Ivanov To: pmg-devel@lists.proxmox.com Date: Wed, 18 Nov 2020 15:52:53 +0100 Message-Id: <20201118145254.26214-2-s.ivanov@proxmox.com> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20201118145254.26214-1-s.ivanov@proxmox.com> References: <20201118145254.26214-1-s.ivanov@proxmox.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-SPAM-LEVEL: Spam detection results: 0 AWL -1.066 Adjusted score from AWL reputation of From: address KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Subject: [pmg-devel] [PATCH pmg-api 1/2] fix clustersync after node-deletion X-BeenThere: pmg-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox Mail Gateway development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 18 Nov 2020 14:53:47 -0000 This patch creates the spoolsdirs for a newly joining clusternode on the master (/var/spool/pmg/cluster//(spam|attachment|virus). This is necessary in order to prevent a failing cluster-sync for nodes, joining the cluster after that node has been deleted. (This happens if you remove a node from the cluster and directly rejoin it to the same masternode): On the first sync after a node was deleted (there is no section config for a number < maxcid) each node tries to sync the quarantine for the deleted node from the cluster (in order to be promotable to new master). This rsync fails because the spooldir for that node never got created on the master. The spooldir for a node gets created on the master on the first sync of a node which can be 2 minutes after joining the cluster (and leaving it again). Reported via our enterprise support portal. Signed-off-by: Stoiko Ivanov --- src/PMG/API2/Cluster.pm | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/PMG/API2/Cluster.pm b/src/PMG/API2/Cluster.pm index ceda100..7eab761 100644 --- a/src/PMG/API2/Cluster.pm +++ b/src/PMG/API2/Cluster.pm @@ -302,6 +302,10 @@ __PACKAGE__->register_method({ $next_cid = ++$master->{maxcid}; } + # create spooldir for new node to prevent problems if it gets + # delete from the cluster before being synced initially + PMG::MailQueue::create_spooldirs($master->{maxcid}); + my $node = { type => 'node', cid => $master->{maxcid}, -- 2.20.1