Update `.golangci.yml` config to resolve deprecation warnings by paulinek13 · Pull Request #4082 · agones-dev/agones

paulinek13 · 2025-01-06T00:08:48Z

What type of PR is this?

/kind cleanup

What this PR does / Why we need it:

This PR updates the .golangci.yml configuration file to resolve deprecation warnings.
The changes include:

Replacing the deprecated run.skip-dirs with issues.exclude-dirs.
Updating output.format to output.formats.
Removing the deprecated megacheck and exportloopref linters.
Updating code comments to reflect the changes.

These updates are necessary to keep the linting process up-to-date and free from deprecation warnings.

Which issue(s) this PR fixes:

Closes #4081

Special notes for your reviewer:

Ran make lint to verify that the warnings have been resolved.

google-cla · 2025-01-06T00:08:52Z

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

markmandel · 2025-01-06T03:14:41Z

/gcbrun

agones-bot · 2025-01-06T04:13:08Z

Build Failed 😭

Build Id: 9782d29a-dc60-41bf-99b1-56df5acad91a

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

markmandel · 2025-01-06T17:55:52Z

Looks like a flake in e2e:

VERBOSE:     framework.go:346: 
VERBOSE:         	Error Trace:	/go/src/agones.dev/agones/test/e2e/framework/framework.go:346
VERBOSE:         	            				/go/src/agones.dev/agones/test/e2e/fleetautoscaler_test.go:1488
VERBOSE:         	Error:      	Received unexpected error:
VERBOSE:         	            	context deadline exceeded
VERBOSE:         	Test:       	TestListAutoscalerAllocated/Scale_Up_Buffer_Percent
VERBOSE:         	Messages:   	error waiting for fleet condition on fleet: simple-fleet-1.09s85n
VERBOSE: --- FAIL: TestListAutoscalerAllocated/Scale_Up_Buffer_Percent (311.66s)
VERBOSE: --- FAIL: TestListAutoscalerAllocated (311.66s)

markmandel · 2025-01-06T18:00:22Z

.golangci.yml

  enable:
    - bodyclose
    - dupl
-    - exportloopref


The warning for this was:

WARN The linter 'exportloopref' is deprecated (since v1.60.2) due to: Since Go1.22 (loopvar) this linter is no longer relevant. Replaced by copyloopvar.

Should we replace it, rather than just delete it? (Same for all the other ones as well?)

Thank you for pointing that out! I’ve updated the changes to replace the deprecated linters rather than just removing them.

oooh! My bad! I read it as addition, not subtraction.

Sorry! Doing too many things at once 🤦🏻

markmandel · 2025-01-06T18:03:10Z

/gcbrun

agones-bot · 2025-01-06T19:29:24Z

Build Succeeded 🥳

Build Id: 6022e24d-fce6-4418-ab67-79d85c49f678

The following development artifacts have been built, and will exist for the next 30 days:

image: us-docker.pkg.dev/agones-images/ci/agones-controller:1.47.0-dev-ec717e3
image: us-docker.pkg.dev/agones-images/ci/agones-extensions:1.47.0-dev-ec717e3
image: us-docker.pkg.dev/agones-images/ci/agones-sdk:1.47.0-dev-ec717e3-linux
image: us-docker.pkg.dev/agones-images/ci/agones-ping:1.47.0-dev-ec717e3
image: us-docker.pkg.dev/agones-images/ci/agones-allocator:1.47.0-dev-ec717e3
Linux C++ SDK (build): agonessdk-1.47.0-dev-ec717e3-linux-arch_64.tar.gz
SDK Server: agonessdk-server-1.47.0-dev-ec717e3.zip

A preview of the website (the last 30 builds are retained):

https://ec717e3-dot-preview-dot-agones-images.appspot.com/

To install this version:

git fetch https://github.com/googleforgames/agones.git pull/4082/head:pr_4082 && git checkout pr_4082
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.47.0-dev-ec717e3

