Skip to content
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

r/db_parameter_group: Allow underscores in names #1460

Merged
merged 1 commit into from
Aug 21, 2017
Merged

r/db_parameter_group: Allow underscores in names #1460

merged 1 commit into from
Aug 21, 2017

Conversation

kirillshevch
Copy link
Contributor

No description provided.

@kirillshevch kirillshevch changed the title fix underscores usign in rds db name fix underscores using in rds db name Aug 21, 2017
@Ninir Ninir added the enhancement Requests to existing resources that expand the functionality or scope. label Aug 21, 2017
@Ninir Ninir changed the title fix underscores using in rds db name r/db_parameter_group: Allow underscores in names Aug 21, 2017
Copy link
Contributor

@Ninir Ninir left a comment

Choose a reason for hiding this comment

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

Hey @kirillshevch

Thanks for this. Just checked in the console, looks good to me :)
For the record, how did you get into this issue and fix?

@Ninir Ninir added the waiting-response Maintainers are waiting on response from community or contributor. label Aug 21, 2017
@kirillshevch
Copy link
Contributor Author

@Ninir reproduced with Terraform v0.10.2

  1. My current state of AWS was exported by terraforming
  2. I got rds.tf
resource "aws_db_instance" "cryptostatus" {
    identifier                = "cryptostatus"
    allocated_storage         = 20
    storage_type              = "gp2"
    engine                    = "postgres"
    engine_version            = "9.6.1"
    instance_class            = "db.t2.large"
    name                      = "cryptostatus_production"
    username                  = "deployer"
    password                  = "xxxxxxxx"
    port                      = 5432
    publicly_accessible       = false
    availability_zone         = "us-west-2a"
    security_group_names      = []
    vpc_security_group_ids    = ["sg-98f2e1e1"]
    db_subnet_group_name      = "default"
    parameter_group_name      = "default.postgres9.6"
    multi_az                  = false
    backup_retention_period   = 7
    backup_window             = "09:40-10:10"
    maintenance_window        = "sat:07:59-sat:08:29"
    final_snapshot_identifier = "cryptostatus-final"
}
resource "aws_db_instance" "translations" {
    identifier                = "translations"
    allocated_storage         = 5
    storage_type              = "gp2"
    engine                    = "postgres"
    engine_version            = "9.6.1"
    instance_class            = "db.t2.micro"
    name                      = "cryptostatus_translations_production"
    username                  = "deployer"
    password                  = "xxxxxxxx"
    port                      = 5432
    publicly_accessible       = true
    availability_zone         = "us-west-2a"
    security_group_names      = []
    vpc_security_group_ids    = ["sg-98f2e1e1"]
    db_subnet_group_name      = "default"
    parameter_group_name      = "default.postgres9.6"
    multi_az                  = true
    backup_retention_period   = 7
    backup_window             = "06:10-06:40"
    maintenance_window        = "sat:12:21-sat:12:51"
    final_snapshot_identifier = "translations-final"
}

  1. terraform plan
2 error(s) occurred:

* aws_db_parameter_group.default-postgres9-5: only lowercase alphanumeric characters and hyphens allowed in "name"
* aws_db_parameter_group.default-postgres9-6: only lowercase alphanumeric characters and hyphens allowed in "name"

@Ninir
Copy link
Contributor

Ninir commented Aug 21, 2017

Perfect thank you :)

Thanks for the work here!

@Ninir Ninir merged commit ab09191 into hashicorp:master Aug 21, 2017
@kirillshevch kirillshevch deleted the bugfix/allow-underscore-rds branch August 21, 2017 21:16
nbaztec pushed a commit to nbaztec/terraform-provider-aws that referenced this pull request Sep 26, 2017
…rscore-rds

r/db_parameter_group: Allow underscores in names
@ghost
Copy link

ghost commented Apr 11, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!

@ghost ghost locked and limited conversation to collaborators Apr 11, 2020
@breathingdust breathingdust removed the waiting-response Maintainers are waiting on response from community or contributor. label Sep 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Requests to existing resources that expand the functionality or scope.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants