-
Notifications
You must be signed in to change notification settings - Fork 24.3k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Master stability health indicator part 1 (when a master has been seen…
… recently) (#86524) This is the first PR for the master stability check, which is part of the health API. It handles the case when we have seen a master node recently. The more complicated case when we have not seen a master node recently will be in subsequent PRs.
- Loading branch information
Showing
12 changed files
with
1,219 additions
and
202 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
pr: 86524 | ||
summary: Master stability health indicator part 1 (when a master has been seen recently) | ||
area: Health | ||
type: feature | ||
issues: [] |
8 changes: 4 additions & 4 deletions
8
rest-api-spec/src/yamlRestTest/resources/rest-api-spec/test/health/10_basic.yml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,14 +1,14 @@ | ||
--- | ||
"cluster health basic test": | ||
- skip: | ||
version: "- 8.2.99" | ||
reason: "summary text was updated in 8.3.0" | ||
version: "- 8.3.99" | ||
reason: "health was only added in 8.2.0, and master_is_stable in 8.4.0" | ||
|
||
- do: | ||
_internal.health: {} | ||
|
||
- is_true: cluster_name | ||
- match: { status: "green" } | ||
- match: { components.cluster_coordination.status: "green" } | ||
- match: { components.cluster_coordination.indicators.instance_has_master.status: "green" } | ||
- match: { components.cluster_coordination.indicators.instance_has_master.summary: "Health coordinating instance has an elected master node." } | ||
- match: { components.cluster_coordination.indicators.master_is_stable.status: "green" } | ||
- match: { components.cluster_coordination.indicators.master_is_stable.summary: "The cluster has a stable master node" } |
14 changes: 6 additions & 8 deletions
14
rest-api-spec/src/yamlRestTest/resources/rest-api-spec/test/health/20_component.yml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,18 +1,16 @@ | ||
--- | ||
"cluster health test drilling down into a component": | ||
- skip: | ||
version: "- 8.2.99" | ||
reason: "health drilldown was only added in 8.3.0" | ||
version: "- 8.3.99" | ||
reason: "health drilldown was only added in 8.3.0, and master_is_stable in 8.4.0" | ||
|
||
- do: | ||
_internal.health: | ||
component: cluster_coordination | ||
|
||
- is_true: cluster_name | ||
- match: { components.cluster_coordination.status: "green" } | ||
- match: { components.cluster_coordination.indicators.instance_has_master.status: "green" } | ||
- match: { components.cluster_coordination.indicators.instance_has_master.summary: "Health coordinating instance has an elected master node." } | ||
- is_true: components.cluster_coordination.indicators.instance_has_master.details.coordinating_node.node_id | ||
- is_true: components.cluster_coordination.indicators.instance_has_master.details.coordinating_node.name | ||
- is_true: components.cluster_coordination.indicators.instance_has_master.details.master_node.node_id | ||
- is_true: components.cluster_coordination.indicators.instance_has_master.details.master_node.name | ||
- match: { components.cluster_coordination.indicators.master_is_stable.status: "green" } | ||
- match: { components.cluster_coordination.indicators.master_is_stable.summary: "The cluster has a stable master node" } | ||
- is_true: components.cluster_coordination.indicators.master_is_stable.details.current_master | ||
- is_true: components.cluster_coordination.indicators.master_is_stable.details.recent_masters |
16 changes: 7 additions & 9 deletions
16
rest-api-spec/src/yamlRestTest/resources/rest-api-spec/test/health/30_feature.yml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,18 +1,16 @@ | ||
--- | ||
"cluster health test drilling down into a component and a feature": | ||
- skip: | ||
version: "- 8.2.99" | ||
reason: "health drilldown was only added in 8.3.0" | ||
version: "- 8.3.99" | ||
reason: "health drilldown was only added in 8.3.0, and master_is_stable in 8.4.0" | ||
|
||
- do: | ||
_internal.health: | ||
component: cluster_coordination | ||
feature: instance_has_master | ||
feature: master_is_stable | ||
|
||
- is_true: cluster_name | ||
- match: { components.cluster_coordination.indicators.instance_has_master.status: "green" } | ||
- match: { components.cluster_coordination.indicators.instance_has_master.summary: "Health coordinating instance has an elected master node." } | ||
- is_true: components.cluster_coordination.indicators.instance_has_master.details.coordinating_node.node_id | ||
- is_true: components.cluster_coordination.indicators.instance_has_master.details.coordinating_node.name | ||
- is_true: components.cluster_coordination.indicators.instance_has_master.details.master_node.node_id | ||
- is_true: components.cluster_coordination.indicators.instance_has_master.details.master_node.name | ||
- match: { components.cluster_coordination.indicators.master_is_stable.status: "green" } | ||
- match: { components.cluster_coordination.indicators.master_is_stable.summary: "The cluster has a stable master node" } | ||
- is_true: components.cluster_coordination.indicators.master_is_stable.details.current_master | ||
- is_true: components.cluster_coordination.indicators.master_is_stable.details.recent_masters |
77 changes: 0 additions & 77 deletions
77
...ava/org/elasticsearch/cluster/coordination/InstanceHasMasterHealthIndicatorServiceIT.java
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.