From 0b0b4f2710589b5d395f2fbc5e7992dbe295de28 Mon Sep 17 00:00:00 2001 From: Yash Tibrewal Date: Tue, 13 May 2025 16:17:27 +0000 Subject: [PATCH 1/2] L123: Remove C++ OpenCensus and dependent APIs --- L123-remove-cpp-opencensus-apis.md | 44 ++++++++++++++++++++++++++++++ 1 file changed, 44 insertions(+) create mode 100644 L123-remove-cpp-opencensus-apis.md diff --git a/L123-remove-cpp-opencensus-apis.md b/L123-remove-cpp-opencensus-apis.md new file mode 100644 index 000000000..a3a8ec25c --- /dev/null +++ b/L123-remove-cpp-opencensus-apis.md @@ -0,0 +1,44 @@ +## L123: Deprecate and Remove C++ OpenCensus and dependent APIs + +* Author(s): Yash Tibrewal (@yashykt) +* Approver: Mark Roth (@markdroth) , Esun Kim (@veblush) +* Status: Draft +* Implemented in: C++ +* Last updated: May 12, 2025 +* Discussion at: (filled after thread exists) + +## Abstract + +Remove gRPC C++ OpenCensus and dependent APIs. + +## Background + +OpenCensus was +[sunsetted in May, 2023](https://opentelemetry.io/blog/2023/sunsetting-opencensus/), +and the +[C++ OpenCensus repository](https://github.com/census-instrumentation/opencensus-cpp) +has been archived (no longer maintained). + +Furthermore, the last available commit of OpenCensus C++ does not build with the +[latest candidate release of the Abseil library](https://github.com/abseil/abseil-cpp/releases/tag/20250512.rc1) +(OpenCensus C++ relied on internal headers and methods from Abseil.) + +## Proposal + +Deprecate the +[C++ OpenCensus](https://github.com/grpc/grpc/blob/v1.72.x/include/grpcpp/opencensus.h) +and dependent +[gRPC C++ GCP Observability](https://github.com/grpc/grpc/blob/master/include/grpcpp/ext/gcp_observability.h) +APIs and remove them in a subsequent release. Users still utilizing OpenCensus +APIs should migrate to OpenTelemetry. + +## Rationale + +Given gRPC C++'s dependency on Abseil, maintaining OpenCensus support in future +versions is not feasible. Deleting these APIs will allow gRPC C++ users to +upgrade the version of the Abseil library used. + +## Implementation + +https://github.com/grpc/grpc/pull/39554 marks these APIs as deprecated. A +subsequent release will delete these APIs. From 17e311e91fdf2b0c1f29949551a19ee34749692b Mon Sep 17 00:00:00 2001 From: Yash Tibrewal Date: Tue, 13 May 2025 17:57:18 +0000 Subject: [PATCH 2/2] Add google group discussion thread --- L123-remove-cpp-opencensus-apis.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/L123-remove-cpp-opencensus-apis.md b/L123-remove-cpp-opencensus-apis.md index a3a8ec25c..91e542011 100644 --- a/L123-remove-cpp-opencensus-apis.md +++ b/L123-remove-cpp-opencensus-apis.md @@ -2,10 +2,10 @@ * Author(s): Yash Tibrewal (@yashykt) * Approver: Mark Roth (@markdroth) , Esun Kim (@veblush) -* Status: Draft +* Status: In Review * Implemented in: C++ -* Last updated: May 12, 2025 -* Discussion at: (filled after thread exists) +* Last updated: May 13, 2025 +* Discussion at: https://groups.google.com/g/grpc-io/c/ASVIZhcZgl0 ## Abstract