opentelemetry-python icon indicating copy to clipboard operation
opentelemetry-python copied to clipboard

Upgrade OTLP proto to v1.7. Add `event_name` to `LogRecord` API/SDK/PROTO

Open DylanRussell opened this issue 7 months ago • 1 comments

Description

Upgrade to proto 1.7. The file diffs make it very hard to tell what has actually changed. Wish there were a better way to see actual diffs.

From reading through https://github.com/open-telemetry/opentelemetry-proto/releases -- most of the changes are to profiling which we don't support AFAICT, so I don't think we have to worry about that at all.

The only other notable change is that the EventName field was added to the LogRecord proto... I opened up https://github.com/open-telemetry/opentelemetry-python/issues/4644 for that.

Type of change

Please delete options that are not relevant.

  • [x ] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Unit tests

Does This PR Require a Contrib Repo Change?

  • [ ] Yes. - Link to PR:
  • [x ] No.

Checklist:

  • [ x] Followed the style guidelines of this project
  • [ x] Changelogs have been updated
  • [ x] Unit tests have been added
  • [x ] Documentation has been updated

DylanRussell avatar Jun 18 '25 20:06 DylanRussell

Hmm we could do it separately.. Proto change doesn't require it. I'm heading out for weekend but can break it out on Monday if you think that's cleaner

DylanRussell avatar Jun 20 '25 21:06 DylanRussell

Ok I backed off the changes adding support for event name

DylanRussell avatar Jun 23 '25 13:06 DylanRussell

Once this is submitted I'll make a new PR for those

DylanRussell avatar Jun 23 '25 13:06 DylanRussell

Created https://github.com/open-telemetry/opentelemetry-python/pull/4652 to add the field to the LogRecord

DylanRussell avatar Jun 23 '25 13:06 DylanRussell