Repository navigation
Set go version in one place - #8774
Conversation
Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8774 +/- ##
==========================================
- Coverage 59.89% 55.72% -4.17%
==========================================
Files 595 595
Lines 40481 40481
==========================================
- Hits 24248 22560 -1688
- Misses 14406 16209 +1803
+ Partials 1827 1712 -115 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
ytimocin
left a comment
There was a problem hiding this comment.
How do we know if these will break the workflows or not?
I ran these workflows on the fork when I was testing. These are the ones I know will run on a fork.
|
# Description While working on [updating the go version](#8774) there was a need to install the correct version of `golanglint-ci`. The [documentation](https://golangci-lint.run/welcome/install/#install-from-sources) recommends using a binary installation, so I added that to the devcontainer specification (`devcontainer.json`). As part of that change I moved the `postCreateCommand` into a script. I also fixed broken links to `golanglint-ci` in the Radius contributing docs. Development environment setup: * [`.devcontainer/devcontainer.json`](diffhunk://#diff-24ad71c8613ddcf6fd23818cb3bb477a1fb6d83af4550b0bad43099813088686L45-R45): Changed the `postCreateCommand` to use a new script `.devcontainer/post-create.sh` instead of a long inline command. * [`.devcontainer/post-create.sh`](diffhunk://#diff-bcf66a848607b49d2a45edfad434f6808017ee9d18f5a20dde6670745010d89aR1-R20): Created a new script to handle post-create tasks, including setting up the workspace as a safe directory, installing `golangci-lint`, and other Go tools. Documentation updates: fix broken links * [`docs/contributing/contributing-code/contributing-code-first-commit/first-commit-00-prerequisites/index.md`](diffhunk://#diff-1b65ceaf649b58267d9199873c6df394b0bc30508b0bfb540cc247e0028af6a1L31-R31): Updated the link for `golangci-lint` installation to the welcome page. * [`docs/contributing/contributing-code/contributing-code-first-commit/first-commit-06-creating-a-pr/index.md`](diffhunk://#diff-39e98fec72f851ba1d675c0fc5fd47452569b04c7e85331c257b790c31f967feL17-R17): Updated the link for `golangci-lint` in the pre-PR checklist. * [`docs/contributing/contributing-code/contributing-code-prerequisites/README.md`](diffhunk://#diff-b3cdecf6f61c777e0a2e7089a64f8f49aeef24995ae79aa62a215f8911929cb8L102-R102): Updated the link for `golangci-lint` in the list of core dependencies. ## Type of change - This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Radius (issue link optional). Fixes: N/A ## Contributor checklist Please verify that the PR meets the following requirements, where applicable: - An overview of proposed schema changes is included in a linked GitHub issue. - [ ] Yes - [x] Not applicable - A design document PR is created in the [design-notes repository](https://github.com/radius-project/design-notes/), if new APIs are being introduced. - [ ] Yes - [x] Not applicable - The design document has been reviewed and approved by Radius maintainers/approvers. - [ ] Yes - [x] Not applicable - A PR for the [samples repository](https://github.com/radius-project/samples) is created, if existing samples are affected by the changes in this PR. - [ ] Yes - [x] Not applicable - A PR for the [documentation repository](https://github.com/radius-project/docs) is created, if the changes in this PR affect the documentation or any user facing updates are made. - [ ] Yes - [x] Not applicable - A PR for the [recipes repository](https://github.com/radius-project/recipes) is created, if existing recipes are affected by the changes in this PR. - [ ] Yes - [x] Not applicable Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
Description
This PR addresses issue #8374 by consolidating all references to the Go version into the
go.modfile, so that the Go version can be set in one place. Previously, the Go version was also set in the GitHub workflow files, which resulted in different versions being used across different workflows.This PR also updates the Go version from 1.23.0 to 1.24.1, which was released on Feb 11, with the minor version update on March 4. This update is included in this PR because Go version 1.23.0 had reported issues with caching errors in GitHub workflows. A side-effect of updating the Go version is that go module caching is now working, which reduces workflow times. For example, the non-cloud tests took 15 minutes on the forked repo, down from 20 minutes before the update.
As part of updating the Go version to 1.24.1, the
golangci-linttool is updated inlint.yamlto version 1.64.6.A similar PR is created on the
bicep-types-awsrepo.Workflow updates:
.github/workflows/build.yaml: Removed theGOVERenvironment variable and updated the Go setup steps to usego-version-file: go.modandcache-dependency-path: go.suminstead ofgo-version: ${{ env.GOVER }}..github/workflows/functional-test-cloud.yaml: Removed theGOVERenvironment variable and updated the Go setup steps to usego-version-file: go.modandcache-dependency-path: go.suminstead ofgo-version: ${{ env.GOVER }}..github/workflows/functional-test-noncloud.yaml: Removed theGOVERenvironment variable and updated the Go setup steps to usego-version-file: go.modandcache-dependency-path: go.suminstead ofgo-version: ${{ env.GOVER }}..github/workflows/lint.yaml: Removed theGOVERenvironment variable and updated the Go setup steps to usego-version-file: go.modandcache-dependency-path: go.suminstead ofgo-version: ${{ env.GOVER }}..github/workflows/long-running-azure.yaml: Removed theGOVERenvironment variable and updated the Go setup steps to usego-version-file: go.modandcache-dependency-path: go.suminstead ofgo-version: ${{ env.GOVER }}..github/workflows/publish-docs.yaml: Removed theGOVERenvironment variable and updated the Go setup steps to usego-version-file: radius/go.modandcache-dependency-path: radius/go.suminstead ofgo-version: ${{ env.GOVER }}.pkg/cli/clients/management.go: this change was prompted by the updated version ofgolangci-lint.Go version update:
go.mod: Updated the Go version from1.23.0to1.24.1.Testing
Not all workflows can be run from a fork, but these tests ran successfully:
Type of change
Fixes: #8374
Contributor checklist
Please verify that the PR meets the following requirements, where applicable: