Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Debug v0.33 go 1.22 debug #6813

Closed
wants to merge 11 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
7 changes: 5 additions & 2 deletions .github/workflows/cd.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,13 @@ name: CD

on:
push:
tags:
tags:
- '*'
- "!daily-*"

env:
GO_VERSION: "1.22"

jobs:
docker-push:
name: Push to container registry
Expand All @@ -15,7 +18,7 @@ jobs:
uses: actions/setup-go@v4
timeout-minutes: 10 # fail fast. sometimes this step takes an extremely long time
with:
go-version: "1.20"
go-version: ${{ env.GO_VERSION }}
- name: Checkout repo
uses: actions/checkout@v2
# Provide Google Service Account credentials to Github Action, allowing interaction with the Google Container Registry
Expand Down
20 changes: 17 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ on:
- master

env:
GO_VERSION: "1.20"
GO_VERSION: "1.22"

concurrency:
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.run_id }}
Expand All @@ -42,8 +42,22 @@ jobs:
with:
go-version: ${{ env.GO_VERSION }}
cache: true
- name: Install protoc
run: |
sudo apt-get update
sudo apt-get install -y protobuf-compiler
protoc --version
- name: Install protoc-gen-go and protoc-gen-go-grpc
run: |
go install google.golang.org/protobuf/cmd/[email protected]
go install google.golang.org/grpc/cmd/[email protected]
echo "$(go env GOPATH)/bin" >> $GITHUB_PATH
- name: Validate plugins
run: |
which protoc-gen-go
which protoc-gen-go-grpc
- name: Run go generate
run: go generate
run: go generate ./...
working-directory: ${{ matrix.dir }}
- name: Run golangci-lint
uses: golangci/golangci-lint-action@v3
Expand All @@ -54,7 +68,7 @@ jobs:
working-directory: ${{ matrix.dir }}
# https://github.com/golangci/golangci-lint-action/issues/244
skip-cache: true


tidy:
name: Tidy
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/flaky-test-monitor.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ env:
BIGQUERY_DATASET: production_src_flow_test_metrics
BIGQUERY_TABLE: skipped_tests
BIGQUERY_TABLE2: test_results
GO_VERSION: "1.20"
GO_VERSION: "1.22"
SKIPPED_TESTS_FILE: skipped-tests
RESULTS_FILE: test-results
COMMIT_SHA: ${{ github.sha }}
Expand Down
5 changes: 4 additions & 1 deletion .github/workflows/tools.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,9 @@ on:
required: false
type: boolean

env:
GO_VERSION: "1.22"

jobs:
build-publish:
name: Build boot tools
Expand All @@ -31,7 +34,7 @@ jobs:
- name: Setup Go
uses: actions/setup-go@v4
with:
go-version: "1.20"
go-version: ${{ env.GO_VERSION }}
- name: Checkout repo
uses: actions/checkout@v3
with:
Expand Down
14 changes: 7 additions & 7 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ unittest-main:
.PHONY: install-mock-generators
install-mock-generators:
cd ${GOPATH}; \
go install github.com/vektra/mockery/v2@v2.21.4; \
go install github.com/vektra/mockery/v2@v2.43.2; \
go install github.com/golang/mock/[email protected];

.PHONY: install-tools
Expand Down Expand Up @@ -650,7 +650,7 @@ docker-push-collection-without-adx:
docker-push-collection-without-netgo-without-adx:
docker push "$(CONTAINER_REGISTRY)/collection:$(IMAGE_TAG_NO_NETGO_NO_ADX)"

.PHONY: docker-push-collection-arm
.PHONY: docker-push-collection-arm
docker-push-collection-arm:
docker push "$(CONTAINER_REGISTRY)/collection:$(IMAGE_TAG_ARM)"

Expand All @@ -670,7 +670,7 @@ docker-push-consensus-without-adx:
docker-push-consensus-without-netgo-without-adx:
docker push "$(CONTAINER_REGISTRY)/consensus:$(IMAGE_TAG_NO_NETGO_NO_ADX)"

.PHONY: docker-push-consensus-arm
.PHONY: docker-push-consensus-arm
docker-push-consensus-arm:
docker push "$(CONTAINER_REGISTRY)/consensus:$(IMAGE_TAG_ARM)"

Expand All @@ -694,7 +694,7 @@ docker-push-execution-without-adx:
docker-push-execution-without-netgo-without-adx:
docker push "$(CONTAINER_REGISTRY)/execution:$(IMAGE_TAG_NO_NETGO_NO_ADX)"

.PHONY: docker-push-execution-arm
.PHONY: docker-push-execution-arm
docker-push-execution-arm:
docker push "$(CONTAINER_REGISTRY)/execution:$(IMAGE_TAG_ARM)"

Expand All @@ -718,7 +718,7 @@ docker-push-verification-corrupt:
docker-push-verification-without-netgo-without-adx:
docker push "$(CONTAINER_REGISTRY)/verification:$(IMAGE_TAG_NO_NETGO_NO_ADX)"

.PHONY: docker-push-verification-arm
.PHONY: docker-push-verification-arm
docker-push-verification-arm:
docker push "$(CONTAINER_REGISTRY)/verification:$(IMAGE_TAG_ARM)"

Expand All @@ -742,7 +742,7 @@ docker-push-access-corrupt:
docker-push-access-without-netgo-without-adx:
docker push "$(CONTAINER_REGISTRY)/access:$(IMAGE_TAG_NO_NETGO_NO_ADX)"

.PHONY: docker-push-access-arm
.PHONY: docker-push-access-arm
docker-push-access-arm:
docker push "$(CONTAINER_REGISTRY)/access:$(IMAGE_TAG_ARM)"

Expand All @@ -763,7 +763,7 @@ docker-push-observer-without-adx:
docker-push-observer-without-netgo-without-adx:
docker push "$(CONTAINER_REGISTRY)/observer:$(IMAGE_TAG_NO_NETGO_NO_ADX)"

.PHONY: docker-push-observer-arm
.PHONY: docker-push-observer-arm
docker-push-observer-arm:
docker push "$(CONTAINER_REGISTRY)/observer:$(IMAGE_TAG_ARM)"

Expand Down
Loading
Loading