-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Fix dashboard links in Add Data UI #30860
Conversation
@@ -22,7 +22,7 @@ import { TUTORIAL_CATEGORY } from '../../../common/tutorials/tutorial_category'; | |||
import { onPremInstructions, cloudInstructions, onPremCloudInstructions } from '../../../common/tutorials/filebeat_instructions'; | |||
|
|||
export function apacheLogsSpecProvider(server, context) { | |||
const moduleName = 'apache2'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ruflin when I'm fixing the links, I saw this. Should this be apache
instead of apache2
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch, this should be apache.
Pinging @elastic/beats |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. 1 minor comment.
}, | ||
}), | ||
euiIconType: 'logoApache', | ||
artifacts: { | ||
dashboards: [ | ||
{ | ||
id: 'Filebeat-Apache2-Dashboard', | ||
id: 'Filebeat-Apache-Dashboard-ecs', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you double check if we renamed the ID in the Beats repo for the dashboard?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ruflin Yeah I tested this and it works, also found the id name in filebeat https://github.com/elastic/beats/blob/master/filebeat/module/apache/_meta/kibana/7/dashboard/Filebeat-apache.json#L647
retest |
💚 Build Succeeded |
* Fix dashboard links in Add Data UI * Fix dashboard links in logging add data ui
* Fix dashboard links in Add Data UI * Fix dashboard links in logging add data ui
Beats dashboard IDs were adjusted to not conflict with the 6.x dashboards so all the Add Data UI tutorials for Beats are broken. This PR is to update the links in kibana to fix Add Data UI when clicking for example
AWS metrics dashboard
.closes #30805