Skip to content

Conversation

@idastambuk
Copy link
Contributor

@idastambuk idastambuk commented Aug 1, 2025

We got a bug report that from v12 on two cloudwatch queries of different types (metrics + logs) are not shown in the panel.
The reason for this is that the first query is always cancelled:
Screenshot 2025-08-01 at 16 03 49

I think at some point backend srv started cancelling queries if they had the same requestId as another query. In Cloudwatch we merge the logs and the metrics observables, but they're still separate calls (with the same id), however they have the same id.
Appending the query type seems to get the queries to finish executing:
Screenshot 2025-08-01 at 15 53 15

@idastambuk idastambuk force-pushed the 106584-cloudwatch-data-source-cannot-display-both-cloudwatch-metric-and-cloudwatch-log-in-the-same-panel branch from 2762738 to 3f366e9 Compare August 4, 2025 18:49
return mockFrames;
}

export function genMockCloudWatchLogsRequest(overrides: Partial<CloudWatchLogsRequest> = {}) {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this isn't being used anywhere

@idastambuk idastambuk marked this pull request as ready for review August 5, 2025 09:37
@idastambuk idastambuk requested a review from a team as a code owner August 5, 2025 09:37
@idastambuk idastambuk requested review from kevinwcyu and njvrzm and removed request for a team August 5, 2025 09:37
@idastambuk idastambuk changed the title Append query type to the request id CloudWatch: Append query type to the request id Aug 10, 2025
@idastambuk idastambuk merged commit c39c6a2 into main Aug 11, 2025
110 checks passed
@idastambuk idastambuk deleted the 106584-cloudwatch-data-source-cannot-display-both-cloudwatch-metric-and-cloudwatch-log-in-the-same-panel branch August 11, 2025 13:20
njvrzm pushed a commit that referenced this pull request Sep 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants