Skip to content

Commit

Permalink
Merge pull request #15 from abhi-dev91/main
Browse files Browse the repository at this point in the history
Removed unwanted variables for kms policy arn
  • Loading branch information
RohitSquareops authored Dec 1, 2023
2 parents 15931b5 + 7ed8ece commit 675e8a6
Show file tree
Hide file tree
Showing 5 changed files with 3 additions and 11 deletions.
1 change: 0 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -180,7 +180,6 @@ In this module, we have implemented the following CIS Compliance checks for EKS:
| <a name="input_image_high_threshold_percent"></a> [image\_high\_threshold\_percent](#input\_image\_high\_threshold\_percent) | The percentage of disk usage at which garbage collection should be triggered. | `number` | `60` | no |
| <a name="input_image_low_threshold_percent"></a> [image\_low\_threshold\_percent](#input\_image\_low\_threshold\_percent) | The percentage of disk usage at which garbage collection took place. | `number` | `40` | no |
| <a name="input_eventRecordQPS"></a> [eventRecordQPS](#input\_eventRecordQPS) | The maximum number of events created per second. | `number` | `5` | no |
| <a name="input_kms_policy_arn"></a> [kms\_policy\_arn](#input\_kms\_policy\_arn) | The KMS policy ARN used for encrypting Kubernetes PVC. | `string` | `""` | no |
| <a name="input_associate_public_ip_address"></a> [associate\_public\_ip\_address](#input\_associate\_public\_ip\_address) | Set to true to enable network interface for launch template. | `bool` | `false` | no |
| <a name="input_enable_monitoring"></a> [enable\_monitoring](#input\_enable\_monitoring) | Specify whether to enable monitoring for nodes. | `bool` | `true` | no |
| <a name="input_min_size"></a> [min\_size](#input\_min\_size) | The minimum number of nodes for the node group. | `string` | `"1"` | no |
Expand Down
3 changes: 1 addition & 2 deletions examples/complete-ipv6/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,6 @@ module "eks" {
cluster_log_types = ["api", "audit", "authenticator", "controllerManager", "scheduler"]
default_addon_enabled = local.default_addon_enabled
eks_nodes_keypair_name = module.key_pair_eks.key_pair_name
kms_policy_arn = module.eks.kms_policy_arn
private_subnet_ids = module.vpc.private_subnets
cluster_log_retention_in_days = 30
cluster_endpoint_public_access = true
Expand Down Expand Up @@ -171,7 +170,7 @@ module "managed_node_group_production" {
worker_iam_role_name = module.eks.worker_iam_role_name
eks_nodes_keypair_name = module.key_pair_eks.key_pair_name
k8s_labels = {
"Infra-Services" = "true"
"Addon-Services" = "true"
}
tags = local.additional_aws_tags
ipv6_enabled = local.ipv6_enabled
Expand Down
3 changes: 1 addition & 2 deletions examples/complete/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,6 @@ module "eks" {
create_aws_auth_configmap = true
default_addon_enabled = local.default_addon_enabled
eks_nodes_keypair_name = module.key_pair_eks.key_pair_name
kms_policy_arn = module.eks.kms_policy_arn
aws_auth_roles = [
{
rolearn = "arn:aws:iam::222222222222:role/service-role"
Expand Down Expand Up @@ -164,7 +163,7 @@ module "managed_node_group_production" {
worker_iam_role_name = module.eks.worker_iam_role_name
eks_nodes_keypair_name = module.key_pair_eks.key_pair_name
k8s_labels = {
"Infra-Services" = "true"
"Addon-Services" = "true"
}
tags = local.additional_aws_tags
}
2 changes: 1 addition & 1 deletion main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -218,7 +218,7 @@ EOF

resource "aws_iam_role_policy_attachment" "eks_kms_worker_policy_attachment" {
role = aws_iam_role.node_role.name
policy_arn = var.kms_policy_arn
policy_arn = aws_iam_policy.kubernetes_pvc_kms_policy.arn
}

resource "aws_iam_role_policy_attachment" "eks_worker_policy" {
Expand Down
5 changes: 0 additions & 5 deletions variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -141,11 +141,6 @@ variable "eventRecordQPS" {
default = 5
}

variable "kms_policy_arn" {
description = "The KMS policy ARN used for encrypting Kubernetes PVC."
type = string
default = ""
}

variable "associate_public_ip_address" {
description = "Set to true to enable network interface for launch template."
Expand Down

0 comments on commit 675e8a6

Please sign in to comment.