Merge branch 'main' of https://github.com/malkia/opentelemetry-cpp

Author: malkia

.bazelrc

@@ -60,15 +60,46 @@ build:macos --host_action_env=CC=/opt/homebrew/opt/llvm/bin/clang
 
 # Windows specific asan setup
 # common:asanwin --copt -fsanitize=address,bool,float-cast-overflow,integer-divide-by-zero,null,return,returns-nonnull-attribute,shift-exponent,signed-integer-overflow,unreachable,vla-bound
-common:asanwin --copt -fsanitize=address,bool,float-cast-overflow,integer-divide-by-zero,null,return,returns-nonnull-attribute,shift-exponent,unreachable,vla-bound
-common:asanwin --copt -fsanitize-address-use-after-scope
-common:asanwin --copt -fno-sanitize-recover=all
+#common:asanwin --copt -fsanitize=address,bool,float-cast-overflow,integer-divide-by-zero,null,return,returns-nonnull-attribute,shift-exponent,unreachable,vla-bound
+# vptr is not support for "clang-cl"
+common:asanwin --copt -fsanitize=address,undefined,implicit-integer-truncation,implicit-integer-arithmetic-value-change,implicit-conversion,integer,nullability,float-divide-by-zero,local-bounds
 common:asanwin --copt -fno-sanitize-ignorelist --copt -MD
-common:asanwin --test_env=ASAN_OPTIONS=report_globals=0:detect_odr_violation=0
+#alignment,bool,builtin,bounds,array-bounds,enum,float-cast-overflow,integer-divide-by-zero,function,implicit-unsigned-integer-truncation,implicit-signed-integer-truncation,implicit-integer-sign-change,null,return,returns-nonnull-attribute,shift-exponent,unreachable,vla-bound
+common:asanwin --copt -fsanitize-address-use-after-scope
+common:asanwin --copt -fsanitize-address-globals-dead-stripping
+common:asanwin --copt -fsanitize-address-outline-instrumentation
+common:asanwin --copt -fsanitize-address-use-after-return=always
+common:asanwin --copt -fsanitize-stable-abi
+common:asanwin --copt -fno-sanitize-merge
+common:asanwin --copt -fsanitize-recover=all
+
+# Gives:
+#    lld-link: error: /alternatename: conflicts: __sanitizer_on_print=__sanitizer_on_print__def
+#    Use --verbose_failures to see the command lines of failed build steps.
+# common:asanwin --copt -fsanitize-stats
+
+# Gives:
+#    lld-link: error: duplicate symbol: __cfi_check_fail
+#    >>> defined at runfiles.lib(runfiles.obj)
+# common:asanwin --copt -fsanitize-cfi-cross-dso
+
+# Gives multiple, like
+#   lld-link: error: duplicate symbol: __odr_asan_gen_??_R0?AVios_base@std@@@8
+#   >>> defined at runfiles.lib(runfiles.obj)
+# common:asanwin --copt -fsanitize-address-use-odr-indicator
+
+# https://github.com/llvm/llvm-project/blob/main/compiler-rt/lib/asan/asan_flags.inc
+common:asanwin --test_env="ASAN_OPTIONS=report_globals=0:detect_odr_violation=0:verbosity=1:debug=1:strict_init_order=1:check_initialization_order=1:detect_stack_use_after_return=1:print_stats=1:verbosity=2:atexit=1:alloc_dealloc_mismatch=1:strict_string_checks=1:halt_on_error=0:malloc_context_size=0:windows_hook_rtl_allocators=1:print_scariness=1"
+
+# https://github.com/llvm/llvm-project/blob/main/compiler-rt/lib/ubsan/ubsan_flags.inc
+common:asanwin --test_env="UBSAN_OPTIONS=print_stacktrace=1:report_error_type=1"
+# common:asanwin --test_env="ASAN_OPTIONS=help=1"
 common:asanwin --cc_output_directory_tag=asanwin
 common:asanwin --@otel_sdk//:asanwin=true
-
-# --config=asan : Address Sanitizer.
+test:asanwin --test_timeout=3600
+# common:asanwin -fno-sanitize-coverage=<value>
+# common:asanwin -fno-sanitize-recover=<value>
+ 
 common:asan --copt -DADDRESS_SANITIZER
 common:asan --copt -fsanitize=address,bool,float-cast-overflow,integer-divide-by-zero,null,return,returns-nonnull-attribute,shift-exponent,signed-integer-overflow,unreachable,vla-bound
 common:asan --copt -fsanitize-address-use-after-scope
@@ -170,6 +201,8 @@ build --host_copt="-DGRPC_NO_ABSL_SYNC"
 # disable all GRPC experiments
 # build --define=grpc_experiments_are_final=true
 
+common:devenv --run_under='"c:/windows/system32/cmd.exe" /C start "bazel debug exe" /wait devenv /debugexe '
+
 ## This is what my ../top.bazelrc contains (not in the repo as local to my machine)
 # build --disk_cache=f:/b/d
 # common --repository_cache=f:/b/r

.bazelversion

@@ -1 +1 @@
-8.2.1
+8.3.0rc3

.github/workflows/benchmark.yml

@@ -40,7 +40,7 @@ jobs:
           mv api-benchmark_result.json benchmarks
           mv sdk-benchmark_result.json benchmarks
           mv exporters-benchmark_result.json benchmarks
