opentelemetry-cpp-contrib icon indicating copy to clipboard operation
opentelemetry-cpp-contrib copied to clipboard

Update build-matrix to include ubuntu 22.04

Open cedricziel opened this issue 3 years ago • 4 comments
trafficstars

cedricziel avatar Jun 09 '22 08:06 cedricziel

@cedricziel - It looks like CI is failing. Seems gRPC/boringssl build is breaking with the newer gcc compiler. otel-cpp is still not tested on the ubuntu 22.04. Looks like gRPC needs to be upgraded. fyi - @tobiasstadler is approver for Nginx.

lalitb avatar Jun 14 '22 06:06 lalitb

@cedricziel - It looks like CI is failing. Seems gRPC/boringssl build is breaking with the newer gcc compiler. otel-cpp is still not tested on the ubuntu 22.04. Looks like gRPC needs to be upgraded. fyi - @tobiasstadler is approver for Nginx.

Could we support ubuntu 22.04 in otel-cpp for now?

ThomsonTan avatar Jun 21 '22 16:06 ThomsonTan

Could we support ubuntu 22.04 in otel-cpp for now?

Yes, we should do it once gRPC gets upgraded. And also, would be good to add support matrix for OTLP-gRPC for various compiler/package versions.

lalitb avatar Jun 21 '22 18:06 lalitb

There are some warnings that will break the compiling when using new gcc and clang.I have some patches for the compiling problems in another project, it may be helpful.

  • BoringSSL: https://github.com/atframework/cmake-toolset/blob/main/ports/ssl/boringssl/boringssl-4fb158925f7753d80fb858cb0239dff893ef9f15.patch
  • gRPC 1.46.2: https://github.com/atframework/cmake-toolset/blob/main/ports/grpc/grpc-v1.46.2.patch
  • gRPC 1.43.2: https://github.com/atframework/cmake-toolset/blob/main/ports/grpc/grpc-v1.43.2.patch

owent avatar Jun 22 '22 02:06 owent