Merge pull request #62 from astronomer/fix_tfdeprications

fix resource deprecations
astronomer · Jul 28, 2022 · 1b0a00e · 1b0a00e
2 parents a6088d3 + 89e87f9
commit 1b0a00e
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 8 deletions.
diff --git a/bastion.tf b/bastion.tf
@@ -47,15 +47,15 @@ resource "aws_key_pair" "bastion_ssh_key" {
   public_key = tls_private_key.ssh_key[0].public_key_openssh
 }
 
-resource "local_file" "bastion_ssh_key_private" {
-  count             = var.enable_bastion ? 1 : 0
-  filename          = pathexpand(format("~/.ssh/%s_bastion_ssh_key", var.deployment_id))
-  sensitive_content = tls_private_key.ssh_key[0].private_key_pem
+resource "local_sensitive_file" "bastion_ssh_key_private" {
+  count    = var.enable_bastion ? 1 : 0
+  filename = pathexpand(format("~/.ssh/%s_bastion_ssh_key", var.deployment_id))
+  content  = tls_private_key.ssh_key[0].private_key_pem
 
   # make correct permissions on file
   # make correct permissions on file
   provisioner "local-exec" {
-    command = "chmod 400 ${local_file.bastion_ssh_key_private[0].filename}"
+    command = "chmod 400 ${local_sensitive_file.bastion_ssh_key_private[0].filename}"
   }
 }
 

diff --git a/certificate.tf b/certificate.tf
@@ -20,7 +20,6 @@ resource "tls_private_key" "cert_private_key" {
 }
 
 resource "tls_cert_request" "req" {
-  key_algorithm   = "RSA"
   private_key_pem = tls_private_key.cert_private_key.private_key_pem
   dns_names       = ["*.${var.deployment_id}.${var.route53_domain}"]
 

diff --git a/examples/from_scratch/main.tf b/examples/from_scratch/main.tf
@@ -20,7 +20,7 @@ module "astronomer_aws_with_vpc" {
   }
 }
 
-resource "local_file" "kubeconfig" {
+resource "local_sensitive_file" "kubeconfig" {
   depends_on = [module.astronomer_aws_with_vpc]
   content    = module.astronomer_aws_with_vpc.kubeconfig
   filename   = "${path.root}/kubeconfig-${var.deployment_id}"
@@ -29,7 +29,7 @@ resource "local_file" "kubeconfig" {
 terraform {
   required_providers {
     acme = {
-      source = "terraform-providers/acme"
+      source = "vancluever/acme"
     }
   }
   required_version = ">= 0.13"