ci: use setup-go for Windows and hardcode GO_VERSION

pendo324 commented 2 weeks ago

Issue #, if available:

Description of changes

Adds actions/setup-go to Windows workflows
Does not use check-latest: true because our dependencies require 1.22.0, and go considers 1.22 (without the patch version) to not satisfy that criteria. We will have to periodically manually update GO_VERSION to align with new Go releases.

Testing done

None, need to test this on the runners themselves.

Contribution agreement

[x] I've reviewed the guidance in CONTRIBUTING.md

License Acceptance

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

austinvazquez commented 2 weeks ago

Curious if the enable cache change is intentional? For macOS e2e testing workflows cache writes takes 6-7 minutes due to cache size. These are also evicted quite quickly as we only have the default 10GB GitHub cache.

pendo324 commented 1 week ago

Curious if the enable cache change is intentional? For macOS e2e testing workflows cache writes takes 6-7 minutes due to cache size. These are also evicted quite quickly as we only have the default 10GB GitHub cache.

I thought that issue was resolved upstream, but I see they only have a workaround for Windows, not macOS. I'll make a new revision with cahceing disabled.

runfinch / finch