Skip to content

Commit

Permalink
Replace 'master' terminology with 'cluster manager' in 'plugins' dire…
Browse files Browse the repository at this point in the history
…ctory (#3329)

* Replace 'master' terminology with 'cluster manager' in 'plugins' directory

Signed-off-by: Tianli Feng <[email protected]>

* Replace master_node with cluster_manager_node in cluster.state API test

Signed-off-by: Tianli Feng <[email protected]>
(cherry picked from commit 8460bcf)
  • Loading branch information
Tianli Feng authored and github-actions[bot] committed May 16, 2022
1 parent 7e064a2 commit cff86af
Show file tree
Hide file tree
Showing 17 changed files with 50 additions and 50 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@
- do:
cluster.state: {}

# Get master node id
- set: { master_node: master }
# Get cluster-manager node id
- set: { cluster_manager_node: cluster_manager }

- do:
nodes.info: {}

- contains: { nodes.$master.plugins: { name: discovery-azure-classic } }
- contains: { nodes.$cluster_manager.plugins: { name: discovery-azure-classic } }
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@
*/
@ClusterScope(scope = Scope.TEST, numDataNodes = 0, numClientNodes = 0)
public class Ec2DiscoveryUpdateSettingsTests extends AbstractAwsTestCase {
public void testMinimumMasterNodesStart() {
public void testMinimumClusterManagerNodesStart() {
Settings nodeSettings = Settings.builder().put(DiscoveryModule.DISCOVERY_SEED_PROVIDERS_SETTING.getKey(), "ec2").build();
internalCluster().startNode(nodeSettings);

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@
- do:
cluster.state: {}

# Get master node id
- set: { master_node: master }
# Get cluster-manager node id
- set: { cluster_manager_node: cluster_manager }

- do:
nodes.info: {}

- contains: { nodes.$master.plugins: { name: discovery-ec2 } }
- contains: { nodes.$cluster_manager.plugins: { name: discovery-ec2 } }
Original file line number Diff line number Diff line change
Expand Up @@ -85,11 +85,11 @@ protected Settings nodeSettings(int nodeOrdinal) {
}

public void testJoin() {
// start master node
final String masterNode = internalCluster().startClusterManagerOnlyNode();
registerGceNode(masterNode);
// start cluster-manager node
final String clusterManagerNode = internalCluster().startClusterManagerOnlyNode();
registerGceNode(clusterManagerNode);

ClusterStateResponse clusterStateResponse = client(masterNode).admin()
ClusterStateResponse clusterStateResponse = client(clusterManagerNode).admin()
.cluster()
.prepareState()
.setMasterNodeTimeout("1s")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@
- do:
cluster.state: {}

# Get master node id
- set: { master_node: master }
# Get cluster-manager node id
- set: { cluster_manager_node: cluster_manager }

- do:
nodes.info: {}

- contains: { nodes.$master.plugins: { name: discovery-gce } }
- contains: { nodes.$cluster_manager.plugins: { name: discovery-gce } }
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,12 @@
reason: "contains is a newly added assertion"
features: contains

# Get master node id
# Get cluster-manager node id
- do:
cluster.state: {}
- set: { master_node: master }
- set: { cluster_manager_node: cluster_manager }

- do:
nodes.info: {}

- contains: { nodes.$master.plugins: { name: custom-significance-heuristic } }
- contains: { nodes.$cluster_manager.plugins: { name: custom-significance-heuristic } }
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@
- do:
cluster.state: {}

# Get master node id
- set: { master_node: master }
# Get cluster-manager node id
- set: { cluster_manager_node: cluster_manager }

- do:
nodes.info: {}

- contains: { nodes.$master.plugins: { name: custom-suggester } }
- contains: { nodes.$cluster_manager.plugins: { name: custom-suggester } }
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@
- do:
cluster.state: {}

# Get master node id
- set: { master_node: master }
# Get cluster-manager node id
- set: { cluster_manager_node: cluster_manager }

- do:
nodes.info: {}

- contains: { nodes.$master.plugins: { name: painless-whitelist } }
- contains: { nodes.$cluster_manager.plugins: { name: painless-whitelist } }
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@
- do:
cluster.state: {}

# Get master node id
- set: { master_node: master }
# Get cluster-manager node id
- set: { cluster_manager_node: cluster_manager }

- do:
nodes.info: {}

- contains: { nodes.$master.plugins: { name: example-rescore } }
- contains: { nodes.$cluster_manager.plugins: { name: example-rescore } }
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@
- do:
cluster.state: {}

# Get master node id
- set: { master_node: master }
# Get cluster-manager node id
- set: { cluster_manager_node: cluster_manager }

- do:
nodes.info: {}

- contains: { nodes.$master.plugins: { name: script-expert-scoring } }
- contains: { nodes.$cluster_manager.plugins: { name: script-expert-scoring } }
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,10 @@
- do:
cluster.state: {}

- set: {master_node: master}
- set: { cluster_manager_node: cluster_manager }

- do:
nodes.info: {}

- contains: { 'nodes.$master.plugins': { name: ingest-attachment } }
- contains: { 'nodes.$master.ingest.processors': { type: attachment } }
- contains: { 'nodes.$cluster_manager.plugins': { name: ingest-attachment } }
- contains: { 'nodes.$cluster_manager.ingest.processors': { type: attachment } }
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@
- do:
cluster.state: {}

# Get master node id
- set: { master_node: master }
# Get cluster-manager node id
- set: { cluster_manager_node: cluster_manager }

- do:
nodes.info: {}

- contains: { nodes.$master.plugins: { name: repository-azure } }
- contains: { nodes.$cluster_manager.plugins: { name: repository-azure } }
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@
- do:
cluster.state: {}

# Get master node id
- set: { master_node: master }
# Get cluster-manager node id
- set: { cluster_manager_node: cluster_manager }

- do:
nodes.info: {}

- contains: { nodes.$master.plugins: { name: repository-gcs } }
- contains: { nodes.$cluster_manager.plugins: { name: repository-gcs } }
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,13 @@
- do:
cluster.state: {}

# Get master node id
- set: { master_node: master }
# Get cluster-manager node id
- set: { cluster_manager_node: cluster_manager }

- do:
nodes.info: {}

- contains: { nodes.$master.plugins: { name: repository-hdfs } }
- contains: { nodes.$cluster_manager.plugins: { name: repository-hdfs } }
---
#
# Check that we can't use file:// repositories or anything like that
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,13 @@
- do:
cluster.state: {}

# Get master node id
- set: { master_node: master }
# Get cluster-manager node id
- set: { cluster_manager_node: cluster_manager }

- do:
nodes.info: {}

- contains: { nodes.$master.plugins: { name: repository-hdfs } }
- contains: { nodes.$cluster_manager.plugins: { name: repository-hdfs } }
---
#
# Check that we can't use file:// repositories or anything like that
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@
- do:
cluster.state: {}

# Get master node id
- set: { master_node: master }
# Get cluster-manager node id
- set: { cluster_manager_node: cluster_manager }

- do:
nodes.info: {}

- contains: { nodes.$master.plugins: { name: repository-s3 } }
- contains: { nodes.$cluster_manager.plugins: { name: repository-s3 } }
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@
- do:
cluster.state: {}

# Get master node id
- set: { master_node: master }
# Get cluster-manager node id
- set: { cluster_manager_node: cluster_manager }

- do:
nodes.info: {}

- contains: { nodes.$master.plugins: { name: store-smb } }
- contains: { nodes.$cluster_manager.plugins: { name: store-smb } }

0 comments on commit cff86af

Please sign in to comment.