Skip to content

Conversation

@jai
Copy link
Contributor

@jai jai commented Jan 29, 2022

Info Required for All Commits

Description:

  • refactor(storage/rds): Enable enhanced logging
  • refactor(storage/rds): Enable multi-AZ on prod
  • refactor(storage/rds): Enable deletion protection
  • refactor(storage/rds): Enable enhanced monitoring

This change is Reviewable

@jai jai requested review from a team as code owners January 29, 2022 08:44
@jai jai requested a review from JoseFMP January 29, 2022 08:44
@jai jai requested a review from Judrummer January 29, 2022 08:44
@github-actions
Copy link

Terraform Format and Style 🖌success

Terraform Initialization ⚙️success

Terraform Validation 🤖Success! The configuration is valid.

Terraform Plan 📖failure

Show Plan
Running plan in the remote backend. Output will stream here. Pressing Ctrl-C
will stop streaming the logs, but will not stop the plan running remotely.

Preparing the remote plan...

To view this run in a browser, visit:
https://app.terraform.io/app/honestbank/argoflow-aws-infrastructure/runs/run-Gk6nh5s55XbZEwzn

Waiting for the plan to start...

Terraform v1.0.7
on linux_amd64
Configuring remote state backend...
Initializing Terraform configuration...
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 5:5: variable "aws_eks_cluster_primary_security_group_id" {
│ 
│ The root module input variable "aws_eks_cluster_primary_security_group_id"
│ is not set, and has no default value. Use a -var or -var-file command line
│ argument to provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 22:22: variable "aws_region" {
│ 
│ The root module input variable "aws_region" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 27:27: variable "aws_secretsmanager_account_id" {
│ 
│ The root module input variable "aws_secretsmanager_account_id" is not set,
│ and has no default value. Use a -var or -var-file command line argument to
│ provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 32:32: variable "aws_vpc_id" {
│ 
│ The root module input variable "aws_vpc_id" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 37:37: variable "aws_vpc_private_subnets" {
│ 
│ The root module input variable "aws_vpc_private_subnets" is not set, and
│ has no default value. Use a -var or -var-file command line argument to
│ provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 42:42: variable "aws_vpc_public_subnets" {
│ 
│ The root module input variable "aws_vpc_public_subnets" is not set, and has
│ no default value. Use a -var or -var-file command line argument to provide
│ a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 60:60: variable "eks_cluster_name" {
│ 
│ The root module input variable "eks_cluster_name" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 65:65: variable "kubeflow_cluster_oidc_provider_arn" {
│ 
│ The root module input variable "kubeflow_cluster_oidc_provider_arn" is not
│ set, and has no default value. Use a -var or -var-file command line
│ argument to provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 69:69: variable "route53_subdomain" {
│ 
│ The root module input variable "route53_subdomain" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 74:74: variable "stage" {
│ 
│ The root module input variable "stage" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.

Pusher: @jai, Action: pull_request, Working Directory: ``, Workflow: Terraform GitHub Actions

@jai jai requested a review from MXfive January 29, 2022 08:48
@jai jai marked this pull request as draft January 29, 2022 08:48
@github-actions
Copy link

Terraform Format and Style 🖌success

Terraform Initialization ⚙️success

Terraform Validation 🤖Success! The configuration is valid.

Terraform Plan 📖failure

Show Plan
Running plan in the remote backend. Output will stream here. Pressing Ctrl-C
will stop streaming the logs, but will not stop the plan running remotely.

Preparing the remote plan...

To view this run in a browser, visit:
https://app.terraform.io/app/honestbank/argoflow-aws-infrastructure/runs/run-5noVdZm2pysKNGBf

Waiting for the plan to start...

Terraform v1.0.7
on linux_amd64
Configuring remote state backend...
Initializing Terraform configuration...
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 5:5: variable "aws_eks_cluster_primary_security_group_id" {
│ 
│ The root module input variable "aws_eks_cluster_primary_security_group_id"
│ is not set, and has no default value. Use a -var or -var-file command line
│ argument to provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 22:22: variable "aws_region" {
│ 
│ The root module input variable "aws_region" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 27:27: variable "aws_secretsmanager_account_id" {
│ 
│ The root module input variable "aws_secretsmanager_account_id" is not set,
│ and has no default value. Use a -var or -var-file command line argument to
│ provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 32:32: variable "aws_vpc_id" {
│ 
│ The root module input variable "aws_vpc_id" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 37:37: variable "aws_vpc_private_subnets" {
│ 
│ The root module input variable "aws_vpc_private_subnets" is not set, and
│ has no default value. Use a -var or -var-file command line argument to
│ provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 42:42: variable "aws_vpc_public_subnets" {
│ 
│ The root module input variable "aws_vpc_public_subnets" is not set, and has
│ no default value. Use a -var or -var-file command line argument to provide
│ a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 60:60: variable "eks_cluster_name" {
│ 
│ The root module input variable "eks_cluster_name" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 65:65: variable "kubeflow_cluster_oidc_provider_arn" {
│ 
│ The root module input variable "kubeflow_cluster_oidc_provider_arn" is not
│ set, and has no default value. Use a -var or -var-file command line
│ argument to provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 69:69: variable "route53_subdomain" {
│ 
│ The root module input variable "route53_subdomain" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 74:74: variable "stage" {
│ 
│ The root module input variable "stage" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.

Pusher: @jai, Action: pull_request, Working Directory: ``, Workflow: Terraform GitHub Actions

@jai jai requested review from bludot and poom January 29, 2022 08:48
@jai jai added the WIP label Jan 29, 2022
@pull-request-size pull-request-size bot added size/L and removed size/M labels Jan 29, 2022
@github-actions
Copy link

Terraform Format and Style 🖌success

Terraform Initialization ⚙️success

Terraform Validation 🤖Success! The configuration is valid.

Terraform Plan 📖failure

Show Plan
Running plan in the remote backend. Output will stream here. Pressing Ctrl-C
will stop streaming the logs, but will not stop the plan running remotely.

Preparing the remote plan...

To view this run in a browser, visit:
https://app.terraform.io/app/honestbank/argoflow-aws-infrastructure/runs/run-9Jf9XPf9VAcWvP9p

Waiting for the plan to start...

Terraform v1.0.7
on linux_amd64
Configuring remote state backend...
Initializing Terraform configuration...
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 5:5: variable "aws_eks_cluster_primary_security_group_id" {
│ 
│ The root module input variable "aws_eks_cluster_primary_security_group_id"
│ is not set, and has no default value. Use a -var or -var-file command line
│ argument to provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 22:22: variable "aws_region" {
│ 
│ The root module input variable "aws_region" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 27:27: variable "aws_secretsmanager_account_id" {
│ 
│ The root module input variable "aws_secretsmanager_account_id" is not set,
│ and has no default value. Use a -var or -var-file command line argument to
│ provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 32:32: variable "aws_vpc_id" {
│ 
│ The root module input variable "aws_vpc_id" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 37:37: variable "aws_vpc_private_subnets" {
│ 
│ The root module input variable "aws_vpc_private_subnets" is not set, and
│ has no default value. Use a -var or -var-file command line argument to
│ provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 42:42: variable "aws_vpc_public_subnets" {
│ 
│ The root module input variable "aws_vpc_public_subnets" is not set, and has
│ no default value. Use a -var or -var-file command line argument to provide
│ a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 60:60: variable "eks_cluster_name" {
│ 
│ The root module input variable "eks_cluster_name" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 65:65: variable "kubeflow_cluster_oidc_provider_arn" {
│ 
│ The root module input variable "kubeflow_cluster_oidc_provider_arn" is not
│ set, and has no default value. Use a -var or -var-file command line
│ argument to provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 69:69: variable "route53_subdomain" {
│ 
│ The root module input variable "route53_subdomain" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 74:74: variable "stage" {
│ 
│ The root module input variable "stage" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.

Pusher: @jai, Action: pull_request, Working Directory: ``, Workflow: Terraform GitHub Actions

@github-actions
Copy link

Terraform Format and Style 🖌success

Terraform Initialization ⚙️success

Terraform Validation 🤖Success! The configuration is valid.

Terraform Plan 📖failure

Show Plan
Running plan in the remote backend. Output will stream here. Pressing Ctrl-C
will stop streaming the logs, but will not stop the plan running remotely.

Preparing the remote plan...

To view this run in a browser, visit:
https://app.terraform.io/app/honestbank/argoflow-aws-infrastructure/runs/run-NQdcEuT2Myvi9uuD

Waiting for the plan to start...

Terraform v1.0.7
on linux_amd64
Configuring remote state backend...
Initializing Terraform configuration...
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 5:5: variable "aws_eks_cluster_primary_security_group_id" {
│ 
│ The root module input variable "aws_eks_cluster_primary_security_group_id"
│ is not set, and has no default value. Use a -var or -var-file command line
│ argument to provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 22:22: variable "aws_region" {
│ 
│ The root module input variable "aws_region" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 27:27: variable "aws_secretsmanager_account_id" {
│ 
│ The root module input variable "aws_secretsmanager_account_id" is not set,
│ and has no default value. Use a -var or -var-file command line argument to
│ provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 32:32: variable "aws_vpc_id" {
│ 
│ The root module input variable "aws_vpc_id" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 37:37: variable "aws_vpc_private_subnets" {
│ 
│ The root module input variable "aws_vpc_private_subnets" is not set, and
│ has no default value. Use a -var or -var-file command line argument to
│ provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 42:42: variable "aws_vpc_public_subnets" {
│ 
│ The root module input variable "aws_vpc_public_subnets" is not set, and has
│ no default value. Use a -var or -var-file command line argument to provide
│ a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 60:60: variable "eks_cluster_name" {
│ 
│ The root module input variable "eks_cluster_name" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 65:65: variable "kubeflow_cluster_oidc_provider_arn" {
│ 
│ The root module input variable "kubeflow_cluster_oidc_provider_arn" is not
│ set, and has no default value. Use a -var or -var-file command line
│ argument to provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 69:69: variable "route53_subdomain" {
│ 
│ The root module input variable "route53_subdomain" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 74:74: variable "stage" {
│ 
│ The root module input variable "stage" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.

Pusher: @jai, Action: pull_request, Working Directory: ``, Workflow: Terraform GitHub Actions

@jai jai force-pushed the jai/spe-491-update-config-of-kubeflow-rds-instances branch from 71fe7c1 to 2b6273f Compare January 29, 2022 09:21
@github-actions
Copy link

Terraform Format and Style 🖌success

Terraform Initialization ⚙️success

Terraform Validation 🤖Success! The configuration is valid.

Terraform Plan 📖failure

Show Plan
Running plan in the remote backend. Output will stream here. Pressing Ctrl-C
will stop streaming the logs, but will not stop the plan running remotely.

Preparing the remote plan...

To view this run in a browser, visit:
https://app.terraform.io/app/honestbank/argoflow-aws-infrastructure/runs/run-1Jz5gisZEy5R9QVR

Waiting for the plan to start...

Terraform v1.0.7
on linux_amd64
Configuring remote state backend...
Initializing Terraform configuration...
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 5:5: variable "aws_eks_cluster_primary_security_group_id" {
│ 
│ The root module input variable "aws_eks_cluster_primary_security_group_id"
│ is not set, and has no default value. Use a -var or -var-file command line
│ argument to provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 22:22: variable "aws_region" {
│ 
│ The root module input variable "aws_region" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 27:27: variable "aws_secretsmanager_account_id" {
│ 
│ The root module input variable "aws_secretsmanager_account_id" is not set,
│ and has no default value. Use a -var or -var-file command line argument to
│ provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 32:32: variable "aws_vpc_id" {
│ 
│ The root module input variable "aws_vpc_id" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 37:37: variable "aws_vpc_private_subnets" {
│ 
│ The root module input variable "aws_vpc_private_subnets" is not set, and
│ has no default value. Use a -var or -var-file command line argument to
│ provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 42:42: variable "aws_vpc_public_subnets" {
│ 
│ The root module input variable "aws_vpc_public_subnets" is not set, and has
│ no default value. Use a -var or -var-file command line argument to provide
│ a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 60:60: variable "eks_cluster_name" {
│ 
│ The root module input variable "eks_cluster_name" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 65:65: variable "kubeflow_cluster_oidc_provider_arn" {
│ 
│ The root module input variable "kubeflow_cluster_oidc_provider_arn" is not
│ set, and has no default value. Use a -var or -var-file command line
│ argument to provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 69:69: variable "route53_subdomain" {
│ 
│ The root module input variable "route53_subdomain" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 74:74: variable "stage" {
│ 
│ The root module input variable "stage" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.

Pusher: @jai, Action: pull_request, Working Directory: ``, Workflow: Terraform GitHub Actions

@github-actions
Copy link

Terraform Format and Style 🖌success

Terraform Initialization ⚙️success

Terraform Validation 🤖Success! The configuration is valid.

Terraform Plan 📖failure

Show Plan
Running plan in the remote backend. Output will stream here. Pressing Ctrl-C
will stop streaming the logs, but will not stop the plan running remotely.

Preparing the remote plan...

To view this run in a browser, visit:
https://app.terraform.io/app/honestbank/argoflow-aws-infrastructure/runs/run-Ff9WBcRabNj7tRCj

Waiting for the plan to start...

Terraform v1.0.7
on linux_amd64
Configuring remote state backend...
Initializing Terraform configuration...
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 5:5: variable "aws_eks_cluster_primary_security_group_id" {
│ 
│ The root module input variable "aws_eks_cluster_primary_security_group_id"
│ is not set, and has no default value. Use a -var or -var-file command line
│ argument to provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 22:22: variable "aws_region" {
│ 
│ The root module input variable "aws_region" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 27:27: variable "aws_secretsmanager_account_id" {
│ 
│ The root module input variable "aws_secretsmanager_account_id" is not set,
│ and has no default value. Use a -var or -var-file command line argument to
│ provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 32:32: variable "aws_vpc_id" {
│ 
│ The root module input variable "aws_vpc_id" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 37:37: variable "aws_vpc_private_subnets" {
│ 
│ The root module input variable "aws_vpc_private_subnets" is not set, and
│ has no default value. Use a -var or -var-file command line argument to
│ provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 42:42: variable "aws_vpc_public_subnets" {
│ 
│ The root module input variable "aws_vpc_public_subnets" is not set, and has
│ no default value. Use a -var or -var-file command line argument to provide
│ a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 60:60: variable "eks_cluster_name" {
│ 
│ The root module input variable "eks_cluster_name" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 65:65: variable "kubeflow_cluster_oidc_provider_arn" {
│ 
│ The root module input variable "kubeflow_cluster_oidc_provider_arn" is not
│ set, and has no default value. Use a -var or -var-file command line
│ argument to provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 69:69: variable "route53_subdomain" {
│ 
│ The root module input variable "route53_subdomain" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on inputs.tf line 74:74: variable "stage" {
│ 
│ The root module input variable "stage" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.

Pusher: @jai, Action: pull_request, Working Directory: ``, Workflow: Terraform GitHub Actions

@jai jai marked this pull request as ready for review January 29, 2022 09:34
@jai jai changed the title feat(storage/rds): Fix Checkov findings feat(storage/rds): Fix Checkov findings [SPE-491, SPE-488, SPE-489, SPE-91] Jan 29, 2022
username = "kubeflow"
password = "kubeflow" #tfsec:ignore:GEN003
username = "kubeflow"
password = "kubeflow" #tfsec:ignore:GEN003
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Local exec provisioner to run an atomic password changing script via a null resource that depends on this kubeflow_db resource is I think the cleanest way.

@poom poom removed request for Judrummer and bludot September 6, 2023 03:35
@poom poom removed the request for review from JoseFMP September 6, 2023 03:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants