search

iterative / terraform-provider-iterative

☁️ Terraform plugin for machine learning workloads: spot instance recovery & auto-termination | AWS, GCP, Azure, Kubernetes
https://registry.terraform.io/providers/iterative/iterative/latest/docs
Apache License 2.0
290 stars 27 forks source link

Turning `task` into an importable module #724

Open 0x2b3bfa0 opened 1 year ago

0x2b3bfa0 commented 1 year ago

Terraform providers modules are named in such a way that prevents importing them. Now that we're going to use the task package from several repositories, we'll need to perform some modifications so it can be imported.

Option A

Rename terraform-provider-iterative to github.com/iterative/terraform-provider-iterative here and on every imaginable import:

https://github.com/iterative/terraform-provider-iterative/blob/85645f465d82dc849dc51fcbd54a395ac185b6b6/go.mod#L1

Option B

Create a new github.com/iterative/leo repository/module and move the task package (also cmd) and children there.

0x2b3bfa0 commented 1 year ago

(I prefer option B)

tasdomas commented 1 year ago

To be clear - the way terraform provider modules are named has no technical reason behind it. It's just a means of not having to maintain a stable API. Nothing will break if we change the module name of terraform-provider-iterative.