Skip to content

Commit dc38b1a

Browse files
authored
Merge branch 'main' into dmehala/capture-span-start-asap
2 parents 914b50b + 286d0e3 commit dc38b1a

36 files changed

+644
-201
lines changed

.bazelrc.absl

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,12 @@
77
# This bazelrc defines the `DD_USE_ABSEIL_FOR_ENVOY` preprocessor macro, and so
88
# the resulting library will use `absl::string_view` and `absl::optional`
99
# instead of their standard (`std`) equivalents.
10+
#
11+
# Additionally, we configure Abseil to not use std library types by setting
12+
# the appropriate ABSL_OPTION_* macros, replacing the need for patching.
1013

1114
build --enable_platform_specific_config
1215

13-
build:linux --cxxopt='-std=c++17' --cxxopt='-Wall' --cxxopt='-Wextra' --cxxopt='-pedantic' --cxxopt='-DDD_USE_ABSEIL_FOR_ENVOY'
14-
build:macos --cxxopt='-std=c++17' --cxxopt='-Wall' --cxxopt='-Wextra' --cxxopt='-pedantic' --cxxopt='-DDD_USE_ABSEIL_FOR_ENVOY'
15-
build:windows --cxxopt='/std:c++17' --cxxopt='/DDD_USE_ABSEIL_FOR_ENVOY' --linkopt='ws2_32.lib'
16+
build:linux --cxxopt='-std=c++17' --cxxopt='-Wall' --cxxopt='-Wextra' --cxxopt='-pedantic' --cxxopt='-DDD_USE_ABSEIL_FOR_ENVOY' --cxxopt='-DABSL_OPTION_USE_STD_OPTIONAL=0' --cxxopt='-DABSL_OPTION_USE_STD_STRING_VIEW=0' --cxxopt='-DABSL_OPTION_USE_STD_VARIANT=0'
17+
build:macos --cxxopt='-std=c++17' --cxxopt='-Wall' --cxxopt='-Wextra' --cxxopt='-pedantic' --cxxopt='-DDD_USE_ABSEIL_FOR_ENVOY' --cxxopt='-DABSL_OPTION_USE_STD_OPTIONAL=0' --cxxopt='-DABSL_OPTION_USE_STD_STRING_VIEW=0' --cxxopt='-DABSL_OPTION_USE_STD_VARIANT=0'
18+
build:windows --cxxopt='/std:c++17' --cxxopt='/DDD_USE_ABSEIL_FOR_ENVOY' --cxxopt='/DABSL_OPTION_USE_STD_OPTIONAL=0' --cxxopt='/DABSL_OPTION_USE_STD_STRING_VIEW=0' --cxxopt='/DABSL_OPTION_USE_STD_VARIANT=0' --linkopt='ws2_32.lib'

.github/workflows/codeql-analysis.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ jobs:
2525
uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
2626

2727
- name: Initialize CodeQL
28-
uses: github/codeql-action/init@2d92b76c45b91eb80fc44c74ce3fce0ee94e8f9d # v3.29.5
28+
uses: github/codeql-action/init@3599b3baa15b485a2e49ef411a7a4bb2452e7f93 # v3.29.5
2929
with:
3030
languages: ${{ matrix.language }}
3131

@@ -36,4 +36,4 @@ jobs:
3636
run: bin/cmake-build --preset=ci-codeql
3737

3838
- name: Perform CodeQL Analysis
39-
uses: github/codeql-action/analyze@2d92b76c45b91eb80fc44c74ce3fce0ee94e8f9d # v3.29.5
39+
uses: github/codeql-action/analyze@3599b3baa15b485a2e49ef411a7a4bb2452e7f93 # v3.29.5

