Fix reference to empty eks resource when creating S3 (#92)

* only do the permission assignment when eks is enabled

* add changes to the src recipes

* revert changes on the legacy aws-modular recipe

* format file

Author: wjayesh

src/mlstacks/terraform/aws-modular/eks.tf

@@ -85,7 +85,7 @@ resource "aws_eks_node_group" "nodegroup" {
 }
 
 resource "aws_iam_role" "ng" {
-  count = local.enable_eks || var.enable_artifact_store ? 1 : 0
+  count = local.enable_eks ? 1 : 0
 
   name_prefix = "${local.prefix}-${local.eks.cluster_name}-ng"
   assume_role_policy = jsonencode({

src/mlstacks/terraform/aws-modular/s3.tf

@@ -25,13 +25,13 @@ resource "aws_s3_bucket_public_access_block" "example" {
 }
 
 resource "aws_s3_bucket_policy" "allow_access_from_another_account" {
-  count  = var.enable_artifact_store ? 1 : 0
+  count  = local.enable_eks ? 1 : 0
   bucket = aws_s3_bucket.zenml-artifact-store[0].id
   policy = data.aws_iam_policy_document.allow_access_from_another_account[0].json
 }
 
 data "aws_iam_policy_document" "allow_access_from_another_account" {
-  count = var.enable_artifact_store ? 1 : 0
+  count = local.enable_eks ? 1 : 0
   statement {
     principals {
       type        = "AWS"