-      - uses: actions/upload-artifact@6027e3dd177782cd8ab9af838c04fd81a07f1d47 # main March 2025
+      - uses: actions/upload-artifact@de65e23aa2b7e23d713bb51fbfcb6d502f8667d8 # main March 2025
         with:
           name: benchmark_results
           path: benchmarks
@@ -58,7 +58,7 @@ jobs:
           egress-policy: audit
 
       - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
-      - uses: actions/download-artifact@d3f86a106a0bac45b974a628896c90dbdf5c8093 # main March 2025
+      - uses: actions/download-artifact@448e3f862ab3ef47aa50ff917776823c9946035b # main March 2025
         with:
           name: benchmark_results
           path: benchmarks

.github/workflows/otel_sdk.yml

@@ -22,7 +22,7 @@ jobs:
       # - uses: Cyberboss/install-winget@6f566e9e227561d5d31e45d65f9b777b3fa8f56f
       - run: winget list --accept-source-agreements --disable-interactivity
       - run: winget install bazelisk -r bazel --accept-package-agreements --disable-interactivity --silent
-      - run: winget install -e LLVM.LLVM -v 20.1.6 --accept-package-agreements --disable-interactivity --silent || echo swalloing errors
+      - run: winget install -e LLVM.LLVM -v 20.1.7 --accept-package-agreements --disable-interactivity --silent || echo swalloing errors
       - uses: actions/checkout@v4
       - uses: actions/cache/restore@v4
         with:

.github/workflows/project_management_comment.yml

@@ -20,7 +20,7 @@ jobs:
           egress-policy: audit
 
       - name: Add comment
-        uses: peter-evans/create-or-update-comment@e5c3b7cef58546e37d0e9e64128fa39d4c31706a
+        uses: peter-evans/create-or-update-comment@09354dd9af7b86869f7f4989b2a52d8eb1881a9c
         with:
           issue-number: ${{ github.event.issue.number }}
           body: |

MODULE.bazel

@@ -4,8 +4,8 @@ module(
     compatibility_level = 0,
 )
 
-bazel_dep(name = "abseil-cpp", version = "20250512.0")
-bazel_dep(name = "aspect_bazel_lib", version = "2.19.2")
+bazel_dep(name = "abseil-cpp", version = "20250512.1")
+bazel_dep(name = "aspect_bazel_lib", version = "2.19.4")
 bazel_dep(name = "bazel_skylib", version = "1.7.1")
 bazel_dep(name = "boringssl", version = "0.20250514.0")
 bazel_dep(name = "curl", version = "8.8.0.bcr.3")
@@ -14,7 +14,7 @@ bazel_dep(name = "google_benchmark", version = "1.9.4")
 bazel_dep(name = "googletest", version = "1.17.0")
 bazel_dep(name = "re2", version = "2024-07-02.bcr.1")
 bazel_dep(name = "rules_go", version = "0.55.1")
-bazel_dep(name = "gazelle", version = "0.43.0")
+bazel_dep(name = "gazelle", version = "0.44.0")
 bazel_dep(name = "stardoc", version = "0.8.0")
 bazel_dep(name = "rules_android", version = "0.6.4")
 bazel_dep(name = "grpc", version = "1.72.0")
@@ -27,12 +27,12 @@ single_version_override(
 )
 bazel_dep(name = "nlohmann_json", version = "3.12.0")
 bazel_dep(name = "platforms", version = "1.0.0")
-bazel_dep(name = "rules_cc", version = "0.1.1")
+bazel_dep(name = "rules_cc", version = "0.1.2")
 bazel_dep(name = "rules_pkg", version = "1.1.0")
 bazel_dep(name = "rules_proto", version = "7.1.0")
 bazel_dep(name = "zlib", version = "1.3.1.bcr.6")
 bazel_dep(name = "opentracing-cpp", version = "1.6.0")
-bazel_dep(name = "rules_multitool", version = "1.3.0")
+bazel_dep(name = "rules_multitool", version = "1.5.0")
 bazel_dep(name = "protoc-gen-validate", version = "1.2.1.bcr.1")
 bazel_dep(name = "c-ares", version = "1.34.5")
 archive_override(
@@ -44,7 +44,7 @@ archive_override(
 bazel_dep(name = "protobuf", version = "31.1")
 bazel_dep(name = "opentelemetry-proto", version = "1.7.0")
 bazel_dep(name = "prometheus-cpp", version = "1.3.0.bcr.2")
-bazel_dep(name = "civetweb", version = "1.16.bcr.1")
+bazel_dep(name = "civetweb", version = "1.16.bcr.2")
 
 use_repo(
     use_extension("@rules_cc//cc:extensions.bzl", "cc_configure_extension"),

api/test/nostd/string_view_test.cc

@@ -72,7 +72,7 @@ TEST(StringViewTest, SubstrPortion)
   EXPECT_EQ("12", s.substr(3, 2));
 }
 
-TEST(StringViewTest, SubstrOutOfRange)
+static void __attribute__((no_sanitize("address"))) StringViewTest_SubstrOutOfRange_NoSanatize()
 {
   string_view s = "abc123";
 #if __EXCEPTIONS || (defined(OPENTELEMETRY_STL_VERSION) && (OPENTELEMETRY_STL_VERSION >= 2017))
@@ -82,6 +82,11 @@ TEST(StringViewTest, SubstrOutOfRange)
 #endif
 }
 
+TEST(StringViewTest, SubstrOutOfRange)
+{
+  StringViewTest_SubstrOutOfRange_NoSanatize();
+}
+
 TEST(StringViewTest, FindSingleCharacter)
 {
   string_view s = "abc";