Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Logs: Add show context to dashboard panel #80403

Merged
merged 9 commits into from Jan 12, 2024

Conversation

svennergr
Copy link
Contributor

What is this feature?

This PR adds the "Log Context" functionality to the LogsPanel, i.e. it enables this feature for logs panels in dashboards.

Special notes for your reviewer:

  • add a logs panel to a dashboard and find the show context toggle

@svennergr svennergr added area/panel/logs no-backport Skip backport of PR no-changelog Skip including change in changelog/release notes labels Jan 11, 2024
@svennergr svennergr requested a review from a team as a code owner January 11, 2024 21:49
@grafana-delivery-bot grafana-delivery-bot bot added this to the 10.3.x milestone Jan 11, 2024
@svennergr svennergr requested review from a team and Eve832 as code owners January 12, 2024 12:19
@svennergr svennergr requested review from joshhunt, ashharrison90 and jackw and removed request for a team January 12, 2024 12:19
@grafana-pr-automation grafana-pr-automation bot requested review from a team and removed request for a team January 12, 2024 12:24
@grafana-pr-automation grafana-pr-automation bot requested review from a team and sunker and removed request for a team January 12, 2024 13:13
@svennergr svennergr removed the request for review from sunker January 12, 2024 13:14
@svennergr svennergr removed the request for review from sunker January 12, 2024 16:15
{showCommonLabels && !isAscending && renderCommonLabels()}
<LogRows
logRows={logRows}
// always return true for now as this propery will
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there more to this comment?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Lol, removed that. Good catch!

Copy link
Contributor

@gtk-grafana gtk-grafana left a comment

Choose a reason for hiding this comment

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

Looks good and is working for me locally. (since mixed issue appears to be in main, and not introduced by this PR)

@grafana-pr-automation grafana-pr-automation bot requested review from a team and sunker and removed request for a team January 12, 2024 16:19
@svennergr svennergr removed the request for review from sunker January 12, 2024 16:29
@grafana-pr-automation grafana-pr-automation bot requested review from a team and mckn and removed request for a team January 12, 2024 16:31
@svennergr svennergr enabled auto-merge (squash) January 12, 2024 16:42
@grafana-pr-automation grafana-pr-automation bot requested review from a team and removed request for a team January 12, 2024 17:12
@svennergr svennergr removed the request for review from mckn January 12, 2024 17:12
@svennergr svennergr merged commit 4e47416 into main Jan 12, 2024
24 checks passed
@svennergr svennergr deleted the svennergr/show-context-dashboard branch January 12, 2024 17:19
@grafana-delivery-bot grafana-delivery-bot bot modified the milestones: 10.3.x, 10.4.x Jan 12, 2024
@svennergr svennergr added add to changelog and removed no-changelog Skip including change in changelog/release notes labels Jan 24, 2024
s0lesurviv0r pushed a commit to s0lesurviv0r/grafana that referenced this pull request Feb 3, 2024
* Logs: Add show context to dashboard panel

* add prop to enable show context toggle

* update test

* adjust tests

* add query targets as a dependency

* extract `useDatasourcesFromTargets` hook

* add tests

* remove comment
@aangelisc aangelisc modified the milestones: 10.4.x, 10.4.0 Mar 6, 2024
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.

None yet

4 participants