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

[WIP] Logstash Netflow module tutorial #5

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
65 changes: 65 additions & 0 deletions src/core_plugins/kibana/server/tutorials/netflow/index.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,65 @@
import { TUTORIAL_CATEGORY } from '../../../common/tutorials/tutorial_category';
import { INSTRUCTION_VARIANT } from '../../../common/tutorials/instruction_variant';

export function netflowSpecProvider() {
return {
id: 'netflow',
name: 'Netflow',
category: TUTORIAL_CATEGORY.SECURITY,
shortDescription: 'Collect Netflow records sent by a Netflow exporter',
longDescription: 'The Logstash Netflow module simplifies the collection, normalization, and visualization of network flow data. ' +
'With a single command, the module parses network flow data, indexes the events into Elasticsearch, and installs a suite of Kibana ' +
'dashboards to get you exploring your data immediately. Logstash modules support Netflow Version 5 and 9. [Learn more]' +
'({config.elastic_docs.website_url}/guide/en/logstash/{config.elastic_docs.link_version}/netflow-module.html) about the Netflow ' +
'module.',
//iconPath: '', TODO
completionTimeMinutes: 10,
//previewImagePath: 'kibana-apache.png', TODO
params: [
{
'netflow.var.input.udp.port': {
type: 'number', // TODO: Make this a const as well?
Copy link
Author

Choose a reason for hiding this comment

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

@nreese What do you think of making these consts/enums as well?

Copy link
Owner

Choose a reason for hiding this comment

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

good idea. I have started a new file for param type constants that you can use - https://github.com/nreese/kibana/blob/home_phase_two/src/core_plugins/kibana/common/tutorials/param_types.js

defaultValue: 2055
}
}
],
instructionSets: [
{
title: 'Getting Started',
instructionVariants: [
{
id: INSTRUCTION_VARIANT.OSX,
instructions: [
{
title: 'Download and install Logstash',
textPre: 'Skip this step if Logstash is already installed. First time using Logstash? See the ' +
'[Getting Started Guide]({config.elastic_docs.website_url}/guide/en/logstash/{config.elastic_docs.link_version}' +
'/getting-started-with-logstash.html).',
commands: [
'curl -L -O https://artifacts.elastic.co/downloads/logstash/logstash-{config.kibana.version}.tar.gz',
'tar xzvf logstash-{config.kibana.version}.tar.gz'
]
},
{
title: 'Setup the Netflow module',

Choose a reason for hiding this comment

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

Should say "Set up" (two words). See http://grammarist.com/spelling/set-up-vs-setup/

textPre: 'In the Logstash install directory, run the following command to setup the Netflow module.',

Choose a reason for hiding this comment

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

same here. setup > set up.

commands: [
'./bin/logstash --modules netflow --setup',

Choose a reason for hiding this comment

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

Are there any platforms where the leading ./ actually required to run Logstash? Not sure, but I'm able to run Logstash on mac with bin/logstash, and that's what we use all over the Logstash docs. TBH, I wish Beats could do the same.

],
textPost: 'The --setup option creates a `netflow-*` index pattern in Elasticsearch and imports' +

Choose a reason for hiding this comment

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

Surround --setup in backtics as you have in other places.

' Kibana dashboards and visualizations. Running `--setup` is a one-time setup step. Omit this step' +

Choose a reason for hiding this comment

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

We should standardize on the text that we use in the tutorials to explain --setup. I think the Filebeat modules use something slightly different. For UI text especially, we should be super concise.

Choose a reason for hiding this comment

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

Note that here setup (one word) is used correctly. :-)

' for subsequent runs of the module to avoid overwriting existing Kibana dashboards.'
},
{
title: 'Start Logstash',
commands: [
'./bin/logstash --modules netflow -M netflow.var.input.udp.port={params.netflow.var.input.udp.port}'
]
}
]
}
]
}
]
};
}
2 changes: 2 additions & 0 deletions src/core_plugins/kibana/server/tutorials/register.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import { nginxLogsSpecProvider } from './nginxLogs';
import { nginxMetricsSpecProvider } from './nginxMetrics';
import { mysqlLogsSpecProvider } from './mysqlLogs';
import { mysqlMetricsSpecProvider } from './mysqlMetrics';
import { netflowSpecProvider } from './netflow';

export function registerTutorials(server) {
server.registerTutorial(systemLogsSpecProvider);
Expand All @@ -16,4 +17,5 @@ export function registerTutorials(server) {
server.registerTutorial(nginxMetricsSpecProvider);
server.registerTutorial(mysqlLogsSpecProvider);
server.registerTutorial(mysqlMetricsSpecProvider);
server.registerTutorial(netflowSpecProvider);
}