Skip to content

Conversation

Linu-Elias
Copy link
Contributor

Proposed commit message

add panel filter options to all dashboard links

"options": {
    "openInNewTab": false,
     "useCurrentDateRange": true,
     "useCurrentFilters": false
     }

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

@Linu-Elias Linu-Elias requested review from a team as code owners August 19, 2025 06:23
@Linu-Elias Linu-Elias self-assigned this Aug 19, 2025
@elastic-vault-github-plugin-prod

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

changes:
- description: Fix Dashboard Links Panel by setting `useCurrentFilters` to `false`
type: bugfix
link: https://github.com/elastic/integrations/pull/14607
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you fix this for the rest of the integrations as well?

Suggested change
link: https://github.com/elastic/integrations/pull/14607
link: https://github.com/elastic/integrations/pull/14974

@muthu-mps muthu-mps requested a review from Copilot August 19, 2025 07:36
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes Dashboard Links Panel filter options for multiple integration packages by setting useCurrentFilters to false to prevent unwanted filter inheritance when navigating between dashboards.

  • Adds standardized options configuration to all dashboard link panels across affected packages
  • Replaces markdown-based navigation with proper links panel components for better UX
  • Updates version numbers and changelog entries for all modified packages

Reviewed Changes

Copilot reviewed 28 out of 28 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
packages/system/manifest.yml Version bump to 2.5.3
packages/system/kibana/dashboard/*.json Adds filter options to dashboard links and converts navigation from markdown to links panel
packages/nginx_ingress_controller_otel/manifest.yml Version bump to 0.2.1
packages/nginx_ingress_controller_otel/kibana/dashboard/*.json Adds filter options to dashboard links
packages/nginx/manifest.yml Version bump to 2.3.1
packages/nginx/kibana/dashboard/*.json Adds filter options to dashboard links
packages/nagios_xi/manifest.yml Version bump to 1.7.1
packages/nagios_xi/kibana/dashboard/*.json Adds filter options to dashboard links
packages/iis/manifest.yml Version bump to 1.24.1
packages/iis/kibana/dashboard/*.json Adds filter options to dashboard links
packages/*/changelog.yml Changelog entries documenting the bugfix

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

