[cluster-autoscaler-1.31] GHA config

.github/workflows/ca-test.yaml

@@ -0,0 +1,41 @@
+name: Cluster Autoscaler
+
+on:
+  push:
+    paths:
+      - 'cluster-autoscaler/**'
+  pull_request:
+    paths:
+      - 'cluster-autoscaler/**'
+
+env:
+  GOPATH: ${{ github.workspace }}/go
+
+permissions:
+  contents: read
+  checks: write
+
+jobs:
+  test:
+    name: test
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/[email protected]
+        with:
+          path: ${{ env.GOPATH }}/src/k8s.io/autoscaler
+
+      - name: Set up Go
+        uses: actions/[email protected]
+        with:
+          go-version: '1.23.11'
+          cache-dependency-path: |
+             ${{ env.GOPATH}}/src/k8s.io/autoscaler/cluster-autoscaler/go.sum
+      - name: Apt-get
+        run: sudo apt-get install libseccomp-dev -qq
+
+      - name: Test
+        working-directory: ${{ env.GOPATH }}/src/k8s.io/autoscaler
+        run: hack/for-go-proj.sh cluster-autoscaler
+        env:
+          GO111MODULE: auto
+          PROJECT_NAMES: ""

.github/workflows/ci.yaml → .github/workflows/verify.yaml

@@ -1,4 +1,4 @@
-name: Tests
+name: Verify Go
 
 on:
   - push
@@ -9,19 +9,25 @@ env:
 
 permissions:
   contents: read
+  checks: write
 
 jobs:
-  test-and-verify:
+  verify:
+    name: verify
     runs-on: ubuntu-latest
     steps:
-      - name: Set up Go
-        uses: actions/setup-go@v2
+      - uses: actions/[email protected]
         with:
-          go-version: '1.22.2'
+          path: ${{ env.GOPATH }}/src/k8s.io/autoscaler
 
-      - uses: actions/checkout@v2
+      - name: Set up Go
+        uses: actions/[email protected]
         with:
-          path: ${{ env.GOPATH }}/src/k8s.io/autoscaler
+          go-version: '1.22.12'
+          cache-dependency-path: |
+             ${{ env.GOPATH}}/src/k8s.io/autoscaler/cluster-autoscaler/go.sum
+             ${{ env.GOPATH}}/src/k8s.io/autoscaler/vertical-pod-autoscaler/go.sum
+             ${{ env.GOPATH}}/src/k8s.io/autoscaler/vertical-pod-autoscaler/e2e/go.sum
 
       - name: Apt-get
         run: sudo apt-get install libseccomp-dev -qq
@@ -36,10 +42,4 @@ jobs:
         working-directory: ${{ env.GOPATH }}/src/k8s.io/autoscaler
         run: hack/verify-all.sh -v
         env:
-          GO111MODULE: auto
-
-      - name: Test
-        working-directory: ${{ env.GOPATH }}/src/k8s.io/autoscaler
-        run: hack/for-go-proj.sh test
-        env:
-          GO111MODULE: auto
+          GO111MODULE: auto

hack/for-go-proj.sh

@@ -19,54 +19,39 @@ set -o pipefail
 set -o nounset
 
 CONTRIB_ROOT="$(dirname ${BASH_SOURCE})/.."
-PROJECT_NAMES=(addon-resizer vertical-pod-autoscaler)
 
 if [[ $# -ne 1 ]]; then
-  echo "missing subcommand: [build|install|test]"
+  echo "missing subcommand: [cluster-autoscaler|vertical-pod-autoscaler|addon-resizer]"
   exit 1
 fi
 
-CMD="${1}"
-
-case "${CMD}" in
-  "build")
+PROJECT="${1}"
+
+case "${PROJECT}" in
+  "cluster-autoscaler")
+    pushd ${CONTRIB_ROOT}/cluster-autoscaler/
+    # TODO: #8127 - Use default analyzers set by `go test` to include `printf` analyzer.
+    # Default analyzers that go test runs according to https://github.com/golang/go/blob/52624e533fe52329da5ba6ebb9c37712048168e0/src/cmd/go/internal/test/test.go#L649
+    # This doesn't include the `printf` analyzer until cluster-autoscaler libraries are updated.
+    ANALYZERS="atomic,bool,buildtags,directive,errorsas,ifaceassert,nilfunc,slog,stringintconv,tests"
+    go test -count=1 ./... -vet="${ANALYZERS}"
+    popd
     ;;
