Skip to content

Commit d028b5e

Browse files
hanwen-clusterhanwen-pcluste
authored andcommitted
Get instances from DynamoDB in batches of 100
DynamoDB batch_get_item API limit the number of input to 100. This change fixes cluster creation with >100 compute nodes Signed-off-by: Hanwen <[email protected]>
1 parent 870aa39 commit d028b5e

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

cookbooks/aws-parallelcluster-slurm/files/default/head_node_checks/common/constants.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212

1313
# BOTO
1414
BOTO_CONFIG = Config(retries={"max_attempts": 60})
15-
BOTO_PAGINATION_CONFIG = {"PageSize": 500}
15+
BOTO_PAGINATION_CONFIG = {"PageSize": 100}
1616

1717
# TAGS
1818
CLUSTER_NAME_TAG = "parallelcluster:cluster-name"

test/unit/head_node_checks/test_check_cluster_ready.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ def _mocked_request_describe_instances(cluster_name: str, node_types: [str], com
4141
{"Name": "tag:parallelcluster:node-type", "Values": node_types},
4242
{"Name": "instance-state-name", "Values": ["running"]},
4343
],
44-
"MaxResults": 500,
44+
"MaxResults": 100,
4545
},
4646
generate_error=False,
4747
error_code=None,

0 commit comments

Comments
 (0)