diff --git a/.github/workflows/validate.yml b/.github/workflows/validate.yml index 055340640f2..74cc90b14ff 100644 --- a/.github/workflows/validate.yml +++ b/.github/workflows/validate.yml @@ -7,6 +7,8 @@ on: - master - release-* pull_request: +env: + GO_VERSION: 1.17.x jobs: @@ -14,14 +16,16 @@ jobs: runs-on: ubuntu-20.04 steps: - uses: actions/checkout@v3 + - uses: actions/setup-go@v2 + with: + go-version: "${{ env.GO_VERSION }}" - name: install deps run: | sudo apt -q update sudo apt -q install libseccomp-dev - - uses: golangci/golangci-lint-action@v2 + - uses: golangci/golangci-lint-action@v3 with: - # must be specified without patch version - version: v1.42 + version: v1.44 lint-extra: # Extra linters, only checking new code from pull requests. @@ -31,16 +35,18 @@ jobs: contents: read steps: - uses: actions/checkout@v3 + - uses: actions/setup-go@v2 + with: + go-version: "${{ env.GO_VERSION }}" - name: install deps run: | sudo apt -q update sudo apt -q install libseccomp-dev - - uses: golangci/golangci-lint-action@v2 + - uses: golangci/golangci-lint-action@v3 with: only-new-issues: true args: --config .golangci-extra.yml - # must be specified without patch version - version: v1.43 + version: v1.44 compile-buildtags: @@ -53,7 +59,7 @@ jobs: - name: install go uses: actions/setup-go@v2 with: - go-version: 1.x # Latest stable + go-version: "${{ env.GO_VERSION }}" - name: compile with no build tags run: make BUILDTAGS="" @@ -120,7 +126,7 @@ jobs: - name: install go uses: actions/setup-go@v2 with: - go-version: 1.x # Latest stable + go-version: "${{ env.GO_VERSION }}" - name: cache go mod and $GOCACHE uses: actions/cache@v2 with: diff --git a/libcontainer/cgroups/ebpf/devicefilter/devicefilter_test.go b/libcontainer/cgroups/ebpf/devicefilter/devicefilter_test.go index a8fc562d038..d279335821d 100644 --- a/libcontainer/cgroups/ebpf/devicefilter/devicefilter_test.go +++ b/libcontainer/cgroups/ebpf/devicefilter/devicefilter_test.go @@ -153,8 +153,7 @@ func TestDeviceFilter_Privileged(t *testing.T) { Allow: true, }, } - expected := - ` + expected := ` // load parameters into registers 0: LdXMemW dst: r2 src: r1 off: 0 imm: 0 1: And32Imm dst: r2 imm: 65535 diff --git a/libcontainer/process_linux.go b/libcontainer/process_linux.go index ae91d5b900f..c2ee2da6b13 100644 --- a/libcontainer/process_linux.go +++ b/libcontainer/process_linux.go @@ -39,13 +39,9 @@ type parentProcess interface { // startTime returns the process start time. startTime() (uint64, error) - signal(os.Signal) error - externalDescriptors() []string - setExternalDescriptors(fds []string) - forwardChildLogs() chan error } diff --git a/update.go b/update.go index d02e7af90d3..9ce5a2e835b 100644 --- a/update.go +++ b/update.go @@ -194,7 +194,6 @@ other options are ignored. opt string dest *uint64 }{ - {"cpu-period", r.CPU.Period}, {"cpu-rt-period", r.CPU.RealtimePeriod}, {"cpu-share", r.CPU.Shares}, @@ -211,7 +210,6 @@ other options are ignored. opt string dest *int64 }{ - {"cpu-quota", r.CPU.Quota}, {"cpu-rt-runtime", r.CPU.RealtimeRuntime}, } {