Merge branch 'main' into main

Author: RichardChukwu

exporters/otlp/include/opentelemetry/exporters/otlp/otlp_log_recordable.h

@@ -76,10 +76,7 @@ class OtlpLogRecordable final : public opentelemetry::sdk::logs::Recordable
    * @param id the event Id to set
    * @param name  the event name to set
    */
-  void SetEventId(int64_t /* id */, nostd::string_view /* name */) noexcept override
-  {
-    // TODO: export Event Id to OTLP
-  }
+  void SetEventId(int64_t /* id */, nostd::string_view event_name) noexcept override;
 
   /**
    * Set the trace id for this log.

exporters/otlp/src/otlp_log_recordable.cc

@@ -197,6 +197,11 @@ void OtlpLogRecordable::SetBody(const opentelemetry::common::AttributeValue &mes
   OtlpPopulateAttributeUtils::PopulateAnyValue(proto_record_.mutable_body(), message);
 }
 
+void OtlpLogRecordable::SetEventId(int64_t /* id */, nostd::string_view event_name) noexcept
+{
+  proto_record_.set_event_name(event_name.data(), event_name.size());
+}
+
 void OtlpLogRecordable::SetTraceId(const opentelemetry::trace::TraceId &trace_id) noexcept
 {
   if (trace_id.IsValid())

exporters/otlp/test/otlp_log_recordable_test.cc

@@ -174,6 +174,16 @@ TEST(OtlpLogRecordable, SetInstrumentationScope)
   EXPECT_EQ(&rec.GetInstrumentationScope(), inst_lib.get());
 }
 
+TEST(OtlpLogRecordable, SetEventName)
+{
+  OtlpLogRecordable rec;
+
+  nostd::string_view event_name = "Test Event";
+  rec.SetEventId(0, event_name);
+
+  EXPECT_EQ(rec.log_record().event_name(), event_name);
+}
+
 /**
  * AttributeValue can contain different int types, such as int, int64_t,
  * unsigned int, and uint64_t. To avoid writing test cases for each, we can

sdk/src/metrics/instrument_metadata_validator.cc

@@ -9,18 +9,22 @@
 
 #if OPENTELEMETRY_HAVE_WORKING_REGEX
 #  include <regex>
+#else
+#  include <algorithm>
 #endif
 
 OPENTELEMETRY_BEGIN_NAMESPACE
 namespace sdk
 {
 namespace metrics
 {
+#if OPENTELEMETRY_HAVE_WORKING_REGEX
 // instrument-name = ALPHA 0*254 ("_" / "." / "-" / "/" / ALPHA / DIGIT)
 const std::string kInstrumentNamePattern = "[a-zA-Z][-_./a-zA-Z0-9]{0,254}";
 //
 const std::string kInstrumentUnitPattern = "[\x01-\x7F]{0,63}";
 // instrument-unit = It can have a maximum length of 63 ASCII chars
+#endif
 
 InstrumentMetaDataValidator::InstrumentMetaDataValidator()
 #if OPENTELEMETRY_HAVE_WORKING_REGEX