|
47 | 47 | DEFAULT_MIN_COUNT, |
48 | 48 | DELETE_POLICY, |
49 | 49 | DETAILED_MONITORING_ENABLED_DEFAULT, |
| 50 | + EBS_ROOT_VOLUME_SIZE_DEFAULT, |
50 | 51 | EBS_VOLUME_SIZE_DEFAULT, |
51 | 52 | EBS_VOLUME_TYPE_DEFAULT, |
52 | 53 | EBS_VOLUME_TYPE_IOPS_DEFAULT, |
@@ -256,12 +257,10 @@ def __init__(self, size: int = None, delete_on_termination: bool = None, **kwarg |
256 | 257 | super().__init__(**kwargs) |
257 | 258 | # When the RootVolume size is None, EC2 implicitly sets it as the AMI size. |
258 | 259 | # In US Isolated regions, the root volume size cannot be left unspecified, |
259 | | - # so we consider it as the default EBS volume size. |
260 | | - # In theory, the default value should be maximum between the default EBS volume size (35GB) and the AMI size, |
261 | | - # but in US Isolated region this is fine because the only supported AMI as of 2023 Feb |
262 | | - # is the official ParallelCluster AMI for Amazon Linux 2, which has size equal to |
263 | | - # the default EBS volume size (35GB). |
264 | | - self.size = Resource.init_param(size, EBS_VOLUME_SIZE_DEFAULT if get_region().startswith("us-iso") else None) |
| 260 | + # so we consider it as the default EBS root volume size. |
| 261 | + self.size = Resource.init_param( |
| 262 | + size, EBS_ROOT_VOLUME_SIZE_DEFAULT if get_region().startswith("us-iso") else None |
| 263 | + ) |
265 | 264 | # The default delete_on_termination takes effect both on head and compute nodes. |
266 | 265 | # If the default of the head node is to be changed, please separate this class for different defaults. |
267 | 266 | self.delete_on_termination = Resource.init_param(delete_on_termination, default=True) |
|
0 commit comments