profiles: Make mapping in Profile optional

As commented in [0] and discussed in the OTel Profiling SIG meeting, there are situations where a main binary for a Profile can not be identified. For these cases mark the field optional. [0]: #534 (comment) Signed-off-by: Florian Lehner <[email protected]>
open-telemetry · May 3, 2024 · 3b80dc6 · 3b80dc6
1 parent b3060d2
commit 3b80dc6
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/opentelemetry/proto/profiles/v1experimental/pprofextended.proto b/opentelemetry/proto/profiles/v1experimental/pprofextended.proto
@@ -77,6 +77,8 @@ message Profile {
   repeated Sample sample = 2;
   // Mapping from address ranges to the image/binary/library mapped
   // into that address range.  mapping[0] will be the main binary.
+  // If multiple binaries contribute to the Profile and no main
+  // binary can be identified, mapping is optional.
   repeated Mapping mapping = 3;
   // Locations referenced by samples via location_indices.
   repeated Location location = 4;