Skip to content

Commit

Permalink
Bump go minor version and golangci
Browse files Browse the repository at this point in the history
  • Loading branch information
secwall committed Nov 27, 2024
1 parent 73ebf59 commit fbbf193
Show file tree
Hide file tree
Showing 11 changed files with 15 additions and 15 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/codeql.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ on:
- cron: '30 06 * * 6'

env:
GO_VERSION: 1.23.2
GO_VERSION: 1.23.3

jobs:
analyze:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/func-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ on:
- cron: '30 06 * * *'

env:
GO_VERSION: 1.23.2
GO_VERSION: 1.23.3
DOCKER_API_VERSION: 1.43

jobs:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/golangci-lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ on:
branches: [ master ]

env:
GO_VERSION: 1.23.2
GO_VERSION: 1.23.3

permissions:
contents: read
Expand All @@ -24,4 +24,4 @@ jobs:
- name: golangci-lint
uses: golangci/golangci-lint-action@v6
with:
version: v1.60
version: v1.62
2 changes: 1 addition & 1 deletion .github/workflows/jepsen-cluster-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ on:
- cron: '30 06 * * *'

env:
GO_VERSION: 1.23.2
GO_VERSION: 1.23.3

jobs:
test:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/jepsen-sentinel-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ on:
- cron: '30 06 * * *'

env:
GO_VERSION: 1.23.2
GO_VERSION: 1.23.3

jobs:
test:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/unit-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ on:
branches: [ master ]

env:
GO_VERSION: 1.23.2
GO_VERSION: 1.23.3

jobs:
unittest:
Expand Down
4 changes: 2 additions & 2 deletions .golangci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,9 +40,9 @@ linters:
disable-all: true
enable:
- bodyclose
- copyloopvar
- dupl
- errcheck
- exportloopref
- funlen
- gocritic
- gofmt
Expand All @@ -62,7 +62,7 @@ linters:
- whitespace

run:
go: 1.21
go: 1.23
timeout: 5m

severity:
Expand Down
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ format:
goimports -w `find . -name '*.go'`

lint:
docker run --rm -v ${CURDIR}:/app -w /app golangci/golangci-lint:v1.60-alpine golangci-lint run -v
docker run --rm -v ${CURDIR}:/app -w /app golangci/golangci-lint:v1.62-alpine golangci-lint run -v

unittests:
go test ./cmd/... ./internal/...
Expand Down
6 changes: 3 additions & 3 deletions internal/app/state.go
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,9 @@ import (
"github.com/yandex/rdsync/internal/dcs"
)

func (app *App) setStateError(state *HostState, fqdn, error string) {
app.logger.Error("GetHostState error", "fqdn", fqdn, "error", error)
state.Error = error
func (app *App) setStateError(state *HostState, fqdn, message string) {
app.logger.Error("GetHostState error", "fqdn", fqdn, "error", message)
state.Error = message
}

func (app *App) getHostState(fqdn string) *HostState {
Expand Down
2 changes: 1 addition & 1 deletion tests/features/07_cluster_local_repair.feature
Original file line number Diff line number Diff line change
Expand Up @@ -32,5 +32,5 @@ Feature: Cluster mode local node repair
"""
redis-cli -a functestpassword DEBUG SLEEP 600
"""
And I wait for "300" seconds
And I wait for "420" seconds
Then redis host "redis1" should become available within "60" seconds
2 changes: 1 addition & 1 deletion tests/features/07_sentinel_local_repair.feature
Original file line number Diff line number Diff line change
Expand Up @@ -49,5 +49,5 @@ Feature: Sentinel mode local node repair
"""
redis-cli -a functestpassword DEBUG SLEEP 600
"""
And I wait for "300" seconds
And I wait for "420" seconds
Then redis host "redis1" should become available within "60" seconds

0 comments on commit fbbf193

Please sign in to comment.