feat: introduce a new workflow to verify build script on multi platforms (#196)

* feat: introduce new Check workflow

* fix: correct matrix

* fix: Make different matrices run independently of each other

* test: test init_env in the step

* style: resolve fmt issues

Signed-off-by: spencercjh <[email protected]>

* chore: simplify matrix

Signed-off-by: spencercjh <[email protected]>

* chore: rename the wf

Signed-off-by: spencercjh <[email protected]>

* chore: remove irrelevant codes to check format issues

Signed-off-by: spencercjh <[email protected]>

* fix: add missing build step

Signed-off-by: spencercjh <[email protected]>

* fix: add missing clang_num init biz for ubuntu 20.04

Signed-off-by: spencercjh <[email protected]>

---------

Signed-off-by: spencercjh <[email protected]>

Author: spencercjh

.github/workflows/build_verification.yml

@@ -0,0 +1,47 @@
+name: Build Verification
+
+on:
+  workflow_dispatch:
+  push:
+    branches: [ "main", "dev","feature/*" ]
+  pull_request:
+    branches: [ "main", "dev","feature/*" ]
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: ${{ startsWith(github.ref, 'refs/pull/') }}
+
+env:
+  kyanos_log_option: --bpf-event-log-level 5 --conntrack-log-level 5
+
+permissions:
+  contents: read
+
+jobs:
+  verify:
+    timeout-minutes: 30
+    continue-on-error: true
+    # TODO：adapt to different architecture runner
+    runs-on: ${{ matrix.os }}
+    # https://docs.github.com/en/actions/using-github-hosted-runners/using-github-hosted-runners/about-github-hosted-runners#standard-github-hosted-runners-for-public-repositories
+    strategy:
+      matrix:
+        os: [ "ubuntu-20.04","ubuntu-22.04","ubuntu-24.04" ]
+
+    steps:
+      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
+        with:
+          submodules: recursive
+
+      - name: Set up Go
+        uses: actions/setup-go@3041bf56c941b39c61721a86cd11f3bb1338122a # v5
+        with:
+          go-version: '1.23.3'
+
+      - name: Init
+        run: |
+          /bin/bash init_env.sh
+
+      - name: Build
+        run: |
+          make clean && make build-bpf && make

Makefile

@@ -87,4 +87,18 @@ btfgen:
 .DELETE_ON_ERROR:
 
 # keep intermediate (.skel.h, .bpf.o, etc) targets
-.SECONDARY:
+.SECONDARY:
+
+.PHONY: test
+test: test-go
+
+.PHONY: test-go
+test-go:
+	go test -v ./...
+
+.PHONY: format
+format: format-go
+
+.PHONY: format-go
+format-go:
+	gofmt -s -w .

bpf/loader/loader.go

@@ -209,7 +209,7 @@ func filterFunctions(coll *ebpf.CollectionSpec, kernelVersion compatible.KernelV
 	}
 
 	needsDelete := make([]string, 0)
-	for cProgName, _ := range coll.Programs {
+	for cProgName := range coll.Programs {
 		if slices.Index(finalCProgNames, cProgName) == -1 {
 			needsDelete = append(needsDelete, cProgName)
 		}

init_env.sh

@@ -8,7 +8,9 @@ fi
 
 CLANG_NUM=-12
 # shellcheck disable=SC2209
-if [ ${release_num} == "20.10" ]; then
+if [ ${release_num} == "20.04" ]; then
+  CLANG_NUM=-10
+  elif [ ${release_num} == "20.04" ]; then
   CLANG_NUM=-10
   elif [ ${release_num} == "21.04" ]; then
   CLANG_NUM=-11
@@ -21,7 +23,7 @@ if [ ${release_num} == "20.10" ]; then
   elif [ ${release_num} == "23.04" ];then
   CLANG_NUM=-15
   elif [ ${release_num} == "23.10" ];then
-    CLANG_NUM=-15
+  CLANG_NUM=-15
   elif [ ${release_num} == "24.04" ];then
   CLANG_NUM=-18
   else