search

lxc / terraform-provider-incus

Incus provider for Terraform/OpenTofu
https://linuxcontainers.org/incus
Mozilla Public License 2.0
35 stars 8 forks source link

github: Also test on Go 1.20 #4

Closed stgraber closed 7 months ago

adamcstephens commented 7 months ago

Almost all users will be using the pre compiled binaries from the registry. What value do you see in testing the older Go version here?

I normally agree a project should support both releases of Go, but these tests take a number of minutes each.

stgraber commented 7 months ago

I see two benefits from it, one is it gives us a better view of potential Go compiler bugs by testing on more than one release. That's something that's proved itself useful for Incus in the past. The other is to support an older Go release for the few Linux distributions that would like to ship the provider as a package. Distros are quite often one major release behind so making sure we don't rely on the absolute latest features should help.

I don't really see this as adding testing time given it's done in a test matrix and so is run in parallel by Github. It certainly consumes more test runners but it's not like we pay for those ;)

stgraber commented 7 months ago

I found some hits around Gentoo derivatives at least. It's definitely not a very common thing but no real harm in supporting it either.