Skip to content

feat: Support the new OTEL metric format removing the client_id metric point granularity#21048

Merged
kodiakhq[bot] merged 3 commits intomainfrom
feat/improve-telemetry
Jul 17, 2025
Merged

feat: Support the new OTEL metric format removing the client_id metric point granularity#21048
kodiakhq[bot] merged 3 commits intomainfrom
feat/improve-telemetry

Conversation

@murarustefaan
Copy link
Copy Markdown
Member

@murarustefaan murarustefaan commented Jul 15, 2025

Should go with cloudquery/plugin-sdk#2185 in order to allow the CLI to read metrics without the client_id metric point.

old CLI, old SDK
old CLI, new SDK
updated CLI, old SDK
updated CLI, new SDK

The above showcase that Duration metrics are unavailable if using an old CLI version with newer plugin versions, but that's a downside we have to work with.

@murarustefaan murarustefaan marked this pull request as ready for review July 17, 2025 08:55
@murarustefaan murarustefaan requested review from a team, blesniewski, erezrokah and przste-go July 17, 2025 08:55
@murarustefaan murarustefaan added the automerge Automatically merge once required checks pass label Jul 17, 2025
@kodiakhq kodiakhq bot merged commit 6849749 into main Jul 17, 2025
18 checks passed
@kodiakhq kodiakhq bot deleted the feat/improve-telemetry branch July 17, 2025 10:34
kodiakhq bot pushed a commit that referenced this pull request Jul 18, 2025
🤖 I have created a release *beep* *boop*
---


## [6.24.0](cli-v6.23.0...cli-v6.24.0) (2025-07-18)


### Features

* Add pre login next step information to CLI ([#21053](#21053)) ([1bc8f99](1bc8f99))
* Support the new OTEL metric format removing the client_id metric point granularity ([#21048](#21048)) ([6849749](6849749))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/cli automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants