aboutsummaryrefslogtreecommitdiffstats
path: root/controller-server
diff options
context:
space:
mode:
authorJon Marius Venstad <jvenstad@yahoo-inc.com>2018-01-19 14:39:31 +0100
committerJon Marius Venstad <jvenstad@yahoo-inc.com>2018-01-19 14:39:31 +0100
commita2d8de649adb14d00c2294dc659c2c6e4c1dfc30 (patch)
treeabe743c49b71207c46cc4a6d07ec24958939cdb8 /controller-server
parent1bd7e9beedc19b4f5c08805644db55c4c768df10 (diff)
Remove all aliases for each rotation, when unused
Diffstat (limited to 'controller-server')
-rw-r--r--controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/DnsMaintainer.java17
1 files changed, 8 insertions, 9 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/DnsMaintainer.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/DnsMaintainer.java
index 0f957c58a47..180e8f90ae2 100644
--- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/DnsMaintainer.java
+++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/DnsMaintainer.java
@@ -13,7 +13,6 @@ import com.yahoo.vespa.hosted.controller.rotation.RotationRepository;
import java.time.Duration;
import java.util.Map;
-import java.util.Optional;
import java.util.logging.Logger;
/**
@@ -48,14 +47,14 @@ public class DnsMaintainer extends Maintainer {
/** Remove DNS alias for unassigned rotation */
private void removeDnsAlias(Rotation rotation) {
// When looking up CNAME by data, the data must be a FQDN
- Optional<Record> record = nameService.findRecord(Record.Type.CNAME, RecordData.fqdn(rotation.name()));
- record.filter(DnsMaintainer::canUpdate)
- .ifPresent(r -> {
- log.info(String.format("Removing DNS record %s (%s) because it points to the unassigned " +
- "rotation %s (%s)", record.get().id().asString(),
- record.get().name().asString(), rotation.id().asString(), rotation.name()));
- nameService.removeRecord(r.id());
- });
+ nameService.findRecord(Record.Type.CNAME, RecordData.fqdn(rotation.name())).stream()
+ .filter(DnsMaintainer::canUpdate)
+ .forEach(record -> {
+ log.info(String.format("Removing DNS record %s (%s) because it points to the unassigned " +
+ "rotation %s (%s)", record.id().asString(),
+ record.name().asString(), rotation.id().asString(), rotation.name()));
+ nameService.removeRecord(record.id());
+ });
}
/** Returns whether we can update the given record */