diff options
author | Jon Bratseth <bratseth@gmail.com> | 2020-07-02 11:54:43 +0200 |
---|---|---|
committer | Jon Bratseth <bratseth@gmail.com> | 2020-07-02 11:54:43 +0200 |
commit | eb37f7dcbd89143715fbe9b838a91b296db0a3c9 (patch) | |
tree | a065cc8ffe0bee8fc9022fd32a926ff5712542c6 /vdslib | |
parent | b4abecef92c802ee6f779dc77d4328cc6fbf22cb (diff) |
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.
Diffstat (limited to 'vdslib')
-rw-r--r-- | vdslib/src/main/java/com/yahoo/vdslib/distribution/Distribution.java | 7 |
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) { |