chore: upgrade go version to 1.24.4 and golangci-lint to 2.1.6

[slashpai]

No description provided.

[slashpai]

cc @RainbowMango

[RainbowMango]

This is most likely duplicated with #203.

[slashpai]

This is most likely duplicated with #203.

ok I didn't see that, I was updating prometheus-adapter so found need to update dependencies here as well.
In this PR I have updated go version as well bumped k8s dependencies to v0.33.1

[RainbowMango]

Maybe you can help to review that PR, and after that the dependent bot can help to bump to 1.33.1, and then this PR can focus on prometheus-adapter thing.

[slashpai]

I will rebase to just update go

[slashpai]

@RainbowMango PTAL

[RainbowMango]

/assign

[slashpai]

Lint file has been updated by running the command from https://golangci-lint.run/product/migration-guide

[RainbowMango]

I know that migration tool, it changes things sometimes as I mentioned above, removed the timeout setting and removed the typecheck linter.

[slashpai]

Sure let me review again

[richabanker]

/triage accepted

[RainbowMango]

@slashpai any update?

I'm going to tag a new release for the Kubernetes v1.33, it would be great to have this.

[slashpai]

I will have it updated this week, was a bit occupied so couldn't look into this

[RainbowMango]

Hi @slashpai Kindly ping? We are waiting for this PR to cut the v1.33.0 release.
What I'm expected from this like:

diff --git a/.golangci.yml b/.golangci.yml
index 7f712891..5473cc79 100644
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -1,6 +1,7 @@
 version: "2"
+run:
+  timeout: 7m
 linters:
-  default: none
   enable:
     - bodyclose
     - copyloopvar
@@ -25,6 +26,11 @@ linters:
     misspell:
       ignore-rules:
         - creater
+    staticcheck:
+      checks:
+        - all
+        # Disable QF1008 to retain embedded fields for better readability.
+        - "-QF1008"
   exclusions:
     generated: lax
     presets:
diff --git a/pkg/apiserver/endpoints/handlers/get.go b/pkg/apiserver/endpoints/handlers/get.go
index fa0920c1..1ce71623 100644
--- a/pkg/apiserver/endpoints/handlers/get.go
+++ b/pkg/apiserver/endpoints/handlers/get.go
@@ -181,5 +181,5 @@ func getRequestOptions(req *http.Request, scope handlers.RequestScope, into runt
                newQuery[subpathKey] = []string{p}
                query = newQuery
        }
-       return scope.DecodeParameters(query, scope.Kind.GroupVersion(), into)
+       return scope.ParameterCodec.DecodeParameters(query, scope.Kind.GroupVersion(), into)
 }
(END)
     generated: lax
     presets:
diff --git a/pkg/apiserver/endpoints/handlers/get.go b/pkg/apiserver/endpoints/handlers/get.go
index fa0920c1..1ce71623 100644
--- a/pkg/apiserver/endpoints/handlers/get.go
+++ b/pkg/apiserver/endpoints/handlers/get.go
@@ -181,5 +181,5 @@ func getRequestOptions(req *http.Request, scope handlers.RequestScope, into runt
                newQuery[subpathKey] = []string{p}
                query = newQuery
        }
-       return scope.DecodeParameters(query, scope.Kind.GroupVersion(), into)
+       return scope.ParameterCodec.DecodeParameters(query, scope.Kind.GroupVersion(), into)
 }

[slashpai]

@RainbowMango Let me know if these changes ok, these were suggested by linter

[RainbowMango]

We don't need this change as we disabled the check

       # Disable QF1008 to retain embedded fields for better readability.
       - "-QF1008"

[slashpai]

sorry for taking long was on unplanned PTO, I have addressed review comments. Please take a look again

[slashpai]

@RainbowMango PTAL updated with changes

[RainbowMango]

/lgtm
/approve
Thanks

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: RainbowMango, slashpai

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [RainbowMango]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment