Skip to content

Commit

Permalink
Upgrade to opentelemetry-proto v1.1.0 (#104)
Browse files Browse the repository at this point in the history
* Upgrade to opentelemetry-proto v1.1.0

Signed-off-by: Yuri Shkuro <[email protected]>

* separate

Signed-off-by: Yuri Shkuro <[email protected]>

* regen

Signed-off-by: Yuri Shkuro <[email protected]>

---------

Signed-off-by: Yuri Shkuro <[email protected]>
  • Loading branch information
yurishkuro authored Jan 14, 2024
1 parent a97152c commit bdd5ab5
Show file tree
Hide file tree
Showing 3 changed files with 40 additions and 22 deletions.
12 changes: 10 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -91,22 +91,30 @@ PROTOC_INTERNAL := $(PROTOC) \
--csharp_out=internal_access,base_namespace:${PROTO_GEN_CSHARP_DIR} \
--python_out=${PROTO_GEN_PYTHON_DIR}

proto:
.PHONY: proto
proto: proto-prepare proto-api-v2 proto-api-v3

.PHONY: proto-prepare
proto-prepare:
mkdir -p ${PROTO_GEN_GO_DIR} \
${PROTO_GEN_JAVA_DIR} \
${PROTO_GEN_PYTHON_DIR} \
${PROTO_GEN_JS_DIR} \
${PROTO_GEN_CPP_DIR} \
${PROTO_GEN_CSHARP_DIR}

.PHONY: proto-api-v2
proto-api-v2:
$(PROTOC_WITHOUT_GRPC) \
proto/api_v2/model.proto

$(PROTOC_WITH_GRPC) \
proto/api_v2/query.proto \
proto/api_v2/collector.proto \
proto/api_v2/sampling.proto

.PHONY: proto-api-v3
proto-api-v3:
# API v3
$(PROTOC_WITH_GRPC) \
proto/api_v3/query_service.proto
Expand Down
48 changes: 29 additions & 19 deletions swagger/api_v3/query_service.swagger.json
Original file line number Diff line number Diff line change
Expand Up @@ -72,13 +72,13 @@
"type": "object",
"properties": {
"result": {
"$ref": "#/definitions/api_v3SpansResponseChunk"
"$ref": "#/definitions/v1TracesData"
},
"error": {
"$ref": "#/definitions/runtimeStreamError"
}
},
"title": "Stream result of api_v3SpansResponseChunk"
"title": "Stream result of v1TracesData"
}
}
},
Expand Down Expand Up @@ -150,13 +150,13 @@
"type": "object",
"properties": {
"result": {
"$ref": "#/definitions/api_v3SpansResponseChunk"
"$ref": "#/definitions/v1TracesData"
},
"error": {
"$ref": "#/definitions/runtimeStreamError"
}
},
"title": "Stream result of api_v3SpansResponseChunk"
"title": "Stream result of v1TracesData"
}
}
},
Expand Down Expand Up @@ -247,6 +247,11 @@
"type": "integer",
"format": "int64",
"description": "dropped_attributes_count is the number of dropped attributes. If the value is 0,\nthen no attributes were dropped."
},
"flags": {
"type": "integer",
"format": "int64",
"description": "Flags, a bit field. 8 least significant bits are the trace\nflags as defined in W3C Trace Context specification. Readers\nMUST not assume that 24 most significant bits will be zero.\nWhen creating new spans, the most-significant 24-bits MUST be\nzero. To read the 8-bit W3C trace flag (use flags \u0026\nSPAN_FLAGS_TRACE_FLAGS_MASK). [Optional].\n\nSee https://www.w3.org/TR/trace-context-2/#trace-flags for the flag definitions."
}
},
"description": "A pointer from the current span to another span in the same trace or in a\ndifferent trace. For example, this can be used in batching operations,\nwhere a single batch handler processes multiple requests from different\ntraces or when the handler receives a request from a different project."
Expand Down Expand Up @@ -311,19 +316,6 @@
},
"description": "Operation encapsulates information about operation."
},
"api_v3SpansResponseChunk": {
"type": "object",
"properties": {
"resource_spans": {
"type": "array",
"items": {
"$ref": "#/definitions/v1ResourceSpans"
},
"title": "A list of OpenTelemetry ResourceSpans.\nIn case of JSON format the ids (trace_id, span_id, parent_id) are encoded in base64 even though OpenTelemetry specification\nmandates to use hex encoding [2].\nBase64 is chosen to keep compatibility with JSONPb codec.\n[1]: https://github.com/open-telemetry/opentelemetry-proto/blob/main/opentelemetry/proto/trace/v1/trace.proto\n[2]: https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/protocol/otlp.md#otlphttp"
}
},
"description": "Response object with spans."
},
"api_v3TraceQueryParameters": {
"type": "object",
"properties": {
Expand Down Expand Up @@ -529,7 +521,7 @@
},
"schema_url": {
"type": "string",
"description": "This schema_url applies to the data in the \"resource\" field. It does not apply\nto the data in the \"scope_spans\" field which have their own schema_url field."
"description": "The Schema URL, if known. This is the identifier of the Schema that the resource data\nis recorded in. To learn more about Schema URL see\nhttps://opentelemetry.io/docs/specs/otel/schemas/#schema-url\nThis schema_url applies to the data in the \"resource\" field. It does not apply\nto the data in the \"scope_spans\" field which have their own schema_url field."
}
},
"description": "A collection of ScopeSpans from a Resource."
Expand All @@ -550,7 +542,7 @@
},
"schema_url": {
"type": "string",
"description": "This schema_url applies to all spans and span events in the \"spans\" field."
"description": "The Schema URL, if known. This is the identifier of the Schema that the span data\nis recorded in. To learn more about Schema URL see\nhttps://opentelemetry.io/docs/specs/otel/schemas/#schema-url\nThis schema_url applies to all spans and span events in the \"spans\" field."
}
},
"description": "A collection of Spans produced by an InstrumentationScope."
Expand All @@ -577,6 +569,11 @@
"format": "byte",
"description": "The `span_id` of this span's parent span. If this is a root span, then this\nfield must be empty. The ID is an 8-byte array."
},
"flags": {
"type": "integer",
"format": "int64",
"description": "Flags, a bit field. 8 least significant bits are the trace\nflags as defined in W3C Trace Context specification. Readers\nMUST not assume that 24 most significant bits will be zero.\nTo read the 8-bit W3C trace flag, use `flags \u0026 SPAN_FLAGS_TRACE_FLAGS_MASK`.\n\nWhen creating span messages, if the message is logically forwarded from another source\nwith an equivalent flags fields (i.e., usually another OTLP span message), the field SHOULD\nbe copied as-is. If creating from a source that does not have an equivalent flags field\n(such as a runtime representation of an OpenTelemetry span), the high 24 bits MUST\nbe set to zero.\n\n[Optional].\n\nSee https://www.w3.org/TR/trace-context-2/#trace-flags for the flag definitions."
},
"name": {
"type": "string",
"description": "A description of the span's operation.\n\nFor example, the name can be a qualified method name or a file name\nand a line number where the operation is called. A best practice is to use\nthe same display name at the same call point in an application.\nThis makes it easier to correlate spans in different traces.\n\nThis field is semantically required to be set to non-empty string.\nEmpty value is equivalent to an unknown span name.\n\nThis field is required."
Expand Down Expand Up @@ -652,6 +649,19 @@
}
},
"description": "The Status type defines a logical error model that is suitable for different\nprogramming environments, including REST APIs and RPC APIs."
},
"v1TracesData": {
"type": "object",
"properties": {
"resource_spans": {
"type": "array",
"items": {
"$ref": "#/definitions/v1ResourceSpans"
},
"description": "An array of ResourceSpans.\nFor data coming from a single resource this array will typically contain\none element. Intermediary nodes that receive data from multiple origins\ntypically batch the data before forwarding further and in that case this\narray will contain multiple elements."
}
},
"description": "TracesData represents the traces data that can be stored in a persistent storage,\nOR can be embedded by other protocols that transfer OTLP traces data but do\nnot implement the OTLP protocol.\n\nThe main difference between this message and collector protocol is that\nin this message there will not be any \"control\" or \"metadata\" specific to\nOTLP protocol.\n\nWhen new fields are added into this message, the OTLP request MUST be updated\nas well."
}
}
}

0 comments on commit bdd5ab5

Please sign in to comment.