Skip to content

Commit 4283fd4

Browse files
authored
Merge pull request #716 from open-telemetry/main
merged from upstream
2 parents f627643 + be6cb67 commit 4283fd4

File tree

4 files changed

+16
-15
lines changed

4 files changed

+16
-15
lines changed

CHANGELOG.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,9 @@ Increment the:
3636
* [BUILD] Upgrade to rapidyaml 0.10.0
3737
[#3906](https://github.com/open-telemetry/opentelemetry-cpp/pull/3906)
3838

39+
* [EXAMPLES] Deprecated semantic conventions used
40+
[#3905](https://github.com/open-telemetry/opentelemetry-cpp/pull/3905)
41+
3942
Important changes:
4043

4144
* [BUILD] Revisit EventLogger deprecation

examples/common/metrics_foo_library/foo_library.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,7 @@ void foo_library::semconv_observable_counter_example()
169169
auto provider = metrics_api::Provider::GetMeterProvider();
170170
opentelemetry::nostd::shared_ptr<metrics_api::Meter> meter = provider->GetMeter("demo", "1.2.0");
171171
double_observable_counter =
172-
opentelemetry::semconv::system::CreateAsyncDoubleMetricSystemCpuTime(meter.get());
172+
opentelemetry::semconv::system::CreateAsyncDoubleMetricSystemDiskIo(meter.get());
173173
double_observable_counter->AddCallback(MeasurementFetcher::Fetcher, nullptr);
174174
for (uint32_t i = 0; i < 20; ++i)
175175
{

examples/grpc/client.cc

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -63,14 +63,13 @@ class GreeterClient
6363
options.kind = SpanKind::kClient;
6464

6565
std::string span_name = "GreeterClient/Greet";
66-
auto span =
67-
get_tracer("grpc")->StartSpan(span_name,
68-
{{semconv::rpc::kRpcSystem, "grpc"},
69-
{semconv::rpc::kRpcService, "grpc-example.GreetService"},
70-
{semconv::rpc::kRpcMethod, "Greet"},
71-
{semconv::network::kNetworkPeerAddress, ip},
72-
{semconv::network::kNetworkPeerPort, port}},
73-
options);
66+
auto span = get_tracer("grpc")->StartSpan(
67+
span_name,
68+
{{semconv::rpc::kRpcSystemName, "grpc"},
69+
{semconv::rpc::kRpcMethod, "grpc-example.GreetService/Greet"},
70+
{semconv::network::kNetworkPeerAddress, ip},
71+
{semconv::network::kNetworkPeerPort, port}},
72+
options);
7473

7574
auto scope = get_tracer("grpc-client")->WithActiveSpan(span);
7675

@@ -85,7 +84,7 @@ class GreeterClient
8584
if (status.ok())
8685
{
8786
span->SetStatus(StatusCode::kOk);
88-
span->SetAttribute(semconv::rpc::kRpcGrpcStatusCode, status.error_code());
87+
span->SetAttribute(semconv::rpc::kRpcResponseStatusCode, status.error_message());
8988
// Make sure to end your spans!
9089
span->End();
9190
return response.response();
@@ -94,7 +93,7 @@ class GreeterClient
9493
{
9594
std::cout << status.error_code() << ": " << status.error_message() << '\n';
9695
span->SetStatus(StatusCode::kError);
97-
span->SetAttribute(semconv::rpc::kRpcGrpcStatusCode, status.error_code());
96+
span->SetAttribute(semconv::rpc::kRpcResponseStatusCode, status.error_message());
9897
// Make sure to end your spans!
9998
span->End();
10099
return "RPC failed";

examples/grpc/server.cc

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -78,10 +78,9 @@ class GreeterServer final : public Greeter::Service
7878

7979
std::string span_name = "GreeterService/Greet";
8080
auto span = get_tracer("grpc")->StartSpan(span_name,
81-
{{semconv::rpc::kRpcSystem, "grpc"},
82-
{semconv::rpc::kRpcService, "GreeterService"},
83-
{semconv::rpc::kRpcMethod, "Greet"},
84-
{semconv::rpc::kRpcGrpcStatusCode, 0}},
81+
{{semconv::rpc::kRpcSystemName, "grpc"},
82+
{semconv::rpc::kRpcMethod, "GreeterService/Greet"},
83+
{semconv::rpc::kRpcResponseStatusCode, "OK"}},
8584
options);
8685
auto scope = get_tracer("grpc")->WithActiveSpan(span);
8786

0 commit comments

Comments
 (0)