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

Context api actual methods #13

Merged
merged 45 commits into from
Jul 14, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
45 commits
Select commit Hold shift + click to select a range
2973baa
Add a minimal tracer example (#86)
Jun 8, 2020
58736f0
Set span start and end timestamps (#98)
Jun 9, 2020
9e5924c
Add Useful Resources and Example Build Instructions to CONTRIBUTING.m…
HudsonHumphries Jun 9, 2020
b395e1a
Update maintainers/approvers (#107)
reyang Jun 12, 2020
422b7a4
Migrate CI pipeline from CircleCI to GitHub Actions (#103)
Brandon-Kimberly Jun 15, 2020
668bb8b
Fix AtomicSharedPtr for C++20 (#114)
Jun 16, 2020
b472384
Remove Josh from the approvers (#116)
reyang Jun 16, 2020
49eee20
std::result_of removed in C++20 (#115)
maxgolov Jun 16, 2020
9048d45
Remove Tigran Najaryan from Approvers list (#120)
tigrannajaryan Jun 18, 2020
c86794e
Enhancements to PR #86 Add minimal tracer example (#94)
ankit-bhargava Jun 19, 2020
1a54e5d
Add code coverage using codecov.io (#121)
Brandon-Kimberly Jun 19, 2020
9b624b9
Support for attributes on spans (#117)
Jun 23, 2020
70d86d5
Add sampler header file (#118)
ziqizh Jun 23, 2020
6cb38fa
SamplingResult attributes fix (#126)
ziqizh Jun 24, 2020
ed0a36e
Always Off Sampler (#125)
nholbrook Jun 26, 2020
79b6141
Always on sampler (#122)
ziqizh Jun 29, 2020
aa79d9c
Add compare operators to nostd::string_view (#124)
maxgolov Jun 29, 2020
f399bf5
Start zPages README (#131)
jajanet Jun 30, 2020
5f4a464
Implement basic OTLP Recordable (#127)
nadiaciobanu Jul 2, 2020
7486a29
Update OpenTelemetry protobuf definitions (#140)
nadiaciobanu Jul 2, 2020
4128a69
added context header with dummy methods
satac2 Jul 6, 2020
cba6d5f
Build and CMake files
satac2 Jul 6, 2020
b2e876e
auto formatted
satac2 Jul 6, 2020
65f6639
Sampler tracer integration (#128)
ziqizh Jul 7, 2020
5bcf2d5
Add badges to README.md (#157)
huyan0 Jul 8, 2020
0cf5519
Add basic OTLP exporter (#152)
nadiaciobanu Jul 8, 2020
a7480c8
changed variable name from attributes to values
satac2 Jul 9, 2020
96cd32f
Update api/include/opentelemetry/context/context.h
satac2 Jul 9, 2020
d2f979b
Parent-or-else Sampler (#142)
ziqizh Jul 9, 2020
9aa4797
Don't move a constant AttributeValue (#141)
Jul 10, 2020
af0a946
Add benchmark tests for OTLP exporter (#163)
nadiaciobanu Jul 10, 2020
5f0634b
changed to follow the spec and added context_value type
satac2 Jul 13, 2020
5d77daf
Merge branch 'context_api_dummy_methods' of github.com:satac2/opentel…
satac2 Jul 13, 2020
68a1a70
removed tests for this PR
satac2 Jul 13, 2020
5e68574
removed line
satac2 Jul 13, 2020
16a492e
removed unnecessary include
satac2 Jul 13, 2020
497eb67
changed a function to pass by reference
satac2 Jul 13, 2020
479dead
formatted
satac2 Jul 13, 2020
2a78c48
Added tests.
satac2 Jul 13, 2020
f51748f
avoiding ABI compatibility issues
satac2 Jul 13, 2020
9902d5e
Move SpanContext to Separate File (#168)
nholbrook Jul 13, 2020
7d73219
Merge branch 'master' into context_api_dummy_methods
maxgolov Jul 13, 2020
3f85aa5
added throw capture
satac2 Jul 14, 2020
0ec83e8
Merge branch 'context_api_dummy_methods' of github.com:satac2/opentel…
satac2 Jul 14, 2020
21ea872
minor exception syntax error
satac2 Jul 14, 2020
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .bazelrc
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
# bazel configurations for running tests under sanitizers.
# Based on https://github.com/bazelment/trunk/blob/master/tools/bazel.rc

# Needed by gRPC to build on some platforms.
build --copt -DGRPC_BAZEL_BUILD

# --config=asan : Address Sanitizer.
common:asan --copt -fsanitize=address
common:asan --copt -DADDRESS_SANITIZER
Expand Down
2 changes: 1 addition & 1 deletion .bazelversion
Original file line number Diff line number Diff line change
@@ -1 +1 @@
3.2.0
3.3.1
28 changes: 28 additions & 0 deletions .github/.codecov.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
codecov:
require_ci_to_pass: yes
max_report_age: off

coverage:
precision: 2
round: down
range: "80...100"

parsers:
gcov:
branch_detection:
conditional: yes
loop: yes
method: no
macro: no

comment:
layout: "reach,diff,flags,tree"
behavior: default
require_changes: no

# Relative file path fixing.
# CI file paths must match Git file paths.
# This fix removes the "/home/runner/" prefix
# to coverage report file paths.
fixes:
- "/home/runner/::"
2 changes: 1 addition & 1 deletion .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@
# This file controls who is tagged for review for any given pull request.

# For anything not explicitly taken by someone else:
* @g-easy @jmacd @maxgolov @reyang @rnburn @tigrannajaryan
* @open-telemetry/cpp-approvers
200 changes: 200 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,200 @@
name: CI

on:
push:
branches: [ master ]
pull_request:
branches: [ master ]

jobs:
cmake_test:
name: CMake test
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- name: setup
run: |
sudo ./ci/setup_cmake.sh
sudo ./ci/setup_ci_environment.sh
- name: run tests
run: ./ci/do_ci.sh cmake.test

cmake_test_cxx20:
name: CMake C++20 test
runs-on: ubuntu-20.04
steps:
- uses: actions/checkout@v2
- name: setup
run: |
sudo ./ci/setup_ci_environment.sh
sudo ./ci/setup_cmake.sh
- name: run tests
run: ./ci/do_ci.sh cmake.c++20.test

plugin_test:
name: Plugin -> CMake
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- name: setup
run: |
sudo ./ci/setup_cmake.sh
sudo ./ci/setup_ci_environment.sh
- name: run tests
run: ./ci/do_ci.sh cmake.test_example_plugin

gcc_48_test:
name: Legacy Bazel
runs-on: ubuntu-18.04
steps:
- uses: actions/checkout@v2
- name: setup
run: |
sudo ./ci/setup_ci_environment.sh
sudo ./ci/install_bazelisk.sh
sudo ./ci/install_gcc48.sh
- name: run tests
run: ./ci/do_ci.sh bazel.legacy.test

bazel_test:
name: Bazel
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- name: setup
run: |
sudo ./ci/setup_cmake.sh
sudo ./ci/setup_ci_environment.sh
sudo ./ci/install_bazelisk.sh
- name: run tests
run: ./ci/do_ci.sh bazel.test

bazel_noexcept:
name: Bazel noexcept
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- name: setup
run: |
sudo ./ci/setup_cmake.sh
sudo ./ci/setup_ci_environment.sh
sudo ./ci/install_bazelisk.sh
- name: run tests
run: ./ci/do_ci.sh bazel.noexcept

bazel_asan:
name: Bazel asan config
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- name: setup
run: |
sudo ./ci/setup_cmake.sh
sudo ./ci/setup_ci_environment.sh
sudo ./ci/install_bazelisk.sh
- name: run tests
run: ./ci/do_ci.sh bazel.asan

bazel_tsan:
name: Bazel tsan config
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- name: setup
run: |
sudo ./ci/setup_cmake.sh
sudo ./ci/setup_ci_environment.sh
sudo ./ci/install_bazelisk.sh
- name: run tests
run: ./ci/do_ci.sh bazel.tsan

benchmark:
name: Benchmark
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- name: setup
run: |
sudo ./ci/setup_cmake.sh
sudo ./ci/setup_ci_environment.sh
sudo ./ci/install_bazelisk.sh
- name: run tests
run: |
env BENCHMARK_DIR=/benchmark
./ci/do_ci.sh benchmark
- name: Upload benchmark results
uses: actions/upload-artifact@v2
with:
name: benchmark_reports
path: /home/runner/benchmark

format:
name: Format
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- name: setup
run: sudo ./ci/install_format_tools.sh
- name: run tests
run: ./ci/do_ci.sh format

osx_test:
name: Bazel on MacOS
runs-on: macos-latest
steps:
- uses: actions/checkout@v2
- name: run tests
run: ./ci/do_ci.sh bazel.test

windows:
name: CMake -> exporter proto
runs-on: windows-2019
steps:
- uses: actions/checkout@v2
- name: setup
run: |
./ci/setup_windows_cmake.ps1
./ci/setup_windows_ci_environment.ps1
./ci/install_windows_protobuf.ps1
- name: run cmake test
run: ./ci/do_ci.ps1 cmake.test
- name: run otprotocol test
run: ./ci/do_ci.ps1 cmake.exporter.otprotocol.test

windows_bazel:
name: Bazel Windows
runs-on: windows-2019
steps:
- uses: actions/checkout@v2
- name: setup
run: ./ci/install_windows_bazelisk.ps1
- name: run tests
run: ./ci/do_ci.ps1 bazel.build

windows_plugin_test:
name: Plugin -> CMake Windows
runs-on: windows-2019
steps:
- uses: actions/checkout@v2
- name: setup
run: |
./ci/setup_windows_cmake.ps1
./ci/setup_windows_ci_environment.ps1
- name: run tests
run: ./ci/do_ci.ps1 cmake.test_example_plugin

code_coverage:
name: Code coverage
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- name: setup
run: |
sudo ./ci/setup_cmake.sh
sudo ./ci/setup_ci_environment.sh
- name: run tests and generate report
run: ./ci/do_ci.sh code.coverage
- name: upload report
uses: codecov/codecov-action@v1
with:
file: /home/runner/build/coverage.info
11 changes: 10 additions & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,9 @@ cmake_policy(SET CMP0057 NEW)

project(opentelemetry-cpp)

set(CMAKE_CXX_STANDARD 11)
if(NOT DEFINED CMAKE_CXX_STANDARD)
set(CMAKE_CXX_STANDARD 11)
endif()

option(WITH_OTPROTOCOL
"Whether to include the OpenTelemetry Protocol in the SDK" OFF)
Expand All @@ -20,6 +22,13 @@ include(CTest)

find_package(Threads)

if(CMAKE_CXX_COMPILER_ID STREQUAL "MSVC")
# Options for Visual C++ compiler: /Zc:__cplusplus - report an updated value
# for recent C++ language standards. Without this option MSVC returns the
# value of __cplusplus="199711L"
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} /Zc:__cplusplus")
endif()

if(WITH_PROTOBUF)
set(protobuf_MODULE_COMPATIBLE ON)
find_package(Protobuf CONFIG NAMES protobuf)
Expand Down
Loading