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

Elastic Search exporter follow ECS guidelines

Open ShadowMaxLeb opened this issue 1 year ago • 2 comments

Fixes #3091

Changes

Try to follow as much as possible the ECS guidelines and the ECS Logs reference

This impacts fields:

  • timestamp: now set to @timestamp with a Date string
  • body: now set to message
  • name: now set to log.logger
  • severity: now set to log.level
  • attributes: not under the attributes key anymore

ShadowMaxLeb avatar Oct 18 '24 09:10 ShadowMaxLeb

Deploy Preview for opentelemetry-cpp-api-docs canceled.

Name Link
Latest commit fa03dc36dc6de93945f0e2098f0099881f7e3b8a
Latest deploy log https://app.netlify.com/sites/opentelemetry-cpp-api-docs/deploys/671c741ddfaa200008cfc0c4

netlify[bot] avatar Oct 18 '24 09:10 netlify[bot]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.91%. Comparing base (497eaf4) to head (fa03dc3). Report is 149 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3107      +/-   ##
==========================================
+ Coverage   87.12%   87.91%   +0.79%     
==========================================
  Files         200      195       -5     
  Lines        6109     6133      +24     
==========================================
+ Hits         5322     5391      +69     
+ Misses        787      742      -45     

see 98 files with indirect coverage changes

codecov[bot] avatar Oct 18 '24 09:10 codecov[bot]

Thanks for the fix.

@ShadowMaxLeb Please merge main into this PR, to resolve "This branch is out-of-date with the base branch".

marcalff avatar Oct 22 '24 16:10 marcalff

Thanks for the fix.

@ShadowMaxLeb Please merge main into this PR, to resolve "This branch is out-of-date with the base branch".

I will try to update it next week as I cannot do it right now.

MaxLeb avatar Oct 23 '24 19:10 MaxLeb

@marcalff , How can I retry the failed job as it failed because it was lacking a runner ?

ShadowMaxLeb avatar Oct 27 '24 12:10 ShadowMaxLeb