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

Add CI jobs for new and deprecated metrics #1531

Merged
merged 14 commits into from
Aug 12, 2022

Conversation

esigo
Copy link
Member

@esigo esigo commented Jul 30, 2022

Fixes #1494 (issue)

Changes

makes all jobs testing the new metrics and adds bazel and cmake, gcc 4.8 jobs for testing deprecated metrics.

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

@esigo esigo requested a review from a team July 30, 2022 15:41
@codecov
Copy link

codecov bot commented Jul 30, 2022

Codecov Report

Merging #1531 (c776b25) into main (1b9aa31) will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1531      +/-   ##
==========================================
- Coverage   83.38%   83.36%   -0.02%     
==========================================
  Files         156      156              
  Lines        4975     4975              
==========================================
- Hits         4148     4147       -1     
- Misses        827      828       +1     
Impacted Files Coverage Δ
...include/opentelemetry/sdk/common/circular_buffer.h 98.00% <0.00%> (-2.00%) ⬇️

ci/do_ci.sh Show resolved Hide resolved
Copy link
Member

@lalitb lalitb left a comment

Choose a reason for hiding this comment

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

LGTM - few nit comments. Thanks for fixing the memory issues.

.github/workflows/ci.yml Outdated Show resolved Hide resolved
@@ -165,7 +181,27 @@ jobs:
sudo ./ci/install_bazelisk.sh
- name: run tests
run: ./ci/do_ci.sh bazel.test

bazel_deprecated_metrics_test:
Copy link
Member

Choose a reason for hiding this comment

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

nit - add newline before and after this test.

Copy link
Member Author

Choose a reason for hiding this comment

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

thanks, done

esigo and others added 3 commits August 12, 2022 17:22
Co-authored-by: Lalit Kumar Bhasin <lalit_fin@yahoo.com>
@esigo esigo enabled auto-merge (squash) August 12, 2022 15:34
@esigo esigo merged commit d04e1ce into open-telemetry:main Aug 12, 2022
@esigo esigo deleted the new-metrics-CI-jobs branch August 12, 2022 16:31
yxue pushed a commit to yxue/opentelemetry-cpp that referenced this pull request Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] Add ci jobs without ENABLE_METRICS_PREVIEW (bazel) and WITH_METRICS_PREVIEW (cmake)
3 participants