…on warnings - Replace deprecated `run.skip-dirs` with `issues.exclude-dirs` - Update `output.format` to `output.formats` - Replace deprecated linters: - `megacheck` (split into `gosimple`, `staticcheck`, and `unused`) - `exportloopref` (no longer relevant since Go 1.22, replaced by `copyloopvar`) - Remove redundant variable copies flagged by `copyloopvar` (context: https://go.dev/blog/loopvar-preview) - Adjust `defer` statement comments in `localsdk.go` to use `staticcheck` instead of the deprecated `megacheck` Signed-off-by: Paulina Kalicka <71526180+paulinek13@users.noreply.github.com>

markmandel · 2025-01-06T19:59:57Z

/gcbrun

markmandel · 2025-01-06T20:01:30Z

pkg/allocation/converters/converter.go

 func convertInternalLabelSelectorsToLabelSelectors(in []allocationv1.GameServerSelector) []*pb.GameServerSelector {
 	var result []*pb.GameServerSelector
 	for _, l := range in {
-		l := l


Can you explain this pattern? It seems.... weird? 🤔

After enabling the copyloopvar linter, I noticed it flagged instances of redundant code, specifically unnecessary copies of loop variables. For example:

pkg/allocation/converters/converter.go:289:3: The copy of the 'for' variable "l" can be deleted (Go 1.22+) (copyloopvar) l := l

I’ve cleaned up these instances to resolve the linter errors.
Context: https://go.dev/blog/loopvar-preview

Sorry, I was sure I've already added this comment
I hope it answers your question :)

agones-bot · 2025-01-06T21:00:56Z

Build Succeeded 🥳

Build Id: dd857123-5cca-4d29-9e58-730ff8dd7a2a

The following development artifacts have been built, and will exist for the next 30 days:

image: us-docker.pkg.dev/agones-images/ci/agones-controller:1.47.0-dev-6e41528
image: us-docker.pkg.dev/agones-images/ci/agones-extensions:1.47.0-dev-6e41528
image: us-docker.pkg.dev/agones-images/ci/agones-sdk:1.47.0-dev-6e41528-linux
image: us-docker.pkg.dev/agones-images/ci/agones-ping:1.47.0-dev-6e41528
image: us-docker.pkg.dev/agones-images/ci/agones-allocator:1.47.0-dev-6e41528
Linux C++ SDK (build): agonessdk-1.47.0-dev-6e41528-linux-arch_64.tar.gz
SDK Server: agonessdk-server-1.47.0-dev-6e41528.zip

A preview of the website (the last 30 builds are retained):

https://6e41528-dot-preview-dot-agones-images.appspot.com/

To install this version:

git fetch https://github.com/googleforgames/agones.git pull/4082/head:pr_4082 && git checkout pr_4082
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.47.0-dev-6e41528

markmandel

Nice!

markmandel · 2025-01-06T21:38:39Z

.golangci.yml

  enable:
    - bodyclose
    - dupl
-    - exportloopref


oooh! My bad! I read it as addition, not subtraction.

Sorry! Doing too many things at once 🤦🏻

github-actions bot added kind/other size/S labels Jan 6, 2025

markmandel reviewed Jan 6, 2025

View reviewed changes

paulinek13 force-pushed the fix/lint-deprecation-warnings branch from ec717e3 to 6e41528 Compare January 6, 2025 19:35

markmandel reviewed Jan 6, 2025

View reviewed changes

markmandel approved these changes Jan 6, 2025

View reviewed changes

markmandel merged commit 0366002 into agones-dev:main Jan 6, 2025

markmandel added kind/cleanup Refactoring code, fixing up documentation, etc area/build-tools Development tooling. I.e. pretty much everything in the `build` directory. labels Jan 6, 2025

Conversation

paulinek13 commented Jan 6, 2025

Uh oh!

google-cla bot commented Jan 6, 2025

Uh oh!

markmandel commented Jan 6, 2025

Uh oh!

agones-bot commented Jan 6, 2025

Uh oh!

markmandel commented Jan 6, 2025

Uh oh!

markmandel Jan 6, 2025

Choose a reason for hiding this comment

Uh oh!

paulinek13 Jan 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

markmandel Jan 6, 2025

Choose a reason for hiding this comment

Uh oh!

markmandel commented Jan 6, 2025

Uh oh!

agones-bot commented Jan 6, 2025

Uh oh!

markmandel commented Jan 6, 2025

Uh oh!

markmandel Jan 6, 2025

Choose a reason for hiding this comment

Uh oh!

paulinek13 Jan 6, 2025

Choose a reason for hiding this comment

Uh oh!

paulinek13 Jan 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

agones-bot commented Jan 6, 2025

Uh oh!

markmandel left a comment

Choose a reason for hiding this comment

Uh oh!

markmandel Jan 6, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

paulinek13 Jan 6, 2025 •

edited

Loading

paulinek13 Jan 6, 2025 •

edited

Loading