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

[Metricbeat] Add Operating System Name to add_host_metadata processor #8948

Closed
eddieturizo opened this issue Nov 6, 2018 · 2 comments · Fixed by #9405
Closed

[Metricbeat] Add Operating System Name to add_host_metadata processor #8948

eddieturizo opened this issue Nov 6, 2018 · 2 comments · Fixed by #9405
Assignees
Labels
enhancement good first issue Indicates a good issue for first-time contributors Metricbeat Metricbeat module Team:Integrations Label for the Integrations team

Comments

@eddieturizo
Copy link

When using the add_host_metadata Metricbeat processor it would be useful to include the Operating System name so it returns the following field value in Windows

Microsoft Windows Server 2012 R2 Datacenter

Might also be worthy to add the same for other Operating Systems that Metricbeat supports so that you'd have fields similar to

Ubuntu 18.04 LTS
CentOS Linux 7
Red Hat Enterprise Linux Server 7.3

@ruflin
Copy link
Contributor

ruflin commented Nov 7, 2018

@eddieturizo Would you index the OS name as keyword or text?

@ruflin ruflin added the Team:Integrations Label for the Integrations team label Nov 21, 2018
@eddieturizo
Copy link
Author

@eddieturizo Would you index the OS name as keyword or text?

I would index this as keyword. Thank you.

@alvarolobato alvarolobato added the good first issue Indicates a good issue for first-time contributors label Dec 4, 2018
@kaiyan-sheng kaiyan-sheng self-assigned this Dec 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement good first issue Indicates a good issue for first-time contributors Metricbeat Metricbeat module Team:Integrations Label for the Integrations team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants