Upgrade to Terraform 0.13

[invidian]

NOTE: Includes commits from #860.

NOTE: Includes changes from #1027

Closes #788

Signed-off-by: Mateusz Gozdek mateusz@kinvolk.io

[invidian]

From testing on Packet, it seems following string must be added to the Terraform code to get things working:

$ git diff
diff --git a/pkg/platform/packet/template.go b/pkg/platform/packet/template.go
index 6c7ea543..b449f8dd 100644
--- a/pkg/platform/packet/template.go
+++ b/pkg/platform/packet/template.go
@@ -287,4 +287,15 @@ output "lokomotive_values" {
   value     = module.packet-{{.Config.ClusterName}}.lokomotive_values
   sensitive = true
 }
+
+terraform {
+  required_version = ">= 0.13.0"
+
+  required_providers {
+    packet = {
+      source  = "terraform-providers/packet"
+      version = "~> 2.7.3"
+    }
+  }
+}
 `

Note, that we render the terraform {} block together with backend settings, which probably means we should have some data structure containing list of providers which are available from Platform interface, so they can be rendered centrally. Or Platform interface should be able to emit the snippet to terraform {} block. CC @johananl

[invidian]

Rebased to include #1021, so CI can pass.

[knrt10]

Rest looks great

[invidian]

Ah, forgot to mention. This PR includes #1027, so #1027 should get merged first.

[knrt10]

looks great

[surajssd]

The code looks ok to me, @invidian could you rebase and trigger the CI once more?

[invidian]

@surajssd this is already based on latest master. I try to restart AKS job now..

[invidian]

Well, re-pushed anyway, as CI seems to got stuck somehow

[knrt10]

Restarted failed aks and packetARM pipelines