Skip to content

Commit

Permalink
Increase default PCLUSTER_RESERVED_VOLUME_SIZE to 32
Browse files Browse the repository at this point in the history
The vanilla AL2 AMI is 8GB, with additional 32 GB, the outcome 40GB is the same size as our official AMI

Signed-off-by: Hanwen <[email protected]>
  • Loading branch information
hanwen-pcluste committed Aug 22, 2024
1 parent 32aac47 commit 2bd7948
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion cli/src/pcluster/imagebuilder_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
from pcluster.utils import get_url_scheme, yaml_load

ROOT_VOLUME_TYPE = "gp3"
PCLUSTER_RESERVED_VOLUME_SIZE = 27
PCLUSTER_RESERVED_VOLUME_SIZE = 32
AMI_NAME_REQUIRED_SUBSTRING = " {{ imagebuilder:buildDate }}"


Expand Down
4 changes: 2 additions & 2 deletions cli/tests/pcluster/templates/test_imagebuilder_stack.py
Original file line number Diff line number Diff line change
Expand Up @@ -3317,7 +3317,7 @@ def test_imagebuilder_distribution_configuraton(mocker, resource, response, expe
}
],
},
{"Encrypted": False, "VolumeSize": 35, "VolumeType": "gp3"},
{"Encrypted": False, "VolumeSize": 40, "VolumeType": "gp3"},
),
(
{
Expand Down Expand Up @@ -3399,7 +3399,7 @@ def test_imagebuilder_distribution_configuraton(mocker, resource, response, expe
},
{
"Encrypted": True,
"VolumeSize": 77,
"VolumeSize": 82,
"VolumeType": "gp3",
"KmsKeyId": "arn:aws:kms:us-east-1:111122223333:key/1234abcd-12ab-34cd-56ef-1234567890ab",
},
Expand Down

0 comments on commit 2bd7948

Please sign in to comment.