Lalit Kumar Bhasin

Results 425 comments of Lalit Kumar Bhasin
trafficstars

@germankuber We have an open issue #1378 for the problem. There is no solution as of now, though one of the important issues to be resolved. There is some work...

> Why do these exporters get to stay in the core, and the others don't? That seems fairly arbitrary. It's coming from the guidelines - https://github.com/open-telemetry/opentelemetry-specification/blob/v1.26.0/specification/library-guidelines.md#requirements ```markup The SDK implementation...

A quick look into the ubuntu24.04 [docker image ](https://hub.docker.com/_/ubuntu)(of dev-brach) shows below versions ```console root@3d32d77d0cd2:/# cat /etc/os-release | grep VERSION VERSION_ID="24.04" VERSION="24.04 LTS (Noble Numbat)" VERSION_CODENAME=noble root@3d32d77d0cd2:/# apt-cache search clang-format...

> Of course, by the time the runner upgrades to 24.04, we can upgrade clang-format as well if we want to. ubuntu 24.04 is planned to be released on 25th...

Probably we can wait till 25th April to see if clang-format-15 is available in 24.04. If it is available, our decision would be easy to go with the upgrade immediately...

@desmart18 - Is this issue still valid, as private modules repo is already updated

@anod It would be good to upgrade the android build toolset we are using. I am not Java/Android expert, will this anyway change the minimum sdk version we correctly support,...

> it might require to upgrade cmake, is there limitation for cmake version capabilities. Which cmake version to upgrade to ? Ideally it would be good to support 3.1.0 as...

> The failed job report error when compiling upb, which seems not a problem of this PR? Looks like we need to upgrade [com_google_absl](https://github.com/open-telemetry/opentelemetry-cpp/blob/054b0dc207c1f58e290d78cdaac5f314bc328b31/bazel/repository.bzl#L66). Probably something changed in `macos-latest` (clang...

> Curious on what triggered the MacOS build warning. My guess is that XCode update on mac-os14 (which is macos-latest) triggered clang to get updated too. https://github.com/actions/runner-images/pull/9726