"params": {
"fontSize": 12,
"markdown": "### Navigation\n\n#### System\n\n[Logon Activity](#/dashboard/system-2c4debf0-ef4f-4379-99a1-c57c307f23af) \n[Directory \u0026 Account Monitoring](#/dashboard/system-3c46ecdb-0a41-4be3-907e-722de8edac12) \n[System \u0026 Process Activity](#/dashboard/system-0e70e1bd-9a57-4f17-9d96-cc97e3d3a4f9) \n**Policy \u0026 Object Monitoring**\n\n#### Description\n\nThis dashboard provides focused visibility into policy enforcement and object access activities.\n\nIt showcases the top users, domains, and processes involved in object access attempts, along with access permissions and firewall rule usage. Key visualizations highlight rule modifications, failures, and filtering behaviors. Detailed logs and saved searches offer insights into share access, transaction state changes, and firewall filter changes, helping identify policy violations and strengthen host-level security.\n\n**[Integration Page](/app/integrations/detail/system/overview)**",
"markdown": "This dashboard provides focused visibility into policy enforcement and object access activities.\n\nIt showcases the top users, domains, and processes involved in object access attempts, along with access permissions and firewall rule usage. Key visualizations highlight rule modifications, failures, and filtering behaviors. Detailed logs and saved searches offer insights into share access, transaction state changes, and firewall filter changes, helping identify policy violations and strengthen host-level security.\n\n**[Integration Page](/app/integrations/detail/system/overview)\n",
Copy link
Preview

Copilot AI Aug 19, 2025

Choose a reason for hiding this comment

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

The markdown text contains an incomplete closing markdown link syntax. It should end with ** instead of \n". The line should be: **[Integration Page](/app/integrations/detail/system/overview)**

Suggested change
"markdown": "This dashboard provides focused visibility into policy enforcement and object access activities.\n\nIt showcases the top users, domains, and processes involved in object access attempts, along with access permissions and firewall rule usage. Key visualizations highlight rule modifications, failures, and filtering behaviors. Detailed logs and saved searches offer insights into share access, transaction state changes, and firewall filter changes, helping identify policy violations and strengthen host-level security.\n\n**[Integration Page](/app/integrations/detail/system/overview)\n",
"markdown": "This dashboard provides focused visibility into policy enforcement and object access activities.\n\nIt showcases the top users, domains, and processes involved in object access attempts, along with access permissions and firewall rule usage. Key visualizations highlight rule modifications, failures, and filtering behaviors. Detailed logs and saved searches offer insights into share access, transaction state changes, and firewall filter changes, helping identify policy violations and strengthen host-level security.\n\n**[Integration Page](/app/integrations/detail/system/overview)**\n",

Copilot uses AI. Check for mistakes.

"params": {
"fontSize": 12,
"markdown": "### Navigation\n\n#### System\n\n[Logon Activity](#/dashboard/system-2c4debf0-ef4f-4379-99a1-c57c307f23af) \n**Directory \u0026 Account Monitoring** \n[System \u0026 Process Activity](#/dashboard/system-0e70e1bd-9a57-4f17-9d96-cc97e3d3a4f9) \n[Policy \u0026 Object Monitoring](#/dashboard/system-db94459a-7232-4d1b-aa0c-b80dece8bc3a) \n\n#### Description\nThis dashboard focuses on monitoring directory changes and account activities.\n\nIt highlights top users, domains, and Active Directory objects involved in modifications, along with trends in attribute changes and replication events. Password policy checks are tracked by user, workstation, and domain, with detailed logs providing insight into replication syncs and policy enforcement, helping ensure directory integrity and account security.\n\n\n**[Integration Page](/app/integrations/detail/system/overview)**",
"markdown": "This dashboard focuses on monitoring directory changes and account activities.\n\nIt highlights top users, domains, and Active Directory objects involved in modifications, along with trends in attribute changes and replication events. Password policy checks are tracked by user, workstation, and domain, with detailed logs providing insight into replication syncs and policy enforcement, helping ensure directory integrity and account security.\n\n\n**[Integration Page](/app/integrations/detail/system/overview)\n",
Copy link
Preview

Copilot AI Aug 19, 2025

Choose a reason for hiding this comment

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

The markdown text contains an incomplete closing markdown link syntax. It should end with ** instead of \n". The line should be: **[Integration Page](/app/integrations/detail/system/overview)**

Suggested change
"markdown": "This dashboard focuses on monitoring directory changes and account activities.\n\nIt highlights top users, domains, and Active Directory objects involved in modifications, along with trends in attribute changes and replication events. Password policy checks are tracked by user, workstation, and domain, with detailed logs providing insight into replication syncs and policy enforcement, helping ensure directory integrity and account security.\n\n\n**[Integration Page](/app/integrations/detail/system/overview)\n",
"markdown": "This dashboard focuses on monitoring directory changes and account activities.\n\nIt highlights top users, domains, and Active Directory objects involved in modifications, along with trends in attribute changes and replication events. Password policy checks are tracked by user, workstation, and domain, with detailed logs providing insight into replication syncs and policy enforcement, helping ensure directory integrity and account security.\n\n\n**[Integration Page](/app/integrations/detail/system/overview)**\n",

Copilot uses AI. Check for mistakes.

@@ -439,7 +508,7 @@
"id": "",
"params": {
"fontSize": 12,
"markdown": "### Navigation\n\n#### System\n\n**Logon Activity** \n[Directory \u0026 Account Monitoring](#/dashboard/system-3c46ecdb-0a41-4be3-907e-722de8edac12) \n[System \u0026 Process Activity](#/dashboard/system-0e70e1bd-9a57-4f17-9d96-cc97e3d3a4f9) \n[Policy \u0026 Object Monitoring](#/dashboard/system-db94459a-7232-4d1b-aa0c-b80dece8bc3a) \n\n#### Description\n\nThis dashboard is designed to provide focused visibility into logon and logoff activities using the Security data stream.\n\nIt highlights the top 10 group membership logons by account name, visualizes group membership events by logon type and user, and presents detailed insights into trust direction filtering. The dashboard also tracks workstation lock events across users, security IDs, and account domains to support monitoring of authentication patterns and identification of unusual behavior.\n\n**[Integration Page](/app/integrations/detail/system/overview)**",
"markdown": "This dashboard is designed to provide focused visibility into logon and logoff activities using the Security data stream.\n\nIt highlights the top 10 group membership logons by account name, visualizes group membership events by logon type and user, and presents detailed insights into trust direction filtering. The dashboard also tracks workstation lock events across users, security IDs, and account domains to support monitoring of authentication patterns and identification of unusual behavior.\n\n**[Integration Page](/app/integrations/detail/system/overview)\n",
Copy link
Preview

Copilot AI Aug 19, 2025

Choose a reason for hiding this comment

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

The markdown text contains an incomplete closing markdown link syntax. It should end with ** instead of \n". The line should be: **[Integration Page](/app/integrations/detail/system/overview)**

Suggested change
"markdown": "This dashboard is designed to provide focused visibility into logon and logoff activities using the Security data stream.\n\nIt highlights the top 10 group membership logons by account name, visualizes group membership events by logon type and user, and presents detailed insights into trust direction filtering. The dashboard also tracks workstation lock events across users, security IDs, and account domains to support monitoring of authentication patterns and identification of unusual behavior.\n\n**[Integration Page](/app/integrations/detail/system/overview)\n",
"markdown": "This dashboard is designed to provide focused visibility into logon and logoff activities using the Security data stream.\n\nIt highlights the top 10 group membership logons by account name, visualizes group membership events by logon type and user, and presents detailed insights into trust direction filtering. The dashboard also tracks workstation lock events across users, security IDs, and account domains to support monitoring of authentication patterns and identification of unusual behavior.\n\n**[Integration Page](/app/integrations/detail/system/overview)**\n",

Copilot uses AI. Check for mistakes.

"params": {
"fontSize": 12,
"markdown": "### Navigation\n\n#### System\n\n[Logon Activity](#/dashboard/system-2c4debf0-ef4f-4379-99a1-c57c307f23af) \n[Directory \u0026 Account Monitoring](#/dashboard/system-3c46ecdb-0a41-4be3-907e-722de8edac12) \n**System \u0026 Process Activity** \n[Policy \u0026 Object Monitoring](#/dashboard/system-db94459a-7232-4d1b-aa0c-b80dece8bc3a)\n\n#### Description\n\nThis dashboard offers visibility into critical system and process activities, with a focus on file and cryptographic operations.\n\nIt highlights key and file operations by provider, top affected file paths, and device connection trends over time. Saved searches provide detailed insights into cryptographic events, key migrations, and file operations, supporting analysis of system behavior and potential security impacts.\n\n\n**[Integration Page](/app/integrations/detail/system/overview)**",
"markdown": "This dashboard offers visibility into critical system and process activities, with a focus on file and cryptographic operations.\n\nIt highlights key and file operations by provider, top affected file paths, and device connection trends over time. Saved searches provide detailed insights into cryptographic events, key migrations, and file operations, supporting analysis of system behavior and potential security impacts.\n\n\n**[Integration Page](/app/integrations/detail/system/overview)\n",
Copy link
Preview

Copilot AI Aug 19, 2025

Choose a reason for hiding this comment

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

The markdown text contains an incomplete closing markdown link syntax. It should end with ** instead of \n". The line should be: **[Integration Page](/app/integrations/detail/system/overview)**

Suggested change
"markdown": "This dashboard offers visibility into critical system and process activities, with a focus on file and cryptographic operations.\n\nIt highlights key and file operations by provider, top affected file paths, and device connection trends over time. Saved searches provide detailed insights into cryptographic events, key migrations, and file operations, supporting analysis of system behavior and potential security impacts.\n\n\n**[Integration Page](/app/integrations/detail/system/overview)\n",
"markdown": "This dashboard offers visibility into critical system and process activities, with a focus on file and cryptographic operations.\n\nIt highlights key and file operations by provider, top affected file paths, and device connection trends over time. Saved searches provide detailed insights into cryptographic events, key migrations, and file operations, supporting analysis of system behavior and potential security impacts.\n\n\n**[Integration Page](/app/integrations/detail/system/overview)**\n",

Copilot uses AI. Check for mistakes.

@muthu-mps
Copy link
Contributor

@Linu-Elias - Can you confirm the latest integration is installed after making the change and link works as expected?

@elasticmachine
Copy link

💚 Build Succeeded

History

cc @Linu-Elias

Copy link

@Linu-Elias
Copy link
Contributor Author

@Linu-Elias - Can you confirm the latest integration is installed after making the change and link works as expected?

@muthu-mps, Yes, I tested it by installing and checking that the links are working as expected. Now, only the current dashboard filters are applied.

Copy link
Contributor

@muthu-mps muthu-mps left a comment

Choose a reason for hiding this comment

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

LGTM!

@Linu-Elias Linu-Elias merged commit cc4e810 into elastic:main Aug 19, 2025
9 checks passed
@elastic-vault-github-plugin-prod

Package iis - 1.24.1 containing this change is available at https://epr.elastic.co/package/iis/1.24.1/

@elastic-vault-github-plugin-prod

Package nagios_xi - 1.7.1 containing this change is available at https://epr.elastic.co/package/nagios_xi/1.7.1/

@elastic-vault-github-plugin-prod

Package nginx - 2.3.1 containing this change is available at https://epr.elastic.co/package/nginx/2.3.1/

@elastic-vault-github-plugin-prod

Package nginx_ingress_controller_otel - 0.2.1 containing this change is available at https://epr.elastic.co/package/nginx_ingress_controller_otel/0.2.1/

@elastic-vault-github-plugin-prod

Package system - 2.5.3 containing this change is available at https://epr.elastic.co/package/system/2.5.3/

@andrewkroh andrewkroh added Integration:nagios_xi Nagios XI Integration:nginx Nginx Integration:iis IIS Integration:nginx_ingress_controller_otel Nginx Ingress Controller OpenTelemetry Logs Integration:system System dashboard Relates to a Kibana dashboard bug, enhancement, or modification. bugfix Pull request that fixes a bug issue labels Aug 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Pull request that fixes a bug issue dashboard Relates to a Kibana dashboard bug, enhancement, or modification. Integration:iis IIS Integration:nagios_xi Nagios XI Integration:nginx_ingress_controller_otel Nginx Ingress Controller OpenTelemetry Logs Integration:nginx Nginx Integration:system System
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants