diff options
author | Jon Marius Venstad <jvenstad@yahoo-inc.com> | 2018-01-19 14:39:31 +0100 |
---|---|---|
committer | Jon Marius Venstad <jvenstad@yahoo-inc.com> | 2018-01-19 14:39:31 +0100 |
commit | a2d8de649adb14d00c2294dc659c2c6e4c1dfc30 (patch) | |
tree | abe743c49b71207c46cc4a6d07ec24958939cdb8 /controller-server | |
parent | 1bd7e9beedc19b4f5c08805644db55c4c768df10 (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.java | 17 |
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 */ |