Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release v1.0.0 #980

Merged
merged 12 commits into from
Sep 20, 2021
Merged

Release v1.0.0 #980

merged 12 commits into from
Sep 20, 2021

Conversation

lalitb
Copy link
Member

@lalitb lalitb commented Sep 17, 2021

Fixes # (issue)

TC Review : #946

Changes

( From the CHANGELOG.md )

API

  • Document DefaultSpan, remove DefaultTracer (#959)
  • Separate baggage<->Context api from Baggage Propagator (#963)
  • Remove unused public API to_span_ptr (#964)
  • 💥 Make span context management public (#967)
  • Support determining parent span from Context while creating new Span (#969)
  • 💥 Traces: Add note on experimental semantic convention implementation, prefix semantics headers with experimental tag (#970)

SDK

  • Cleanup GetEnvironmentVariable and remove unused variable under NO_GETENV (#976)
  • 💥 Resources: Add note on experimental semantic convention implementation, prefix semantics headers with experimental tag (#970)

OTLP Exporter

  • 🐛 Ignore status description if status code is not Error (#962)
  • 💥 Make Otlp exporter configuration environment variables specs-compliant (#974)

Zipkin Exporter

  • 🐛 Don't set parentId in case parentId is empty (#943)
  • 🚀 Extend zipkin exporter with ability to provide headers (#951)

DOCS

  • 📖 Add Getting started documentation for SDK: (#942)
  • 📖 Remove unnecessary spaces and spelling of gRPC in README (#965)

BUILD

  • Disable bazel build for gcc 4.8, upgrade versions for grpc(v1.39.1) and bazel(4.2.0), document bazel support (#953)
  • Move CMake config template to cmake folder (#958)
  • Enable CMake to search the new package variable <PackageName>_ROOT (#975)
  • 🐛 Do not override CMAKE_CXX_STANDARD (#977)

🚧 Experiemental Features (Will change in future)

  • Semantic Conventions for traces - As the specs is still experimental, the implementation will change in future.
  • Semantic Convention for resource - As the specs is still experimental, the implementation will change in future.
  • Logs and Metrics API & SDK - These are not compliant, and are behind feature flag. Not recommended to use for now.
  • 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 September 17, 2021 00:04
CHANGELOG.md Outdated Show resolved Hide resolved
Co-authored-by: Reiley Yang <[email protected]>
CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Member

@reyang reyang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥇 💯 👍 🎆

@codecov
Copy link

codecov bot commented Sep 17, 2021

Codecov Report

Merging #980 (0b09d7c) into main (43d8fe4) will not change coverage.
The diff coverage is n/a.

❗ Current head 0b09d7c differs from pull request most recent head 7eee01b. Consider uploading reports for the commit 7eee01b to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##             main     #980   +/-   ##
=======================================
  Coverage   95.38%   95.38%           
=======================================
  Files         161      161           
  Lines        6817     6817           
=======================================
  Hits         6502     6502           
  Misses        315      315           
Impacted Files Coverage Δ
ext/test/zpages/tracez_data_aggregator_test.cc 96.40% <ø> (ø)

@lalitb lalitb merged commit 74d4be4 into open-telemetry:main Sep 20, 2021
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.

4 participants