UPDATE CFN PARAM IN JINJA FILE (#198)

* UPDATE CFN PARAM IN JINJA FILE

**Description**
Updated cfn cluster creation template.

**Testing Done**

* FIX UNIT TEST CASES FOR CFN PARAM

**Description**
Updated the unit test cases for the process cfn param util.

**Testing Done**
All the unit test cases pass.

* Remove unused function and fix CloudFormation template issues

**Description**
- Removed redundant _process_cfn_template_content function from init_utils.py
- Fixed missing InstanceGroupSettings1 and RigSettings1 parameters in CFN template by changing loop range from (2,21) to (1,21)
- Removed duplicate load_config_and_validate function definition

**Testing Done**
- Verified CloudFormation template generates all required parameters 1-20
- Confirmed no duplicate function definitions remain
- Updated unit test cases and the whole suite passes

Author: papriwal

src/sagemaker/hyperpod/cli/init_utils.py

@@ -39,35 +39,11 @@ def save_cfn_jinja(directory: str, content: str):
     Path(directory).mkdir(parents=True, exist_ok=True)
     path = os.path.join(directory, "cfn_params.jinja")
 
-    # Process content to handle array conversion
-    processed_content = _process_cfn_template_content(content)
-    
     with open(path, "w", encoding="utf-8") as f:
-        f.write(processed_content)
+        f.write(content)
     click.secho(f"Cloudformation Parameters Jinja template saved to: {path}")
     return path
 
-def _process_cfn_template_content(content: str) -> str:
-    """
-    Process CFN template content to include the full CloudFormation template.
-    """
-    try:
-        # Get the full CloudFormation template
-        full_template = HpClusterStack.get_template()
-    except Exception as e:
-        # Output the error and format as JSON in the file
-        click.secho(f"⚠️ Failed to generate CloudFormation template: {str(e)}", fg="red")        
-        full_template = ""
-    
-    # Add the complete template
-    template_content = f"""
-# CloudFormation Template:
-{full_template}
-"""
-    
-    # Insert the template at the beginning
-    return template_content + "\n" + content
-
 def save_k8s_jinja(directory: str, content: str):
     Path(directory).mkdir(parents=True, exist_ok=True)
     path = os.path.join(directory, "k8s.jinja")
@@ -530,28 +506,6 @@ def load_config_and_validate(dir_path: Path = None) -> Tuple[dict, str, str]:
     return data, template, version
 
 
-def load_config_and_validate(dir_path: Path = None) -> Tuple[dict, str, str]:
-    """
-    Load config.yaml, validate it exists, and extract template and version.
-    Returns (config_data, template, version)
-    Exits on validation errors - use for commands that require valid config.
-    """
-    data, template, version = load_config(dir_path)
-    validation_errors = validate_config_against_model(data, template, version)
-    
-    is_valid = display_validation_results(
-        validation_errors, 
-        success_message="config.yaml is valid!",
-        error_prefix="Config validation errors:"
-    )
-    
-    if not is_valid:
-        sys.exit(1)
-
-        
-    return data, template, version
-
-
 def validate_config_against_model(config_data: dict, template: str, version: str) -> list:
     """
     Validate config data against the appropriate Pydantic model.
@@ -905,4 +859,4 @@ def pascal_to_kebab(pascal_str):
         if char.isupper() and i > 0:
             result.append('-')
         result.append(char.lower())
-    return ''.join(result)
+    return ''.join(result)