diff --git a/config.tf b/config.tf
index 9476676..afbc3b8 100644
--- a/config.tf
+++ b/config.tf
@@ -3,11 +3,17 @@ terraform {
     encrypt = true
     bucket  = "terraform-persistence"
     key     = "terraform-emr-pyspark.tfstate"
-    region  = "eu-central-1"
+    region  = "us-west-2"
+  }
+
+  required_providers {
+    aws = {
+      source  = "hashicorp/aws"
+      version = "~> 5.0"
+    }
   }
 }
 
 provider "aws" {
-  version    = "~> 1.0"
-  region     = "${var.region}"
+  region = var.region
 }
diff --git a/main.tf b/main.tf
index 69bbb2e..3f950aa 100644
--- a/main.tf
+++ b/main.tf
@@ -1,6 +1,6 @@
 module "s3" {
   source = "./modules/s3"
-  name   = "${var.name}"
+  name   = var.name
 }
 
 module "iam" {
@@ -9,26 +9,26 @@ module "iam" {
 
 module "security" {
   source              = "./modules/security"
-  name                = "${var.name}"
-  vpc_id              = "${var.vpc_id}"
-  ingress_cidr_blocks = "${var.ingress_cidr_blocks}"
+  name                = var.name
+  vpc_id              = var.vpc_id
+  ingress_cidr_blocks = var.ingress_cidr_blocks
 }
 
 module "emr" {
   source                    = "./modules/emr"
-  name                      = "${var.name}"
-  release_label             = "${var.release_label}"
-  applications              = "${var.applications}"
-  subnet_id                 = "${var.subnet_id}"
-  key_name                  = "${var.key_name}"
-  master_instance_type      = "${var.master_instance_type}"
-  master_ebs_size           = "${var.master_ebs_size}"
-  core_instance_type        = "${var.core_instance_type}"
-  core_instance_count       = "${var.core_instance_count}"
-  core_ebs_size             = "${var.core_ebs_size}"
-  emr_master_security_group = "${module.security.emr_master_security_group}"
-  emr_slave_security_group  = "${module.security.emr_slave_security_group}"
-  emr_ec2_instance_profile  = "${module.iam.emr_ec2_instance_profile}"
-  emr_service_role          = "${module.iam.emr_service_role}"
-  emr_autoscaling_role      = "${module.iam.emr_autoscaling_role}"
+  name                      = var.name
+  release_label             = var.release_label
+  applications              = var.applications
+  subnet_id                 = var.subnet_id
+  key_name                  = var.key_name
+  master_instance_type      = var.master_instance_type
+  master_ebs_size           = var.master_ebs_size
+  core_instance_type        = var.core_instance_type
+  core_instance_count       = var.core_instance_count
+  core_ebs_size             = var.core_ebs_size
+  emr_master_security_group = module.security.emr_master_security_group
+  emr_slave_security_group  = module.security.emr_slave_security_group
+  emr_ec2_instance_profile  = module.iam.emr_ec2_instance_profile
+  emr_service_role          = module.iam.emr_service_role
+  emr_autoscaling_role      = module.iam.emr_autoscaling_role
 }
diff --git a/modules/emr/main.tf b/modules/emr/main.tf
index a0ffebf..9c9004c 100644
--- a/modules/emr/main.tf
+++ b/modules/emr/main.tf
@@ -1,88 +1,83 @@
 resource "aws_emr_cluster" "emr-spark-cluster" {
-  name                              = "${var.name}"
-  release_label                     = "${var.release_label}"
-  applications                      = "${var.applications}"
+  name                              = var.name
+  release_label                     = var.release_label
+  applications                      = var.applications
   termination_protection            = false
   keep_job_flow_alive_when_no_steps = true
 
   ec2_attributes {
-    subnet_id                         = "${var.subnet_id}"
-    key_name                          = "${var.key_name}"
-    emr_managed_master_security_group = "${var.emr_master_security_group}"
-    emr_managed_slave_security_group  = "${var.emr_slave_security_group}"
-    instance_profile                  = "${var.emr_ec2_instance_profile}"
+    subnet_id                         = var.subnet_id
+    key_name                          = var.key_name
+    emr_managed_master_security_group = var.emr_master_security_group
+    emr_managed_slave_security_group  = var.emr_slave_security_group
+    instance_profile                  = var.emr_ec2_instance_profile
   }
 
-  ebs_root_volume_size = "12"
-
-  instance_group {
+  master_instance_group {
+    instance_type  = var.master_instance_type
+    instance_count = 1
     name           = "EMR master"
-    instance_role  = "MASTER"
-    instance_type  = "${var.master_instance_type}"
-    instance_count = "1"
 
     ebs_config {
-      size                 = "${var.master_ebs_size}"
+      size                 = var.master_ebs_size
       type                 = "gp2"
       volumes_per_instance = 1
     }
   }
 
-  instance_group {
+  core_instance_group {
+    instance_type  = var.core_instance_type
+    instance_count = var.core_instance_count
     name           = "EMR slave"
-    instance_role  = "CORE"
-    instance_type  = "${var.core_instance_type}"
-    instance_count = "${var.core_instance_count}"
 
     ebs_config {
-      size                 = "${var.core_ebs_size}"
+      size                 = var.core_ebs_size
       type                 = "gp2"
       volumes_per_instance = 1
     }
   }
 
-  tags {
+  tags = {
     Name = "${var.name} - Spark cluster"
   }
 
-  service_role     = "${var.emr_service_role}"
-  autoscaling_role = "${var.emr_autoscaling_role}"
+  service_role     = var.emr_service_role
+  autoscaling_role = var.emr_autoscaling_role
 
   bootstrap_action {
     name = "Bootstrap setup."
     path = "s3://${var.name}/scripts/bootstrap_actions.sh"
   }
 
-  step = [
-    {
-      name              = "Copy script file from s3."
-      action_on_failure = "CONTINUE"
+  step {
+    name              = "Copy script file from s3."
+    action_on_failure = "CONTINUE"
 
-      hadoop_jar_step {
-        jar  = "command-runner.jar"
-        args = ["aws", "s3", "cp", "s3://${var.name}/scripts/pyspark_quick_setup.sh", "/home/hadoop/"]
-      }
-    },
-    {
-      name              = "Setup pyspark with conda."
-      action_on_failure = "CONTINUE"
+    hadoop_jar_step {
+      jar  = "command-runner.jar"
+      args = ["aws", "s3", "cp", "s3://${var.name}/scripts/pyspark_quick_setup.sh", "/home/hadoop/"]
+    }
+  }
 
-      hadoop_jar_step {
-        jar  = "command-runner.jar"
-        args = ["sudo", "bash", "/home/hadoop/pyspark_quick_setup.sh"]
-      }
-    },
-  ]
+  step {
+    name              = "Setup pyspark with conda."
+    action_on_failure = "CONTINUE"
+
+    hadoop_jar_step {
+      jar  = "command-runner.jar"
+      args = ["sudo", "bash", "/home/hadoop/pyspark_quick_setup.sh"]
+    }
+  }
 
   configurations_json = <<EOF
-    [
-    {
+[
+  {
     "Classification": "spark-defaults",
-      "Properties": {
+    "Properties": {
       "maximizeResourceAllocation": "true",
       "spark.dynamicAllocation.enabled": "true"
-      }
     }
-  ]
-  EOF
+  }
+]
+EOF
 }
diff --git a/modules/emr/variables.tf b/modules/emr/variables.tf
index 3915b38..64bc4ae 100644
--- a/modules/emr/variables.tf
+++ b/modules/emr/variables.tf
@@ -3,7 +3,7 @@ variable "subnet_id" {}
 variable "key_name" {}
 variable "release_label" {}
 variable "applications" {
-  type = "list"
+  type = list(string)
 }
 variable "master_instance_type" {}
 variable "master_ebs_size" {}
diff --git a/modules/s3/main.tf b/modules/s3/main.tf
index 3d952cb..6200b51 100644
--- a/modules/s3/main.tf
+++ b/modules/s3/main.tf
@@ -1,6 +1,5 @@
 resource "aws_s3_bucket" "create_bucket" {
   bucket = "${var.name}"
-  acl    = "private"
 
   tags = {
     Name        = "Bucket for EMR Bootstrap actions/Steps"
@@ -8,16 +7,31 @@ resource "aws_s3_bucket" "create_bucket" {
   }
 }
 
-resource "aws_s3_bucket_object" "bootstrap_action_file" {
+resource "aws_s3_bucket_ownership_controls" "create_bucket" {
+  bucket = aws_s3_bucket.create_bucket.id
+
+  rule {
+    object_ownership = "BucketOwnerPreferred"
+  }
+}
+
+resource "aws_s3_bucket_acl" "create_bucket" {
+  depends_on = [aws_s3_bucket_ownership_controls.create_bucket]
+
+  bucket = aws_s3_bucket.create_bucket.id
+  acl    = "private"
+}
+
+resource "aws_s3_object" "bootstrap_action_file" {
   bucket     = "${var.name}"
   key        = "scripts/bootstrap_actions.sh"
   source     = "scripts/bootstrap_actions.sh"
-  depends_on = ["aws_s3_bucket.create_bucket"]
+  depends_on = [aws_s3_bucket.create_bucket]
 }
 
-resource "aws_s3_bucket_object" "pyspark_quick_setup_file" {
+resource "aws_s3_object" "pyspark_quick_setup_file" {
   bucket     = "${var.name}"
   key        = "scripts/pyspark_quick_setup.sh"
   source     = "scripts/pyspark_quick_setup.sh"
-  depends_on = ["aws_s3_bucket.create_bucket"]
+  depends_on = [aws_s3_bucket.create_bucket]
 }
diff --git a/modules/security/main.tf b/modules/security/main.tf
index ea8f725..8154c74 100644
--- a/modules/security/main.tf
+++ b/modules/security/main.tf
@@ -1,35 +1,35 @@
 resource "aws_security_group" "emr_master" {
   name                   = "${var.name} - EMR-master"
   description            = "Security group for EMR master."
-  vpc_id                 = "${var.vpc_id}"
+  vpc_id                 = var.vpc_id
   revoke_rules_on_delete = true
 
   ingress {
     from_port   = 22
     to_port     = 22
     protocol    = "tcp"
-    cidr_blocks = ["${var.ingress_cidr_blocks}"]
+    cidr_blocks = var.ingress_cidr_blocks
   }
 
   ingress {
     from_port   = 4040
     to_port     = 4040
     protocol    = "tcp"
-    cidr_blocks = ["${var.ingress_cidr_blocks}"]
+    cidr_blocks = var.ingress_cidr_blocks
   }
 
   ingress {
     from_port   = 8888
     to_port     = 8888
     protocol    = "tcp"
-    cidr_blocks = ["${var.ingress_cidr_blocks}"]
+    cidr_blocks = var.ingress_cidr_blocks
   }
 
   ingress {
     from_port   = 20888
     to_port     = 20888
     protocol    = "tcp"
-    cidr_blocks = ["${var.ingress_cidr_blocks}"]
+    cidr_blocks = var.ingress_cidr_blocks
   }
 
   egress {
@@ -39,7 +39,7 @@ resource "aws_security_group" "emr_master" {
     cidr_blocks = ["0.0.0.0/0"]
   }
 
-  tags {
+  tags = {
     Name = "EMR_master"
   }
 }
@@ -47,14 +47,14 @@ resource "aws_security_group" "emr_master" {
 resource "aws_security_group" "emr_slave" {
   name                   = "${var.name} - EMR-slave"
   description            = "Security group for EMR slave."
-  vpc_id                 = "${var.vpc_id}"
+  vpc_id                 = var.vpc_id
   revoke_rules_on_delete = true
 
   ingress {
     from_port   = 22
     to_port     = 22
     protocol    = "tcp"
-    cidr_blocks = ["${var.ingress_cidr_blocks}"]
+    cidr_blocks = var.ingress_cidr_blocks
   }
 
   egress {
@@ -64,7 +64,7 @@ resource "aws_security_group" "emr_slave" {
     cidr_blocks = ["0.0.0.0/0"]
   }
 
-  tags {
+  tags = {
     Name = "EMR_slave"
   }
 }
diff --git a/modules/security/variables.tf b/modules/security/variables.tf
index 2db78e3..db1a98b 100644
--- a/modules/security/variables.tf
+++ b/modules/security/variables.tf
@@ -1,3 +1,7 @@
 variable "name" {}
 variable "vpc_id" {}
-variable "ingress_cidr_blocks" {}
+
+variable "ingress_cidr_blocks" {
+  description = "List of CIDR blocks to allow incoming traffic from"
+  type        = list(string)
+}
diff --git a/outputs.tf b/outputs.tf
index 01620fc..c19ba5a 100644
--- a/outputs.tf
+++ b/outputs.tf
@@ -1,11 +1,11 @@
 output "id" {
-  value = "${module.emr.id}"
+  value = module.emr.id
 }
 
 output "name" {
-  value = "${module.emr.name}"
+  value = module.emr.name
 }
 
 output "master_public_dns" {
-  value = "${module.emr.master_public_dns}"
+  value = module.emr.master_public_dns
 }
diff --git a/scripts/bootstrap_actions.sh b/scripts/bootstrap_actions.sh
index 2e64090..914a70a 100755
--- a/scripts/bootstrap_actions.sh
+++ b/scripts/bootstrap_actions.sh
@@ -1,15 +1,27 @@
 #!/usr/bin/env bash
 
-# install conda
-wget --quiet https://repo.continuum.io/miniconda/Miniconda3-latest-Linux-x86_64.sh -O ~/miniconda.sh \
-    && /bin/bash ~/miniconda.sh -b -p $HOME/conda
+# Ensure the script fails on any error
+set -e
 
-echo -e '\nexport PATH=$HOME/conda/bin:$PATH' >> $HOME/.bashrc && source $HOME/.bashrc
+# Install Miniconda
+MINICONDA_URL="https://repo.anaconda.com/miniconda/Miniconda3-latest-Linux-x86_64.sh"
+wget --quiet ${MINICONDA_URL} -O ~/miniconda.sh
+/bin/bash ~/miniconda.sh -b -p $HOME/conda
 
-# install packages
-conda install -y notebook=5.7.* jupyter=1.0.* pandas seaborn
+# Initialize Conda for bash shell. This will also update the .bashrc file.
+source $HOME/conda/bin/activate
+conda init bash
 
-#install findspark
+# Update conda
+conda update -n base -c defaults conda -y
+
+# Install packages
+conda install -y notebook jupyter pandas seaborn
+
+# Update pip and install Python packages
+conda install pip -y
 pip install --upgrade pip
-pip install findspark
-pip install sklearn
+pip install findspark sklearn
+
+# Cleanup
+# rm ~/miniconda.sh
diff --git a/scripts/pyspark_quick_setup.sh b/scripts/pyspark_quick_setup.sh
index c0526a4..ee09e86 100755
--- a/scripts/pyspark_quick_setup.sh
+++ b/scripts/pyspark_quick_setup.sh
@@ -1,6 +1,22 @@
 #!/usr/bin/env bash
 
-# bind conda to spark
-echo -e "\nexport PYSPARK_PYTHON=/home/hadoop/conda/bin/python" >> /etc/spark/conf/spark-env.sh
-echo "export PYSPARK_DRIVER_PYTHON=/home/hadoop/conda/bin/jupyter" >> /etc/spark/conf/spark-env.sh
-echo "export PYSPARK_DRIVER_PYTHON_OPTS='notebook --no-browser --port 8888 --ip=\"0.0.0.0\"'" >> /etc/spark/conf/spark-env.sh
+# Ensure the script fails on any error
+set -e
+
+# Set the paths for PySpark's Python and Jupyter Driver
+PYSPARK_PYTHON_PATH="/home/hadoop/conda/bin/python"
+PYSPARK_DRIVER_PATH="/home/hadoop/conda/bin/jupyter"
+PYSPARK_DRIVER_OPTS="notebook --no-browser --port 8888 --ip='0.0.0.0'"
+
+# Append configurations to spark-env.sh
+cat >> /etc/spark/conf/spark-env.sh << EOF
+
+# Bind PySpark to Conda Python installation
+export PYSPARK_PYTHON=${PYSPARK_PYTHON_PATH}
+
+# Set Jupyter as the driver for PySpark
+export PYSPARK_DRIVER_PYTHON=${PYSPARK_DRIVER_PATH}
+
+# Set options for the Jupyter driver
+export PYSPARK_DRIVER_PYTHON_OPTS='${PYSPARK_DRIVER_OPTS}'
+EOF
diff --git a/terraform.tfvars b/terraform.tfvars
index 86e96e9..2503c02 100644
--- a/terraform.tfvars
+++ b/terraform.tfvars
@@ -1,18 +1,18 @@
 # EMR general configurations
-name = "spark-app"
-region = "eu-central-1"
-subnet_id = "<subet_id>"
-vpc_id = "<vpc_id>"
-key_name = "<key_name>"
-ingress_cidr_blocks = "0.0.0.0/0"
-release_label = "emr-5.16.0"
-applications = ["Hadoop", "Spark"]
+name                = "spark-app"
+region              = "eu-central-1"
+subnet_id           = "<subet_id>"
+vpc_id              = "<vpc_id>"
+key_name            = "<key_name>"
+ingress_cidr_blocks = ["0.0.0.0/0"]
+release_label       = "emr-6.12.0"
+applications        = ["Hadoop", "Spark"]
 
 # Master node configurations
-master_instance_type = "m3.xlarge"
-master_ebs_size = "50"
+master_instance_type = "m5.xlarge"
+master_ebs_size      = "50"
 
 # Slave nodes configurations
-core_instance_type = "m3.xlarge"
+core_instance_type  = "m5.xlarge"
 core_instance_count = 1
-core_ebs_size = "50"
+core_ebs_size       = "50"
diff --git a/variables.tf b/variables.tf
index 668b008..c71f807 100644
--- a/variables.tf
+++ b/variables.tf
@@ -1,15 +1,64 @@
-variable "name" {}
-variable "region" {}
-variable "subnet_id" {}
-variable "vpc_id" {}
-variable "key_name" {}
-variable "release_label" {}
+variable "name" {
+  description = "Name for the EMR cluster"
+  type        = string
+}
+
+variable "region" {
+  description = "AWS region where resources will be created"
+  type        = string
+}
+
+variable "subnet_id" {
+  description = "ID of the subnet in which the EMR cluster will be launched"
+  type        = string
+}
+
+variable "vpc_id" {
+  description = "ID of the VPC in which the EMR cluster will reside"
+  type        = string
+}
+
+variable "key_name" {
+  description = "Name of the EC2 key pair for SSH access to the instances"
+  type        = string
+}
+
+variable "release_label" {
+  description = "Release label for the EMR cluster"
+  type        = string
+}
+
 variable "applications" {
-  type = "list"
-}
-variable "master_instance_type" {}
-variable "master_ebs_size" {}
-variable "core_instance_type" {}
-variable "core_instance_count" {}
-variable "core_ebs_size" {}
-variable "ingress_cidr_blocks" {}
+  description = "List of applications to be installed and configured on the EMR cluster"
+  type        = list(string)
+}
+
+variable "master_instance_type" {
+  description = "EC2 instance type for the master node"
+  type        = string
+}
+
+variable "master_ebs_size" {
+  description = "EBS volume size (in GB) for the master node"
+  type        = number
+}
+
+variable "core_instance_type" {
+  description = "EC2 instance type for the core nodes"
+  type        = string
+}
+
+variable "core_instance_count" {
+  description = "Number of core nodes in the EMR cluster"
+  type        = number
+}
+
+variable "core_ebs_size" {
+  description = "EBS volume size (in GB) for the core nodes"
+  type        = number
+}
+
+variable "ingress_cidr_blocks" {
+  description = "List of CIDR blocks to allow incoming traffic from"
+  type        = list(string)
+}