Skip to content

Commit

Permalink
[PLAT-13708] Last backup time initialized during HA sync
Browse files Browse the repository at this point in the history
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
  • Loading branch information
mchiddy committed Apr 25, 2024
1 parent b0c73f4 commit 24f8a2a
Showing 1 changed file with 2 additions and 1 deletion.
Expand Up @@ -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 {
Expand Down

0 comments on commit 24f8a2a

Please sign in to comment.