feat: Add OLTP Exporter Option for Protocol

bunkrur commented 1 year ago

Fixes #

I wanted to clean some stuff up and remove an alien commit from a friend. I feel like this is a happy compromise.

Some issues I ran into in the last attempt were

trying to get the config generator to respect http/protobuff within the config file.
trying to force enum convention in yaml
trying to do validation against the enum (its always gonna be valid unless we treat the input as a string)
trying to change the class structure - which would mean people would have to change their yaml configs due to code generation and I dont want to break that.

bunkrur commented 1 year ago

@tomkerkhove :)

tomkerkhove commented 1 year ago

The PR was only opened yesterday, please bear with me.

tomkerkhove commented 1 year ago

Let me know when you are done incorporating the feedback and I'll do another review - Thanks!

tomkerkhove commented 1 year ago

@bunkrur Any idea if you can complete the last comments or not?

bunkrur commented 1 year ago

I can, yeah just swallowed up recently. Will try by tomorrow

tomkerkhove commented 12 months ago

Let me know how you are doing @bunkrur (no pressure)

tomkerkhove / promitor