Skip to content

Commit

Permalink
fixup! internal/apps: Use latest go version (1.23)
Browse files Browse the repository at this point in the history
  • Loading branch information
felixge committed Sep 14, 2024
1 parent d786217 commit ffa7cfd
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 1 deletion.
2 changes: 2 additions & 0 deletions .github/workflows/smoke-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -106,6 +106,7 @@ jobs:
matrix:
# TODO: cross-compilation from/to different hardware architectures once
# github provides native ARM runners.
# NOTE: Also edit go_min below when changing the go versions here.
go: [ "1.22", "1.23" ]
build-env: [ alpine, bookworm, bullseye ]
build-with-cgo: [ 0, 1 ]
Expand Down Expand Up @@ -184,6 +185,7 @@ jobs:
tags: smoke-test
build-args: |
go=${{ matrix.go }}
go_min=1.22
build_env=${{ matrix.build-env }}
build_tags=${{ matrix.build-tags }}
build_with_vendoring=${{ matrix.build-with-vendoring }}
Expand Down
2 changes: 1 addition & 1 deletion internal/apps/go.mod
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
module github.com/DataDog/dd-trace-go/internal/apps

go 1.23.0
go 1.23

require (
golang.org/x/sync v0.5.0
Expand Down
9 changes: 9 additions & 0 deletions internal/apps/setup-smoke-test/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,15 @@ RUN if [ "${go_libddwaf_ref}" != "" ]; then \
go get -u github.com/DataDog/go-libddwaf/v3@${go_libddwaf_ref}; \
fi


# internal/apps/go.mod is usually on the latest go version, but this would fail
# when we are testing against an older go version.
#
# TODO(fg) this is a workaround. We should probably move setup-smoke-test out
# of internal/apps and into a separate directory.
ARG go_min
RUN go mod edit -go="${go_min}"

RUN go mod tidy

ARG build_with_vendoring
Expand Down

0 comments on commit ffa7cfd

Please sign in to comment.