opentelemetry-cpp
opentelemetry-cpp copied to clipboard
add CI test with gRPC latest release
Fixes #1607 (issue)
Changes
Adds a new CI which tests otlp exporters using the latest gRPC release.
For significant contributions please make sure you have completed the following items:
- [ ]
CHANGELOG.mdupdated for non-trivial changes - [ ] Unit tests have been added
- [ ] Changes in public API reviewed
Codecov Report
Merging #1608 (b2e82b6) into main (5c180a1) will not change coverage. The diff coverage is
n/a.
Thanks for the PR. A couple of nit comments
- Instead of adding a new CI, should we use the latest gRPC for all cmake builds on ubuntu-latest/ubuntu-20.04/ubuntu-20.10. These virtual machine uses g++-9.3 as the default compiler, which builds in c++-14 mode.
- Can we try using the latest gRPC for bazel builds too? Not sure if it will work, as it failed earlier in #1462. We can do it in separate PR if it takes time
- Should we document the gRPC dependency here - https://github.com/open-telemetry/opentelemetry-cpp#supported-development-platforms. We already do it for gcc4.8 for gRPC here.
Thanks for the PR. A couple of nit comments
- Instead of adding a new CI, should we use the latest gRPC for all cmake builds on ubuntu-latest/ubuntu-20.04/ubuntu-20.10. These virtual machine uses g++-9.3 as the default compiler, which builds in c++-14 mode.
- Can we try using the latest gRPC for bazel builds too? Not sure if it will work, as it failed earlier in Upgrade to gRPC 1.46.3 (#1459) #1462. We can do it in separate PR if it takes time
- Should we document the gRPC dependency here - open-telemetry/opentelemetry-cpp#supported-development-platforms. We already do it for gcc4.8 for gRPC here.
One more item is for Windows build, could we also make the same update there?
One more item is for Windows build, could we also make the same update there?
That can be challenging it seems. We use vcpkg to install windows dependencies as here and latest gRPC version in vcpkg is v1.41.0.
@lalitb Windows CMake CI doesn't use gRPC. We use C++14 for building gRPC itself only so I believe we don't need to change our documentation. I would keep this PR as it is.
I raised another PR to bump vcpkg #1633. I created another issue for bazel #1634.
meanwhile gRPC 1.49.1 released.