-
-
Notifications
You must be signed in to change notification settings - Fork 181
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add option to define character set #71
Add option to define character set #71
Conversation
/rebuild-readme |
/test all |
This pull request is now in conflict. Could you fix it @yariksheptykin? 🙏 |
/rebuild-readme |
@yariksheptykin please, rebase the code |
13c15c0
to
f6b316d
Compare
/rebuild-readme |
Hi @maximmi , could you take a look at this MR before it becomes stale again? |
This pull request is now in conflict. Could you fix it @yariksheptykin? 🙏 |
Closing merge request because there does not seem to be much interest in this work. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bridgecrew has found 2 infrastructure configuration errors in this PR ⬇️
@@ -29,6 +29,7 @@ resource "aws_db_instance" "default" { | |||
port = var.database_port | |||
engine = var.engine | |||
engine_version = var.engine_version | |||
character_set_name = var.charset_name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ensure that enhanced monitoring is enabled for Amazon RDS instances
Resource: aws_db_instance.default | ID: BC_AWS_LOGGING_28
How to Fix
resource "aws_db_instance" "default" {
allocated_storage = 10
engine = "mysql"
engine_version = "5.7"
instance_class = "db.t3.micro"
name = "mydb"
username = "foo"
password = "foobarbaz"
parameter_group_name = "default.mysql5.7"
skip_final_snapshot = true
+ monitoring_interval = 5
}
Description
TBADependent Resources
Calculating...
@@ -29,6 +29,7 @@ resource "aws_db_instance" "default" { | |||
port = var.database_port | |||
engine = var.engine | |||
engine_version = var.engine_version | |||
character_set_name = var.charset_name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ensure that RDS instances have Multi-AZ enabled
Resource: aws_db_instance.default | ID: BC_AWS_GENERAL_73
How to Fix
resource "aws_db_instance" "default" {
allocated_storage = 20
storage_type = "gp2"
engine = "mysql"
engine_version = "5.7"
instance_class = "db.t2.micro"
name = "mydb"
+ multi_az = true
}
Description
Amazon RDS Multi-AZ deployments provide enhanced availability for databases within a single region. In the event of a planned or unplanned outage of your DB instance, Amazon RDS automatically switches to a standby replica in another Availability Zone if you have enabled Multi-AZ.RDS Multi-AZ deployments offer the following benefits:
- Enhanced durability.
- Increased availability.
- Protection of your database performance.
- Automatic failover.
Dependent Resources
Calculating...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bridgecrew has found 2 infrastructure configuration errors in this PR ⬇️
@@ -29,6 +29,7 @@ resource "aws_db_instance" "default" { | |||
port = var.database_port | |||
engine = var.engine | |||
engine_version = var.engine_version | |||
character_set_name = var.charset_name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ensure that enhanced monitoring is enabled for Amazon RDS instances
Resource: aws_db_instance.default | ID: BC_AWS_LOGGING_28
How to Fix
resource "aws_db_instance" "default" {
allocated_storage = 10
engine = "mysql"
engine_version = "5.7"
instance_class = "db.t3.micro"
name = "mydb"
username = "foo"
password = "foobarbaz"
parameter_group_name = "default.mysql5.7"
skip_final_snapshot = true
+ monitoring_interval = 5
}
Description
TBADependent Resources
Calculating...
@@ -29,6 +29,7 @@ resource "aws_db_instance" "default" { | |||
port = var.database_port | |||
engine = var.engine | |||
engine_version = var.engine_version | |||
character_set_name = var.charset_name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ensure that RDS instances have Multi-AZ enabled
Resource: aws_db_instance.default | ID: BC_AWS_GENERAL_73
How to Fix
resource "aws_db_instance" "default" {
allocated_storage = 20
storage_type = "gp2"
engine = "mysql"
engine_version = "5.7"
instance_class = "db.t2.micro"
name = "mydb"
+ multi_az = true
}
Description
Amazon RDS Multi-AZ deployments provide enhanced availability for databases within a single region. In the event of a planned or unplanned outage of your DB instance, Amazon RDS automatically switches to a standby replica in another Availability Zone if you have enabled Multi-AZ.RDS Multi-AZ deployments offer the following benefits:
- Enhanced durability.
- Increased availability.
- Protection of your database performance.
- Automatic failover.
Dependent Resources
Calculating...
/test all |
Thanks for the contribution @yariksheptykin -- we appreciate it! Released as 0.37.0. |
Closes #70