diff options
author | Valerij Fredriksen <valerijf@yahooinc.com> | 2022-11-03 09:16:29 +0100 |
---|---|---|
committer | Valerij Fredriksen <valerijf@yahooinc.com> | 2022-11-03 09:16:29 +0100 |
commit | 01b99953932ce2629ee3e71264789a3044d17b65 (patch) | |
tree | 5886a059ed453f2bb719e1ee43392fdb6b31715d /node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance | |
parent | c4c2eb2c42bd7a134b6ea71a0d2db9005b5c5ee0 (diff) |
Trigger quick redeployment for changes by HostResumeProvisioner agent
Diffstat (limited to 'node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance')
2 files changed, 5 insertions, 8 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/ExpeditedChangeApplicationMaintainer.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/ExpeditedChangeApplicationMaintainer.java index fa6b201def4..c99c18262ec 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/ExpeditedChangeApplicationMaintainer.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/ExpeditedChangeApplicationMaintainer.java @@ -107,12 +107,10 @@ public class ExpeditedChangeApplicationMaintainer extends ApplicationMaintainer /** Returns whether to expedite changes performed by agent */ private boolean expediteChangeBy(Agent agent) { - switch (agent) { - case operator: - case RebuildingOsUpgrader: - case HostEncrypter: return true; - } - return false; + return switch (agent) { + case operator, HostEncrypter, HostResumeProvisioner, RebuildingOsUpgrader -> true; + default -> false; + }; } } diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/HostResumeProvisioner.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/HostResumeProvisioner.java index 9dfb752b66e..3fb07496714 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/HostResumeProvisioner.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/HostResumeProvisioner.java @@ -65,9 +65,8 @@ public class HostResumeProvisioner extends NodeRepositoryMaintainer { failures++; log.log(Level.SEVERE, "Failed to provision " + host.hostname() + " with " + children.size() + " children, failing out the host recursively", e); - // Fail out as operator to force a quick redeployment nodeRepository().nodes().failOrMarkRecursively( - host.hostname(), Agent.operator, "Failed by HostProvisioner due to provisioning failure"); + host.hostname(), Agent.HostResumeProvisioner, "Failed by HostResumeProvisioner due to provisioning failure"); } catch (RuntimeException e) { if (e.getCause() instanceof NamingException) log.log(Level.INFO, "Could not provision " + host.hostname() + ", will retry in " + interval() + ": " + Exceptions.toMessageString(e)); |