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

Allow underscores in aws_db_subnet_group name #2604

Merged
merged 1 commit into from
Jul 2, 2015
Merged

Allow underscores in aws_db_subnet_group name #2604

merged 1 commit into from
Jul 2, 2015

Conversation

CpuID
Copy link
Contributor

@CpuID CpuID commented Jul 2, 2015

Docs don't claim they are allowed but they are.

They don't explicitly say they are not allowed... but yea.

http://docs.aws.amazon.com/AmazonRDS/latest/CommandLineReference/CLIReference-cmd-ModifyDBSubnetGroup.html

Constraints: Must contain from 1 to 255 alphanumeric characters or hyphens. First character must be a letter. Cannot end with a hyphen or contain two consecutive hyphens.

For #2603

@radeksimko
Copy link
Member

Interesting to see the mismatch between AWS docs and reality... :-) I will notice them.

It would be nice if we can test this somehow, i.e. verify that AWS actually permits that and more importantly keep verifying per each release that this behaviour doesn't change (as it's undocumented).
I know RDS instance may take ~30mins+ to create/update/delete, that would be too expensive for a simple name validation test, but it's worth checking how long does it take to create & destroy subnet. If it's quick, I'd rather have that test in place.

Would you mind having a look and eventually adding that test (or confirming it takes ages to create/destroy)? If you don't have time, that's ok, I can do it.

@CpuID
Copy link
Contributor Author

CpuID commented Jul 2, 2015

I'll see if I have time to get through this today, see how I go. Will ping you if I get stuck.

@radeksimko
Copy link
Member

I will add the test after merging.

radeksimko added a commit that referenced this pull request Jul 2, 2015
Allow underscores in aws_db_subnet_group name
@radeksimko radeksimko merged commit 127fa30 into hashicorp:master Jul 2, 2015
@ghost
Copy link

ghost commented May 1, 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 have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators May 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants