Skip to content

Add preview feature-flag/macro for logs#807

Merged
lalitb merged 10 commits intoopen-telemetry:mainfrom
lalitb:enable-logs-ff
May 29, 2021
Merged

Add preview feature-flag/macro for logs#807
lalitb merged 10 commits intoopen-telemetry:mainfrom
lalitb:enable-logs-ff

Conversation

@lalitb
Copy link
Copy Markdown
Member

@lalitb lalitb commented May 28, 2021

Changes

Similar to #745, The otel-cpp logs implementation is in preview, so it needs to be behind the feature flag as we are planning for 1.0.0 RC ( 1.0.0 release is only for tracing signal, not metrics and logs).

  • As per the feature flag policy defined in Versioning.md - This PR wraps all the logs code (api, sdk, tests, exporter) under ENABLE_LOGS_PREVIEW hashdef.
#ifdef ENABLE_LOGS_PREVIEW
..
..
#endif 
  • Included cmake option WITH_LOGS_PREVIEW, default switched off. This is enabled during CI to ensure current log implementation doesn't break with any new changes.

The changes look huge as the clang-format does the indentation of preprocessor directives within this wrapper.

For significant contributions please make sure you have completed the following items:

  • CHANGELOG.md updated for non-trivial changes
  • Unit tests have been added
  • Changes in public API reviewed

@lalitb lalitb requested a review from a team May 28, 2021 11:09
@codecov
Copy link
Copy Markdown

codecov Bot commented May 28, 2021

Codecov Report

Merging #807 (f36d751) into main (12e56f9) will increase coverage by 0.18%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #807      +/-   ##
==========================================
+ Coverage   96.01%   96.19%   +0.18%     
==========================================
  Files         176      153      -23     
  Lines        7183     6444     -739     
==========================================
- Hits         6896     6198     -698     
+ Misses        287      246      -41     
Impacted Files Coverage Δ
api/include/opentelemetry/trace/noop.h 84.62% <0.00%> (-7.69%) ⬇️
...clude/opentelemetry/sdk/common/atomic_shared_ptr.h

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.

3 participants