Improvements to handling of cluster removal

- Don't change health status to "initializing" when creating a new VipStatus,
  as 'initializing' now requires all clusters to be up to transition to 'up',
  which means that if we're already up but are missing a cluster we'll go from
  'up' to 'initializing' and stay there.
- Forget up/down status for removed clusters.
- Nicer logging on ignorable reconfiguration errors.

1 files changed, 2 insertions, 5 deletions

diff --git a/vdslib/src/main/java/com/yahoo/vdslib/distribution/Distribution.java b/vdslib/src/main/java/com/yahoo/vdslib/distribution/Distribution.java
index 52708ea6d8e..c9acd625373 100644
--- a/vdslib/src/main/java/com/yahoo/vdslib/distribution/Distribution.java
+++ b/vdslib/src/main/java/com/yahoo/vdslib/distribution/Distribution.java
@@ -97,11 +97,8 @@ public class Distribution {
                         parent.addSubGroup(group);
                     }
                 }
-                if (root == null) {
-                    throw new IllegalStateException("Got config that did not "
-                            + "specify even a root group. Need a root group at"
-                            + "\nminimum:\n" + config.toString());
-                }
+                if (root == null)
+                    throw new IllegalStateException("Config does not specify a root group");
                 root.calculateDistributionHashValues();
                 Distribution.this.config.setRelease(new Config(root, config.redundancy(), config.distributor_auto_ownership_transfer_on_whole_group_down()));
             } catch (ParseException e) {