-  "install")
+  "vertical-pod-autoscaler")
+    pushd ${CONTRIB_ROOT}/vertical-pod-autoscaler
+    go test -count=1  -race $(go list ./... | grep -v /vendor/ | grep -v vertical-pod-autoscaler/e2e | grep -v cluster-autoscaler/apis)
+    popd
+    pushd ${CONTRIB_ROOT}/vertical-pod-autoscaler/e2e
+    go test -run=None ./...
+    popd
     ;;
-  "test")
+  "addon-resizer")
+    pushd ${CONTRIB_ROOT}/addon-resizer
+    godep go test -race $(go list ./... | grep -v /vendor/ | grep -v vertical-pod-autoscaler/e2e)
+    popd
     ;;
   *)
     echo "invalid subcommand: ${CMD}"
     exit 1
     ;;
-esac
-
-for project_name in ${PROJECT_NAMES[*]}; do
-  (
-    export GO111MODULE=auto
-    project=${CONTRIB_ROOT}/${project_name}
-    echo "${CMD}ing ${project}"
-    cd "${project}"
-    case "${CMD}" in
-      "test")
-        if [[ -n $(find . -name "Godeps.json") ]]; then
-          godep go test -race $(go list ./... | grep -v /vendor/ | grep -v vertical-pod-autoscaler/e2e)
-        else
-          go test -race $(go list ./... | grep -v /vendor/ | grep -v vertical-pod-autoscaler/e2e | grep -v cluster-autoscaler/apis)
-        fi
-        ;;
-      *)
-        godep go "${CMD}" ./...
-        ;;
-    esac
-  )
-done;
-
-if [ "${CMD}" = "build" ] || [ "${CMD}" == "test" ]; then
-  pushd ${CONTRIB_ROOT}/vertical-pod-autoscaler/e2e
-  go test -mod vendor -run=None ./...
-  popd
-  pushd ${CONTRIB_ROOT}/cluster-autoscaler/
-  go test ./...
-  popd
-fi
+esac

hack/verify-golint.sh

@@ -24,16 +24,13 @@ cd "${KUBE_ROOT}"
 GOLINT=${GOLINT:-"golint"}
 excluded_packages=(
   '/vendor/'
-  'vertical-pod-autoscaler/pkg/client'
   'cluster-autoscaler/cloudprovider/aws/aws-sdk-go'
   'cluster-autoscaler/cloudprovider/magnum/gophercloud'
   'cluster-autoscaler/cloudprovider/digitalocean/godo'
   'cluster-autoscaler/cloudprovider/bizflycloud/gobizfly'
   'cluster-autoscaler/cloudprovider/brightbox/gobrightbox'
   'cluster-autoscaler/cloudprovider/brightbox/k8ssdk'
-  'cluster-autoscaler/cloudprovider/brightbox/linkheader'
   'cluster-autoscaler/cloudprovider/brightbox/go-cache'
-  'cluster-autoscaler/cloudprovider/externalgrpc/protos'
   'cluster-autoscaler/cloudprovider/exoscale/internal'
   'cluster-autoscaler/cloudprovider/huaweicloud/huaweicloud-sdk-go-v3'
   'cluster-autoscaler/cloudprovider/ionoscloud/ionos-cloud-sdk-go'
@@ -44,7 +41,7 @@ excluded_packages=(
   'cluster-autoscaler/cloudprovider/volcengine/volcengine-go-sdk'
 )
 
-FIND_PACKAGES='go list ./... '
+FIND_PACKAGES='go list ./... 2> /dev/null '
 for package in "${excluded_packages[@]}"; do
      FIND_PACKAGES+="| grep -v ${package} "
 done