Skip to content

Modules: add opentelemetry-proto module with protobuf definitions - #47

Open
inge4pres wants to merge 5 commits into
mainfrom
modules/proto
Open

Modules: add opentelemetry-proto module with protobuf definitions#47
inge4pres wants to merge 5 commits into
mainfrom
modules/proto

Conversation

@inge4pres

@inge4pres inge4pres commented Jul 24, 2026

Copy link
Copy Markdown
Member

Reason for this PR

Closes #19

Details

We add a new folder in the monorepo, hosting the module with the protobuf-generated code.
Same dependencies and buld steps are ported to the main build.zig.
A new build folder component is made for the protobuf module.

Documentation is updated to reflect the multiple modules structure.

inge4pres and others added 5 commits July 24, 2026 15:45
Signed-off-by: inge4pres <fgualazzi@gmail.com>
Signed-off-by: inge4pres <fgualazzi@gmail.com>
Signed-off-by: inge4pres <fgualazzi@gmail.com>
Signed-off-by: inge4pres <francesco.gualazzi@stealthium.io>
Signed-off-by: inge4pres <fgualazzi@gmail.com>
@inge4pres
inge4pres requested a review from a team as a code owner July 24, 2026 17:41
@inge4pres

inge4pres commented Jul 24, 2026

Copy link
Copy Markdown
Member Author

For testing this PR you will need to sync the submodule first.

git submodule update --init opentelemetry-proto/proto-src

and then you can run the build steps

zig build proto-generate
zig build proto-test

@agagniere

Copy link
Copy Markdown
Member

Why use a git submodule here ? we could add it as a lazy dependency in the build.zig.zon

@inge4pres

Copy link
Copy Markdown
Member Author

Why use a git submodule here ? we could add it as a lazy dependency in the build.zig.zon

@agagniere good question 👍🏼
It's what has been used in the previous repo because we wanted to be able to pin it to tagged versions.

Can we do the same with lazy dependencies? I genuinely haven't used them.
Does a lazy dep require a build.zig/build.zig.zon in the remote repo?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add the opentelemetry-proto module to the repo

2 participants