Fix pcluster update behavior with additional instance types data

Additional instance types data must be loaded not only when loading the configuration from file, but also from CloudFormation in order to allow the refresh operations to work correctly.

This commit also marks as ignored the update policy of the `instance_types_data` parameter to avoid changes being displayed just because of the json data re-ordering.

Signed-off-by: ddeidda <[email protected]>

Author: ddeidda

cli/src/pcluster/config/mappings.py

@@ -1032,7 +1032,7 @@
         "type": JsonCfnParam,
         "default": {},
         "cfn_param_mapping": "InstanceTypesData",
-        "update_policy": UpdatePolicy.SUPPORTED
+        "update_policy": UpdatePolicy.IGNORED
     }),
 ]
 

cli/src/pcluster/config/pcluster_config.py

@@ -101,8 +101,9 @@ def __init__(
             self.__init_sections_from_cfn(cluster_name)
         else:
             self.__init_sections_from_file(cluster_label, self.config_parser, fail_on_file_absence)
-            # Load instance types data if present in config file
-            self.__init_additional_instance_types_data()
+
+        # Load instance types data if available
+        self.__init_additional_instance_types_data()
 
         self.__autorefresh = auto_refresh  # Initialization completed