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.

Sign up for GitHub

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

Cloud SQL activation policy selectable #1613

Merged
merged 9 commits into from
Aug 25, 2023
Merged

Cloud SQL activation policy selectable #1613

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
7abbbaa
Update main.tf
cmvalla Aug 23, 2023
82aecae
Update variables.tf
cmvalla Aug 23, 2023
0cfd47f
corrections requested by @juliocc
cmvalla Aug 25, 2023
db625b7
Merge branch 'master' into master
cmvalla Aug 25, 2023
fb58a74
Regeneration of README.md on cloudsql-instance mod
cmvalla Aug 25, 2023
171245c
terraform fmt changes
cmvalla Aug 25, 2023
161c8e2
bug correctionand activation_policy on replicas
cmvalla Aug 25, 2023
a54dab5
activation_policy description change and README.md
cmvalla Aug 25, 2023
961a8de
Bug correction
cmvalla Aug 25, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions modules/cloudsql-instance/main.tf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -119,6 +119,7 @@ resource "google_sql_database_instance" "primary" {
}
}
deletion_protection = var.deletion_protection
activation_policy = var.activation_policy
}

resource "google_sql_database_instance" "replicas" {
Expand Down
10 changes: 10 additions & 0 deletions modules/cloudsql-instance/variables.tf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,15 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
variable "activation_policy" {
description = "(Optional) This specifies when the instance should be active. Can be either ALWAYS, NEVER or ON_DEMAND. Default is ALWAYS"
cmvalla marked this conversation as resolved.
Show resolved Hide resolved
cmvalla marked this conversation as resolved.
Show resolved Hide resolved
default = "ALWAYS"
validation {
condition = var.activation_policy == "NEVER" || var.activation_policy == "ON_DEMAND" || var.activation_policy == "ALWAYS"
error_message = "activation_policy must be ALWAYS, NEVER or ON_DEMAND"
juliocc marked this conversation as resolved.
Show resolved Hide resolved
}
nullable = false
}

variable "allocated_ip_ranges" {
description = "(Optional)The name of the allocated ip range for the private ip CloudSQL instance. For example: \"google-managed-services-default\". If set, the instance ip will be created in the allocated range. The range name must comply with RFC 1035. Specifically, the name must be 1-63 characters long and match the regular expression a-z?."
Expand Down Expand Up @@ -189,3 +198,4 @@ variable "users" {
type = map(string)
default = null
}