feat(render): add parsed messages and decode code (#458)

Rename RustWireMessageGenerator.xtend -> RustMessageGenerator.xtend.

Add code generation for parsed_message.rs that has a struct for each message with fields, including String for the strings sent in the message. Add a decode() method to instantiate each message struct from a buffer containing a message of that type.

Also fix warning in Cargo Generator due to having profile.release in
internal crates. We should have removed it when we started compiling from
the root workspace, since that config belongs there.

(cherry picked from commit a976cab)

Author: yonch

Cargo.toml

@@ -1,4 +1,5 @@
 [workspace]
+resolver = "2"
 members = [
   "crates/kernel-collector-sys",
   "crates/kernel-collector-bin",

cmake/render.cmake

@@ -88,6 +88,7 @@ function(render_compile INPUT_DIR)
         "${OUTPUT_DIR}/${PACKAGE}/${APP}/bpf.h"
         # Rust per-app crate files live under src/
         "${OUTPUT_DIR}/${PACKAGE}/${APP}/src/wire_messages.rs"
+        "${OUTPUT_DIR}/${PACKAGE}/${APP}/src/parsed_message.rs"
         "${OUTPUT_DIR}/${PACKAGE}/${APP}/src/encoder.rs"
         "${OUTPUT_DIR}/${PACKAGE}/${APP}/src/hash.rs"
         "${OUTPUT_DIR}/${PACKAGE}/${APP}/Cargo.toml"
@@ -283,6 +284,7 @@ function(render_compile INPUT_DIR)
       COMMAND ${CMAKE_COMMAND} -E copy_if_different "${_src_dir}/src/lib.rs" "${_dest_dir}/src/lib.rs"
       COMMAND ${CMAKE_COMMAND} -E copy_if_different "${_src_dir}/src/encoder.rs" "${_dest_dir}/src/encoder.rs"
       COMMAND ${CMAKE_COMMAND} -E copy_if_different "${_src_dir}/src/wire_messages.rs" "${_dest_dir}/src/wire_messages.rs"
+      COMMAND ${CMAKE_COMMAND} -E copy_if_different "${_src_dir}/src/parsed_message.rs" "${_dest_dir}/src/parsed_message.rs"
       COMMAND ${CMAKE_COMMAND} -E copy_if_different "${_src_dir}/src/hash.rs" "${_dest_dir}/src/hash.rs"
       DEPENDS render_compile_${PACKAGE}
       VERBATIM

renderc/CMakeLists.txt

@@ -101,7 +101,7 @@ add_custom_command(
     ./io.opentelemetry.render/src/io/opentelemetry/render/generator/TransformBuilderGenerator.xtend
     ./io.opentelemetry.render/src/io/opentelemetry/render/generator/BpfGenerator.xtend
     ./io.opentelemetry.render/src/io/opentelemetry/render/generator/RustEncoderGenerator.xtend
-    ./io.opentelemetry.render/src/io/opentelemetry/render/generator/RustWireMessageGenerator.xtend
+    ./io.opentelemetry.render/src/io/opentelemetry/render/generator/RustMessageGenerator.xtend
     ./io.opentelemetry.render/src/io/opentelemetry/render/generator/RustCargoGenerator.xtend
     ./io.opentelemetry.render/src/io/opentelemetry/render/generator/WriterGenerator.xtend
     ./io.opentelemetry.render/src/io/opentelemetry/render/generator/SpanGenerator.xtend

renderc/io.opentelemetry.render/src/io/opentelemetry/render/generator/AppGenerator.xtend

@@ -14,7 +14,7 @@ class AppGenerator {
   SpanGenerator spanGenerator = new SpanGenerator()
   HashGenerator hashGenerator = new HashGenerator()
   WriterGenerator writerGenerator = new WriterGenerator()
-  RustWireMessageGenerator rustWireMessageGenerator = new RustWireMessageGenerator()
+  RustMessageGenerator rustMessageGenerator = new RustMessageGenerator()
   RustEncoderGenerator rustEncoderGenerator = new RustEncoderGenerator()
   RustCargoGenerator rustCargoGenerator = new RustCargoGenerator()
   MessageGenerator messageGenerator = new MessageGenerator()
@@ -27,7 +27,7 @@ class AppGenerator {
     spanGenerator.doGenerate(app, fsa)
     hashGenerator.doGenerate(app, fsa)
     writerGenerator.doGenerate(app, fsa)
-    rustWireMessageGenerator.doGenerate(app, fsa)
+    rustMessageGenerator.doGenerate(app, fsa)
     rustEncoderGenerator.doGenerate(app, fsa)
     rustCargoGenerator.doGenerateApp(app, fsa)
     messageGenerator.doGenerate(app, fsa)

renderc/io.opentelemetry.render/src/io/opentelemetry/render/generator/RustCargoGenerator.xtend

@@ -56,11 +56,6 @@ class RustCargoGenerator {
     # - staticlib: keeps support for direct C/C++ linking where needed.
     crate-type = ["rlib", "staticlib"]
 
-    [profile.release]
-    opt-level = 3
-    lto = true
-    codegen-units = 1
-
     [dependencies]
     render_parser = { workspace = true }
     '''
@@ -80,20 +75,14 @@ class RustCargoGenerator {
         pub len: u16,
     }
 
-    // Include generated modules from src/
+    // Modules use the standard Rust module system; files live under src/
     #[allow(dead_code)]
-    pub mod wire_messages {
-        include!(concat!(env!("CARGO_MANIFEST_DIR"), "/src/wire_messages.rs"));
-    }
-
-    pub mod encoder {
-        include!(concat!(env!("CARGO_MANIFEST_DIR"), "/src/encoder.rs"));
-    }
-
+    pub mod wire_messages;
     #[allow(dead_code)]
-    pub mod hash {
-        include!(concat!(env!("CARGO_MANIFEST_DIR"), "/src/hash.rs"));
-    }
+    pub mod parsed_message;
+    pub mod encoder;
+    #[allow(dead_code)]
+    pub mod hash;
     '''
   }
 
@@ -110,11 +99,6 @@ class RustCargoGenerator {
     # link per-app crates directly via rlib dependencies.
     crate-type = ["staticlib"]
 
-    [profile.release]
-    opt-level = 3
-    lto = true
-    codegen-units = 1
-
     [dependencies]
     «FOR a : apps»encoder_«pkg»_«a.name» = { path = "«a.name»" }
     «ENDFOR»