Skip to content

Commit 99fedbc

Browse files
author
Himani Anil Deshpande
committed
code formatting
1 parent 7dbd12d commit 99fedbc

File tree

3 files changed

+19
-9
lines changed

3 files changed

+19
-9
lines changed

cli/src/pcluster/config/imagebuilder_config.py

+5-3
Original file line numberDiff line numberDiff line change
@@ -148,8 +148,8 @@ class DisableKernelUpdate(Resource):
148148
"""Represent the DisableKernelUpdate configuration for the ImageBuilder."""
149149

150150
def __init__(
151-
self,
152-
enabled: bool = None,
151+
self,
152+
enabled: bool = None,
153153
):
154154
super().__init__()
155155
self.enabled = Resource.init_param(enabled, default=True)
@@ -362,7 +362,9 @@ def _set_default(self, config: ImageBuilderConfig):
362362
dev_settings.slurm_patches_s3_archive if dev_settings and dev_settings.slurm_patches_s3_archive else ""
363363
)
364364
self.base_os = "{{ build.OperatingSystemName.outputs.stdout }}"
365-
self.disable_kernel_update = {"enabled": "yes"} if config.build.disable_kernel_update.enabled else {"enabled": "no"}
365+
self.disable_kernel_update = (
366+
{"enabled": "yes"} if config.build.disable_kernel_update.enabled else {"enabled": "no"}
367+
)
366368
for key, value in self.__dict__.items():
367369
if not key.startswith("_") and key not in self._cluster_attributes:
368370
self._cluster_attributes.update({key: value})

cli/src/pcluster/templates/imagebuilder_stack.py

+12-5
Original file line numberDiff line numberDiff line change
@@ -388,13 +388,21 @@ def _add_imagebuilder_distribution_configuration(self, ami_tags, build_tags, lam
388388
def _add_imagebuilder_user_data_override(self):
389389

390390
additional_instance_configuration_property = None
391-
if self.config.build and self.config.build.disable_kernel_update and self.config.build.disable_kernel_update.enabled:
392-
imagebuilder_user_data = os.path.join(imagebuilder_utils.get_resources_directory(), "imagebuilder", "user_data.sh")
391+
if (
392+
self.config.build
393+
and self.config.build.disable_kernel_update
394+
and self.config.build.disable_kernel_update.enabled
395+
):
396+
imagebuilder_user_data = os.path.join(
397+
imagebuilder_utils.get_resources_directory(), "imagebuilder", "user_data.sh"
398+
)
393399
with open(imagebuilder_user_data, "r", encoding="utf-8") as user_data_file:
394400
user_data_content = user_data_file.read()
395401

396-
additional_instance_configuration_property=imagebuilder.CfnImageRecipe.AdditionalInstanceConfigurationProperty(
397-
user_data_override=Fn.base64(user_data_content)
402+
additional_instance_configuration_property = (
403+
imagebuilder.CfnImageRecipe.AdditionalInstanceConfigurationProperty(
404+
user_data_override=Fn.base64(user_data_content)
405+
)
398406
)
399407
return additional_instance_configuration_property
400408

@@ -432,7 +440,6 @@ def _add_imagebuilder_image_recipe(self, build_tags, components, lambda_cleanup_
432440

433441
return image_recipe_resource
434442

435-
436443
def _add_imagebuilder_components(self, build_tags, lambda_cleanup_policy_statements):
437444
imagebuilder_resources_dir = os.path.join(imagebuilder_utils.get_resources_directory(), "imagebuilder")
438445

cli/tests/pcluster/config/dummy_imagebuilder_config.py

+2-1
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
AdditionalIamPolicy,
1414
Build,
1515
Component,
16+
DisableKernelUpdate,
1617
DistributionConfiguration,
1718
Iam,
1819
Image,
@@ -23,7 +24,7 @@
2324
LustreClient,
2425
NvidiaSoftware,
2526
UpdateOsPackages,
26-
Volume, DisableKernelUpdate,
27+
Volume,
2728
)
2829

2930
CLASS_DICT = {

0 commit comments

Comments
 (0)