Skip to content

feat: logs and traces dashboard#199

Open
mandryllo wants to merge 3 commits intomasterfrom
feat/logs-traces-dash
Open

feat: logs and traces dashboard#199
mandryllo wants to merge 3 commits intomasterfrom
feat/logs-traces-dash

Conversation

@mandryllo
Copy link
Copy Markdown
Collaborator

@mandryllo mandryllo commented Apr 15, 2026

No description provided.

@mandryllo mandryllo force-pushed the feat/logs-traces-dash branch 3 times, most recently from 9f55015 to 1e10148 Compare April 17, 2026 06:55
Comment thread src/components/grafana/dashboards/builder.ts Outdated
@mandryllo mandryllo force-pushed the feat/logs-traces-dash branch from 45055bd to 33aeb7e Compare April 17, 2026 09:12
@mandryllo mandryllo requested review from bornast and droguljic April 17, 2026 09:14
@droguljic droguljic changed the title Feat: logs and traces dashboard feat: logs and traces dashboard Apr 17, 2026
@mandryllo mandryllo force-pushed the feat/logs-traces-dash branch from f60cb65 to 32e3469 Compare April 20, 2026 08:56
Comment thread src/components/grafana/panels/types.ts Outdated
Comment thread src/components/grafana/variables/index.ts Outdated
Comment thread src/components/grafana/panels/logs.ts Outdated
Comment on lines +13 to +21
`fields @Timestamp
| parse @message '"body":"*"' as body
| parse @message '"res":{"statusCode":*}' as statusCode
| parse @message '"severity_text":"*"' as logLevel
| filter body like /\${search_text}/
| filter \${status_code}
| filter \${log_level}
| sort @timestamp desc
| limit \${limit}`,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

extract this to a variable

Comment thread src/components/grafana/panels/logs.ts Outdated
Comment on lines +22 to +23
[
{
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Also this array could be extracted to a variable

dataSource: string,
logGroupName: string,
expression: string,
transformations: Transformation[],
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

is this prop really required or can we have a functional table panel without it?

TextBoxVariable,
} from './types';

const buildQuery: BuildQuery = options => JSON.stringify(options);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

do we need this abstraction?

Comment on lines +3 to +10
const LIMITS = [
{ text: '20', value: 20 },
{ text: '50', value: 50 },
{ text: '100', value: 100 },
{ text: '250', value: 250 },
{ text: '500', value: 500 },
{ text: '1000', value: 1000 },
];
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

maybe we could use this as defaults and allow the function to receive param with different configuration when needed?

Comment thread src/components/grafana/panels/helpers.ts Outdated
@mandryllo mandryllo force-pushed the feat/logs-traces-dash branch from 32e3469 to d4fde7e Compare April 27, 2026 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants