Plan tests (#166)

* Updated plan-examples.yml to support e2e tests for multi cluster example (#165)

* Updated plan-examples.yml to support e2e tests for multi cluster example

* Put the variables in right place with right values

* (to drop): Allow running tests in branch

---------

Co-authored-by: Rodrigue Koffi <bonclay7@users.noreply.github.com>

* Force running tests

* Update plan-examples.yml

---------

Co-authored-by: Vijay Chintalapati <Vijay.Chintalapati@gmail.com>

.github/workflows/plan-examples.yml

@@ -5,6 +5,10 @@ on:
   pull_request_target:
     branches:
       - main
+  # this will allow to run test on main and not modify the status badge when opening a PR
+  push:
+    branches: ["main"]
+    
   workflow_dispatch:
 
 concurrency:
@@ -90,6 +94,8 @@ jobs:
           TF_VAR_aws_region: us-west-2
           TF_VAR_managed_grafana_workspace_id: ${{ secrets.GRAFANA_WORKSPACE_ID }}
           TF_VAR_eks_cluster_id: e2e-tests
+          TF_VAR_eks_cluster_1_id: e2e-tests
+          TF_VAR_eks_cluster_2_id: e2e-tests
           TF_VAR_managed_prometheus_workspace_ids: ${{ secrets.PROMETHEUS_WORKSPACE_ID }}
         with:
           project-name: terraform-plan
@@ -98,7 +104,9 @@ jobs:
             TF_VAR_aws_region,
             TF_VAR_managed_grafana_workspace_id,
             TF_VAR_eks_cluster_id,
-            TF_VAR_managed_prometheus_workspace_ids
+            TF_VAR_managed_prometheus_workspace_ids,
+            TF_VAR_eks_cluster_1_id,
+            TF_VAR_eks_cluster_2_id
           buildspec-override: |
             version: 0.2
             phases:

.github/workflows/pre-commit.yaml

@@ -8,6 +8,12 @@ on:
       - '**.tf'
       - '**.yml'
       - '**.yaml'
+  push:
+    branches: ["main"]
+    paths:
+      - '**.tf'
+      - '**.yml'
+      - '**.yaml'
 
 env:
   TERRAFORM_DOCS_VERSION: v0.16.0

examples/eks-multicluster/main.tf

@@ -65,8 +65,8 @@ module "eks_cluster_2_monitoring" {
   enable_cert_manager    = true
   enable_java            = true
 
-  # Since the following were enabled in conjunction with the set up of the eks_cluster_1 EKS cluster, we will skip
+  # Since the following were enabled in conjunction with the set up of the
-  # them with the eks_cluster_2 EKS cluster
+  # eks_cluster_1 EKS cluster, we will skip them with the eks_cluster_2 EKS cluster
   enable_dashboards      = false
   enable_alerting_rules  = false
   enable_recording_rules = false