diff options
author | Harald Musum <musum@verizonmedia.com> | 2023-10-10 13:06:51 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-10-10 13:06:51 +0200 |
commit | 69263495d5a827cd230e4ffd832747334f15a037 (patch) | |
tree | e5f4e606ee12e68ab85fe2dc40825bda29e47d20 /node-repository/src/main/java/com | |
parent | 2f635d64e4ba166db3b15854cc0339e0bcdb2f3d (diff) | |
parent | a8c5a3915b42eb1f2412b0a4824e285657e41351 (diff) |
Merge pull request #28848 from vespa-engine/hmusum/log-application-id-when-failing-to-find-last-deploy-time
Fallback to last activation time if we cannot find last deploy time
Diffstat (limited to 'node-repository/src/main/java/com')
-rw-r--r-- | node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/PeriodicApplicationMaintainer.java | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/PeriodicApplicationMaintainer.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/PeriodicApplicationMaintainer.java index 226c3f34c28..3cec8483a45 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/PeriodicApplicationMaintainer.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/PeriodicApplicationMaintainer.java @@ -11,10 +11,12 @@ import com.yahoo.vespa.flags.PermanentFlags; import com.yahoo.vespa.hosted.provision.Node; import com.yahoo.vespa.hosted.provision.NodeList; import com.yahoo.vespa.hosted.provision.NodeRepository; +import com.yahoo.yolean.Exceptions; import java.time.Duration; import java.util.Map; +import static java.util.logging.Level.INFO; import static java.util.stream.Collectors.toMap; /** @@ -38,10 +40,16 @@ public class PeriodicApplicationMaintainer extends ApplicationMaintainer { @Override protected boolean canDeployNow(ApplicationId application) { - return deployer().deployTime(application) - .map(lastDeployTime -> lastDeployTime.isBefore(nodeRepository().clock().instant().minus(minTimeBetweenRedeployments)) - || deployer().readiedReindexingAfter(application, lastDeployTime)) - .orElse(false); + try { + return deployer().deployTime(application) + .map(lastDeployTime -> lastDeployTime.isBefore(nodeRepository().clock().instant().minus(minTimeBetweenRedeployments)) + || deployer().readiedReindexingAfter(application, lastDeployTime)) + .orElse(false); + } + catch (Exception e) { + log.log(INFO, () -> "Failed finding last deploy time or reindexing status for " + application + Exceptions.toMessageString(e)); + throw e; + } } @Override |