From 24f8a2a32f490711372dc7f8e3c478e3f498501e Mon Sep 17 00:00:00 2001 From: Muthu Chidambaram Date: Thu, 25 Apr 2024 20:15:27 +0000 Subject: [PATCH] [PLAT-13708] Last backup time initialized during HA sync Summary: The fix https://phorge.dev.yugabyte.com/D34125 to sync remote configs removed the initialization of the local instance last backup time so subsequent updates were always null. This adds that line back in. Test Plan: Setup HA, ensure that lastBackupTime is displayed and shows up in HA config Reviewers: dshubin, sanketh Reviewed By: dshubin Subscribers: yugaware Differential Revision: https://phorge.dev.yugabyte.com/D34554 --- .../com/yugabyte/yw/common/ha/PlatformReplicationManager.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/managed/src/main/java/com/yugabyte/yw/common/ha/PlatformReplicationManager.java b/managed/src/main/java/com/yugabyte/yw/common/ha/PlatformReplicationManager.java index 937eb497153..57e076b2a63 100644 --- a/managed/src/main/java/com/yugabyte/yw/common/ha/PlatformReplicationManager.java +++ b/managed/src/main/java/com/yugabyte/yw/common/ha/PlatformReplicationManager.java @@ -342,11 +342,12 @@ private synchronized void sync() { return; } - // Update local last backup time if creating the backup succeeded. config .getLocal() .ifPresent( localInstance -> { + // Update local last backup time since creating the backup succeeded. + localInstance.updateLastBackup(); remoteInstances.forEach( instance -> { try {