chore: Add linter for YAML files (#227)

.github/ISSUE_TEMPLATE/config.yaml

@@ -1 +1 @@
-blank_issues_enabled: false
+blank_issues_enabled: false

.github/workflows/check-integration-tests.yml

@@ -57,4 +57,4 @@ jobs:
           MODULE_PATH: ${{ matrix.modules.module_path }}
         with:
           filename: .github/ISSUE_TEMPLATE/missing_integ_tests_issue.md
-          update_existing: false
+          update_existing: false

.github/workflows/module-checks-dev-requirements-change.yml

@@ -2,7 +2,7 @@ name: Module Checks Dev Requirements Change
 
 on:
   push:
-    branches: 
+    branches:
       - "main"
     paths:
       - "requirements-dev.txt"
@@ -95,4 +95,4 @@ jobs:
         run: cd $MODULE_PATH/ && pytest
       - name: NPM Test
         if: ${{ steps.determine-language.outputs.language == 'typescript' }}
-        run: cd $MODULE_PATH/ && npm test
+        run: cd $MODULE_PATH/ && npm test

.github/workflows/module-checks.yml

@@ -2,7 +2,7 @@ name: Module Checks
 
 on:
   push:
-    branches: 
+    branches:
       - "main"
     paths:
       - "modules/**"

.github/workflows/mwaa-requirements.check.yml

@@ -2,7 +2,7 @@ name: MWAA Requirements Check
 
 on:
   push:
-    branches: 
+    branches:
       - "main"
     paths:
       - "data/mwaa/requirements/**"

.github/workflows/one-click-template-lint.yml

@@ -2,7 +2,7 @@ name: Check One-Click Launch Template
 
 on:
   push:
-    branches: 
+    branches:
       - "main"
     paths:
       - "one-click-launch.yaml"

.github/workflows/yaml-lint-check.yml

@@ -0,0 +1,27 @@
+name: Check YAML Files
+
+on:
+  workflow_dispatch:
+
+  push:
+    branches:
+      - "main"
+
+  pull_request:
+    branches:
+      - "main"
+      - "release/*"
+      - "stable"
+
+jobs:
+  lint:
+    name: YAML Lint
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Install yamllint
+        run: pip install yamllint
+
+      - name: Lint YAML files
+        run: yamllint --format github .

.yamllint.yaml

@@ -0,0 +1,9 @@
+---
+extends: default
+
+ignore-from-file: .gitignore
+
+rules:
+  line-length: disable
+  document-start: disable
+  truthy: disable

examples/manifests/deployment.yaml

@@ -33,4 +33,4 @@ targetAccountMappings:
         envVariable: TOOLING_ACCOUNT
     regionMappings:
       - region: us-east-1
-        default: true
+        default: true

examples/manifests/event-bus-modules.yaml

@@ -5,6 +5,6 @@ parameters:
   - name: event_bus_name
     value: mlops-bus
   - name: source_accounts
-    value: '["123123123123"]' # Accounts that must have permissions to put events (source accounts)
+    value: '["123123123123"]'  # Accounts that must have permissions to put events (source accounts)
   - name: tags
-    value: '{"test": "test"}'
+    value: '{"test": "test"}'

examples/manifests/fmops-modules.yaml

@@ -17,4 +17,4 @@ parameters:
       moduleMetadata:
         group: networking
         name: networking
-        key: PrivateSubnetIds
+        key: PrivateSubnetIds

examples/manifests/networking-modules.yaml

@@ -3,4 +3,4 @@ path: git::https://github.com/awslabs/idf-modules.git//modules/network/basic-cdk
 targetAccount: primary
 parameters:
   - name: internet-accessible
-    value: True
+    value: True

examples/manifests/personas-modules.yaml

@@ -2,4 +2,4 @@ name: personas
 path: git::https://github.com/awslabs/aiops-modules.git//modules/examples/personas?ref=release/1.4.0&depth=1
 parameters:
   - name: bucket-name
-    value: my-bucket
+    value: my-bucket

examples/manifests/sagemaker-endpoints-modules.yaml

@@ -38,4 +38,4 @@ parameters:
   - name: managed_instance_scaling
     value: True
   - name: scaling_max_instance_count
-    value: 10
+    value: 10

examples/manifests/sagemaker-hugging-face.yml

@@ -21,4 +21,4 @@ parameters:
       moduleMetadata:
         group: networking
         name: networking
-        key: PrivateSubnetIds
+        key: PrivateSubnetIds

examples/manifests/sagemaker-model-monitoring-modules.yaml

@@ -38,4 +38,4 @@ parameters:
       moduleMetadata:
         group: networking
         name: networking
-        key: PrivateSubnetIds
+        key: PrivateSubnetIds

examples/manifests/sagemaker-model-package-group-modules.yaml

@@ -11,7 +11,7 @@ parameters:
         name: event-bus
         key: EventBusArn
   - name: target_account_ids
-    value: '["444333222555"]' # Accounts that must have read-only permissions on the model pkg group
+    value: '["444333222555"]'  # Accounts that must have read-only permissions on the model pkg group
   - name: sagemaker_project_id
     value: 123123
   - name: sagemaker_project_name
@@ -26,8 +26,8 @@ parameters:
   - name: model_package_group_description
     value: Test model package group module - Target
   - name: target_account_ids
-    value: '["111222333444"]' # Accounts that must have read-only permissions on the model pkg group
+    value: '["111222333444"]'  # Accounts that must have read-only permissions on the model pkg group
   - name: sagemaker_project_id
     value: 123123
   - name: sagemaker_project_name
-    value: test
+    value: test

examples/manifests/sagemaker-model-package-promote-pipeline-modules.yaml

@@ -21,4 +21,4 @@ parameters:
       moduleMetadata:
         group: registry
         name: target-model-package-group
-        key: SagemakerModelPackageGroupName
+        key: SagemakerModelPackageGroupName

examples/manifests/sagemaker-notebook-modules.yaml

@@ -5,15 +5,15 @@ parameters:
     value: dummy
   - name: instance_type
     value: ml.t2.xlarge
-  - name: subnet_ids # Optional parameter, you can remove it safely
+  - name: subnet_ids  # Optional parameter, you can remove it safely
     valueFrom:
       moduleMetadata:
         group: networking
         name: networking
         key: PrivateSubnetIds
-  - name: vpc_id # Optional parameter, you can remove it safely
+  - name: vpc_id  # Optional parameter, you can remove it safely
     valueFrom:
       moduleMetadata:
         group: networking
         name: networking
-        key: VpcId
+        key: VpcId

manifests/bedrock-finetuning-sfn/bedrock-finetuning-modules.yaml

@@ -17,4 +17,4 @@ parameters:
         name: networking
         key: PrivateSubnetIds
   - name: removal-policy
-    value: DESTROY
+    value: DESTROY

manifests/bedrock-finetuning-sfn/deployment.yaml

@@ -14,4 +14,4 @@ targetAccountMappings:
     default: true
     regionMappings:
       - region: us-east-1
-        default: true
+        default: true

manifests/bedrock-finetuning-sfn/networking-modules.yaml

@@ -3,4 +3,4 @@ path: git::https://github.com/awslabs/idf-modules.git//modules/network/basic-cdk
 targetAccount: primary
 parameters:
   - name: internet-accessible
-    value: True
+    value: True

manifests/fmops-qna-rag/deployment.yaml

@@ -16,4 +16,4 @@ targetAccountMappings:
     default: true
     regionMappings:
       - region: us-east-1
-        default: true
+        default: true

manifests/fmops-qna-rag/networking-modules.yaml

@@ -2,4 +2,4 @@ name: networking
 path: git::https://github.com/awslabs/idf-modules.git//modules/network/basic-cdk?ref=release/1.11.0&depth=1
 parameters:
   - name: internet-accessible
-    value: True
+    value: True

manifests/fmops-qna-rag/qna-rag-modules.yaml

@@ -2,7 +2,7 @@ name: qna-rag
 path: git::https://github.com/awslabs/aiops-modules.git//modules/fmops/qna-rag?ref=release/1.4.0&depth=1
 parameters:
   - name: cognito-pool-id
-    #Replace below value with valid congnito pool id
+    # Replace below value with valid congnito pool id
     value: us-east-1_XXXXX
   - name: os-domain-endpoint
     valueFrom:
@@ -21,4 +21,4 @@ parameters:
       moduleMetadata:
         group: networking
         name: networking
-        key: VpcId
+        key: VpcId

manifests/fmops-qna-rag/storage-modules.yaml

@@ -16,4 +16,4 @@ parameters:
       moduleMetadata:
         group: networking
         name: networking
-        key: PrivateSubnetIds
+        key: PrivateSubnetIds

manifests/mlflow-experiments-tracking/deployment.yaml

@@ -20,4 +20,4 @@ targetAccountMappings:
     default: true
     regionMappings:
       - region: us-east-1
-        default: true
+        default: true

manifests/mlflow-experiments-tracking/images-modules.yaml

@@ -7,4 +7,4 @@ parameters:
       moduleMetadata:
         group: storage
         name: ecr-mlflow
-        key: EcrRepositoryName
+        key: EcrRepositoryName

manifests/mlflow-experiments-tracking/mlflow-modules.yaml

@@ -51,4 +51,4 @@ parameters:
         name: mlflow-mysql
         key: SecurityGroupId
   - name: efs-removal-policy
-    value: DESTROY
+    value: DESTROY

manifests/mlflow-experiments-tracking/networking-modules.yaml

@@ -3,4 +3,4 @@ path: git::https://github.com/awslabs/idf-modules.git//modules/network/basic-cdk
 targetAccount: primary
 parameters:
   - name: internet-accessible
-    value: True
+    value: True

manifests/mlflow-experiments-tracking/sagemaker-studio-modules.yaml

@@ -16,15 +16,15 @@ parameters:
         key: PrivateSubnetIds
   - name: data_science_users
     value:
-    - ds-user-1
+      - ds-user-1
   - name: lead_data_science_users
     value:
-    - lead-ds-user-1
+      - lead-ds-user-1
   - name: server_lifecycle_name
     value: studio-auto-shutdown
   - name: studio_bucket_name
     value: mlops-*
   - name: retain_efs
     value: 'False'
   - name: enable_custom_sagemaker_projects
-    value: 'True'
+    value: 'True'

manifests/mlops-sagemaker-multiacc/deployment.yaml

@@ -43,4 +43,4 @@ targetAccountMappings:
         envVariable: PROD_ACCOUNT
     regionMappings:
       - region: us-east-1
-        default: true
+        default: true

manifests/mlops-sagemaker-multiacc/kernels-modules.yaml

@@ -37,4 +37,4 @@ parameters:
   - name: kernel-user-gid
     value: '0'
   - name: kernel-user-home-mount-path
-    value: /root
+    value: /root

manifests/mlops-sagemaker-multiacc/networking-modules.yaml

@@ -17,4 +17,4 @@ path: git::https://github.com/awslabs/idf-modules.git//modules/network/basic-cdk
 targetAccount: prod
 parameters:
   - name: internet-accessible
-    value: True
+    value: True

manifests/mlops-sagemaker-multiacc/sagemaker-model-cicd-modules.yaml

@@ -1,4 +1,4 @@
-name: model # replace with name of the ML model you prefer
+name: model  # replace with name of the ML model you prefer
 path: git::https://github.com/awslabs/aiops-modules.git//modules/sagemaker/sagemaker-model-cicd?ref=release/1.4.0&depth=1
 targetAccount: tooling
 parameters:

manifests/mlops-sagemaker-multiacc/sagemaker-studio-modules.yaml

@@ -16,15 +16,15 @@ parameters:
         key: PrivateSubnetIds
   - name: data_science_users
     value:
-    - ds-user-1
+      - ds-user-1
   - name: lead_data_science_users
     value:
-    - lead-ds-user-1
+      - lead-ds-user-1
   - name: server_lifecycle_name
     value: studio-auto-shutdown
   - name: studio_bucket_name
     value: mlops-*
   - name: retain_efs
     value: 'False'
   - name: enable_custom_sagemaker_projects
-    value: 'True'
+    value: 'True'

manifests/mlops-sagemaker-multiacc/sagemaker-templates-modules.yaml

@@ -61,4 +61,4 @@ parameters:
       moduleMetadata:
         group: networking
         name: networking-prod
-        key: PrivateSubnetIds
+        key: PrivateSubnetIds

manifests/mlops-sagemaker-multiacc/storage-modules.yaml

@@ -18,4 +18,4 @@ parameters:
   - name: encryption-type
     value: KMS
   - name: retention-type
-    value: DESTROY
+    value: DESTROY

manifests/mlops-sagemaker/deployment.yaml

@@ -20,4 +20,4 @@ targetAccountMappings:
     default: true
     regionMappings:
       - region: us-east-1
-        default: true
+        default: true

manifests/mlops-sagemaker/kernels-modules.yaml

@@ -37,4 +37,4 @@ parameters:
   - name: kernel-user-gid
     value: '0'
   - name: kernel-user-home-mount-path
-    value: /root
+    value: /root

manifests/mlops-sagemaker/networking-modules.yaml

@@ -3,4 +3,4 @@ path: git::https://github.com/awslabs/idf-modules.git//modules/network/basic-cdk
 targetAccount: primary
 parameters:
   - name: internet-accessible
-    value: True
+    value: True