[o11y] Fix some attributes to match Otel Semantic Conventions (#45753)

* [o11y] Fix some attributes to match Otel Semantic Conventions

* db.cosmosdb.sub_status_code is now an int
  * Added `azure.cosmosdb.response.sub_status_code` to reflect this too
    since it's the non-deprecated label
* db.cosmosdb.request_charge is now a double
  * Added `azure.cosmosdb.operation.request_charge` to reflect this too

Sources
* https://github.com/open-telemetry/semantic-conventions/blob/723789b86a803492d03191b458138fff6e7a6b9f/docs/registry/attributes/db.md#db-cosmosdb-request-charge
* https://opentelemetry.io/docs/specs/semconv/registry/attributes/azure/#azure-cosmosdb-operation-request-charge

Signed-off-by: lloydmeta <lloydmeta@gmail.com>

* Add some tests to validate types and values.

* Add changelog

Signed-off-by: lloydmeta <lloydmeta@gmail.com>

* * Restore wrong types for existing attributes

* * Remove from bugs fixed

---------

Signed-off-by: lloydmeta <lloydmeta@gmail.com>

Author: lloydmeta

sdk/cosmos/azure-cosmos-tests/src/test/java/com/azure/cosmos/CosmosTracerTest.java

@@ -1479,6 +1479,14 @@ private void verifyOTelTracerAttributes(
 
             assertThat(attributes.get("db.cosmosdb.request_content_length")).isNotNull();
 
+            assertThat(attributes.get("azure.cosmosdb.operation.request_charge")).isNotNull();
+            assertThat(attributes.get("azure.cosmosdb.operation.request_charge")).isInstanceOf(Double.class);
+            assertThat(attributes.get("azure.cosmosdb.operation.request_charge")).isEqualTo(Double.valueOf(ctx.getTotalRequestCharge()));
+
+            assertThat(attributes.get("azure.cosmosdb.response.sub_status_code")).isNotNull();
+            assertThat(attributes.get("azure.cosmosdb.response.sub_status_code")).isInstanceOf(Integer.class);
+            assertThat(attributes.get("azure.cosmosdb.response.sub_status_code")).isEqualTo(Integer.valueOf(ctx.getSubStatusCode()));
+
             assertThat(attributes.get("db.cosmosdb.operation_type")).isEqualTo(ctx.getOperationType());
             if (customOperationId != null) {
                 assertThat(attributes.get("db.cosmosdb.operation_id")).isEqualTo(customOperationId);

sdk/cosmos/azure-cosmos/CHANGELOG.md

@@ -3,6 +3,7 @@
 ### 4.72.0-beta.1 (Unreleased)
 
 #### Features Added
+* Added `azure.cosmosdb.operation.request_charge` and `azure.cosmosdb.response.sub_status_code` trace attributes. - [PR 45753](https://github.com/Azure/azure-sdk-for-java/pull/45753)
 
 #### Breaking Changes
 

sdk/cosmos/azure-cosmos/src/main/java/com/azure/cosmos/implementation/DiagnosticsProvider.java

@@ -1538,10 +1538,18 @@ public void endSpan(CosmosDiagnosticsContext cosmosCtx, Context context, boolean
                 "db.cosmosdb.sub_status_code",
                 Integer.toString(cosmosCtx.getSubStatusCode()),
                 context);
+            tracer.setAttribute(
+                "azure.cosmosdb.response.sub_status_code",
+                cosmosCtx.getSubStatusCode(),
+                context);
             tracer.setAttribute(
                 "db.cosmosdb.request_charge",
                 Float.toString(cosmosCtx.getTotalRequestCharge()),
                 context);
+            tracer.setAttribute(
+                "azure.cosmosdb.operation.request_charge",
+                (double) cosmosCtx.getTotalRequestCharge(),
+                context);
             tracer.setAttribute("db.cosmosdb.request_content_length",cosmosCtx.getMaxRequestPayloadSizeInBytes(), context);
             tracer.setAttribute("db.cosmosdb.max_response_content_length_bytes",cosmosCtx.getMaxResponsePayloadSizeInBytes(), context);
             tracer.setAttribute("db.cosmosdb.retry_count",cosmosCtx.getRetryCount() , context);