fix: Correct mis-spelling in serverless security policy default description #16
Conversation
dim-ops
changed the title
| @@ -64,7 +64,7 @@ resource "aws_opensearchserverless_security_policy" "encryption" { | |||
| resource "aws_opensearchserverless_security_policy" "network" { | |||
| count = var.create && var.create_network_policy ? 1 : 0 | |||
|
|
|||
| description = coalesce(var.network_policy_description, "Newtwork policy for ${var.name} collection") | |||
There was a problem hiding this comment.
I don't know - Newtwork sounds kinda cool 😅
bryantbiggs
changed the title
bryantbiggs
merged commit 844a435
into
terraform-aws-modules:master
## [1.2.2](v1.2.1...v1.2.2) (2024-06-28) ### Bug Fixes * Correct mis-spelling in serverless security policy default description ([#16](#16)) ([844a435](844a435))
|
This PR is included in version 1.2.2 🎉 |
|
@antonbabenko I hesitate to take the time to add the creation of the VPC endpoint to the module, what do you think? https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/opensearch_vpc_endpoint |
|
Scratch that - that implementation would go here but we already have it in the module |
|
I'm going to lock this pull request 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 related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Description
Spelling error. Newtwork => network