.github/workflows/dev.yml

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,9 @@ jobs:
55
format:
66
runs-on: ubuntu-22.04-arm
77
container:
8-
image: datadog/docker-library:dd-trace-cpp-ci-5f5c273-arm64
8+
image: datadog/docker-library:dd-trace-cpp-ci-23768e9-arm64
99
steps:
10-
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
10+
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
1111
- name: Check format
1212
run: bin/check-format
1313
- name: Shellcheck
@@ -29,7 +29,7 @@ jobs:
2929
needs: format
3030
runs-on: ${{ matrix.runner }}
3131
container:
32-
image: datadog/docker-library:dd-trace-cpp-ci-5f5c273-${{matrix.docker-arch}}
32+
image: datadog/docker-library:dd-trace-cpp-ci-23768e9-${{matrix.docker-arch}}
3333
environment:
3434
name: dev
3535
permissions:
@@ -39,7 +39,7 @@ jobs:
3939
BUILD_DIR: .build
4040
DD_API_KEY: ${{ secrets.DD_CI_VIS_API_KEY }}
4141
steps:
42-
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
42+
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
4343
- name: Configure
4444
run: bin/with-toolchain ${{ matrix.toolchain }} cmake . -B .build -DCMAKE_BUILD_TYPE=Debug -DDD_TRACE_BUILD_TESTING=1
4545
- name: Build
@@ -49,7 +49,7 @@ jobs:
4949
- name: Upload test report to Datadog
5050
run: |
5151
curl -L --fail "https://github.com/DataDog/datadog-ci/releases/latest/download/datadog-ci_linux-${{ matrix.arch }}" --output "/usr/local/bin/datadog-ci" && chmod +x /usr/local/bin/datadog-ci
52-
datadog-ci junit upload --service dd-trace-cpp .build/report.xml
52+
datadog-ci junit upload --service dd-trace-cpp --tags test.source.file:test/*.cpp .build/report.xml
5353
5454
build-linux-bazel:
5555
needs: format
@@ -68,9 +68,9 @@ jobs:
6868
docker-arch: amd64
6969
runs-on: ${{ matrix.runner }}
7070
container:
71-
image: datadog/docker-library:dd-trace-cpp-ci-5f5c273-${{matrix.docker-arch}}
71+
image: datadog/docker-library:dd-trace-cpp-ci-23768e9-${{matrix.docker-arch}}
7272
steps:
73-
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
73+
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
7474
- name: Build
7575
run: bin/with-toolchain ${{ matrix.toolchain }} bazelisk --bazelrc=${{ matrix.bazelrc }} build dd_trace_cpp
7676

@@ -84,7 +84,7 @@ jobs:
8484
matrix:
8585
bazelrc: [".bazelrc.absl", ".bazelrc.std"]
8686
steps:
87-
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
87+
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
8888
- name: Install Dependency Manager (scoop)
8989
run: |
9090
Set-ExecutionPolicy -ExecutionPolicy RemoteSigned -Scope CurrentUser
@@ -118,7 +118,7 @@ jobs:
118118
env:
119119
DD_API_KEY: ${{ secrets.DD_CI_VIS_API_KEY }}
120120
steps:
121-
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
121+
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
122122
- name: Install Dependency Manager (scoop)
123123
run: |
124124
Set-ExecutionPolicy -ExecutionPolicy RemoteSigned -Scope CurrentUser
@@ -138,13 +138,13 @@ jobs:
138138
- name: Upload test report to Datadog
139139
run: |
140140
Invoke-WebRequest -Uri "https://github.com/DataDog/datadog-ci/releases/latest/download/datadog-ci_win-x64" -OutFile "datadog-ci.exe"
141-
./datadog-ci.exe junit upload --service dd-trace-cpp report.xml
141+
./datadog-ci.exe junit upload --service dd-trace-cpp --tags test.source.file:test/*.cpp report.xml
142142
143143
coverage:
144144
needs: build-linux-cmake
145145
runs-on: ubuntu-22.04-arm
146146
container:
147-
image: datadog/docker-library:dd-trace-cpp-ci-5f5c273-arm64
147+
image: datadog/docker-library:dd-trace-cpp-ci-23768e9-arm64
148148
environment:
149149
name: dev
150150
permissions:
@@ -153,7 +153,7 @@ jobs:
153153
env:
154154
DD_API_KEY: ${{ secrets.DD_CI_VIS_API_KEY }}
155155
steps:
156-
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
156+
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
157157
- run: bin/test --coverage --verbose
158158
- name: Report Datadog coverage
159159
run: |

.github/workflows/main.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,11 +40,11 @@ jobs:
4040
needs: call-dev-workflow
4141
runs-on: ubuntu-22.04-arm
4242
container:
43-
image: datadog/docker-library:dd-trace-cpp-ci-5f5c273-arm64
43+
image: datadog/docker-library:dd-trace-cpp-ci-23768e9-arm64
4444
env:
4545
DURATION_SEC: 300 # 5min
4646
steps:
47-
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
47+
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
4848
- name: Configure
4949
run: bin/with-toolchain llvm cmake . -B .build -DCMAKE_BUILD_TYPE=Debug -DDD_TRACE_BUILD_FUZZERS=1 -DDD_TRACE_ENABLE_SANITIZE=1 -DDD_TRACE_TRANSPORT=none
5050
- name: Build

.gitignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
/bazel-*
22
/.build/
3+
/build/
4+
/build-*/
35
/.coverage/
6+
build/
47

58
dist/
69
out/

0 commit comments

Comments
 (0)