Merge branch 'main' into fix_iwyu_cleanup_5

Author: marcalff

.bazelrc

@@ -7,6 +7,9 @@
 # Enable automatic configs based on platform
 common --enable_platform_specific_config
 
+# Make globs that don't match anything fail
+common --incompatible_disallow_empty_glob
+
 # Needed by gRPC to build on some platforms.
 build --copt -DGRPC_BAZEL_BUILD
 

CMakeLists.txt

@@ -260,10 +260,6 @@ if(OTELCPP_VERSIONED_LIBS AND NOT BUILD_SHARED_LIBS)
   message(FATAL_ERROR "OTELCPP_VERSIONED_LIBS=ON requires BUILD_SHARED_LIBS=ON")
 endif()
 
-set(OTELCPP_PROTO_PATH
-    ""
-    CACHE PATH "Path to opentelemetry-proto")
-
 if(WIN32)
   option(WITH_ETW "Whether to include the ETW Exporter in the SDK" ON)
 else()

cmake/opentelemetry-cpp-config.cmake.in

@@ -105,7 +105,7 @@ endif()
 
 if(@WITH_ABSEIL@ OR @WITH_OTLP_GRPC@)
   find_package(absl CONFIG)
-elseif(OR @WITH_OTLP_HTTP@ OR @WITH_OTLP_FILE@)
+elseif(@WITH_OTLP_HTTP@ OR @WITH_OTLP_FILE@)
   if("@Protobuf_VERSION@" VERSION_GREATER_EQUAL "3.22.0")
     find_package(absl CONFIG)
   endif()

sdk/include/opentelemetry/sdk/common/empty_attributes.h

@@ -21,12 +21,12 @@ namespace sdk
  * with default attributes.
  */
 static const opentelemetry::common::KeyValueIterableView<
-    std::array<std::pair<std::string, int>, 0>> &
+    std::array<std::pair<std::string, int32_t>, 0>> &
 GetEmptyAttributes() noexcept
 {
-  static const std::array<std::pair<std::string, int>, 0> array{};
+  static const std::array<std::pair<std::string, int32_t>, 0> array{};
   static const opentelemetry::common::KeyValueIterableView<
-      std::array<std::pair<std::string, int>, 0>>
+      std::array<std::pair<std::string, int32_t>, 0>>
       kEmptyAttributes(array);
 
   return kEmptyAttributes;

sdk/include/opentelemetry/sdk/trace/span_data.h

@@ -257,7 +257,7 @@ class SpanData final : public Recordable
                 opentelemetry::common::SystemTimestamp timestamp =
                     opentelemetry::common::SystemTimestamp(std::chrono::system_clock::now()),
                 const opentelemetry::common::KeyValueIterable &attributes =
-                    opentelemetry::common::KeyValueIterableView<std::map<std::string, int>>(
+                    opentelemetry::common::KeyValueIterableView<std::map<std::string, int32_t>>(
                         {})) noexcept override
   {
     SpanDataEvent event(std::string(name), timestamp, attributes);

sdk/src/logs/BUILD

@@ -6,7 +6,6 @@ package(default_visibility = ["//visibility:public"])
 cc_library(
     name = "logs",
     srcs = glob(["**/*.cc"]),
-    hdrs = glob(["**/*.h"]),
     include_prefix = "src/logs",
     deps = [
         "//api",

sdk/src/metrics/BUILD

@@ -6,7 +6,6 @@ package(default_visibility = ["//visibility:public"])
 cc_library(
     name = "metrics",
     srcs = glob(["**/*.cc"]),
-    hdrs = glob(["**/*.h"]),
     include_prefix = "src/metrics",
     deps = [
         "//api",

sdk/src/resource/BUILD

@@ -6,7 +6,6 @@ package(default_visibility = ["//visibility:public"])
 cc_library(
     name = "resource",
     srcs = glob(["**/*.cc"]),
-    hdrs = glob(["**/*.h"]),
     include_prefix = "src/resource",
     deps = [
         "//api",