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] Fix wrong alias in rabbitmq module #11284

Merged
merged 2 commits into from
Mar 18, 2019

Conversation

ruflin
Copy link
Contributor

@ruflin ruflin commented Mar 18, 2019

  • rabbitmq.node.name was duplicated and alias pointed to itself
  • rabbitmq.connection.name was falsely migrated to rabbitmq.name

Also adding tests to Filebeat and Metricbeat to ensure this does not happen again in the future.

Closes #11271

* rabbitmq.node.name was duplicated and alias pointed to itself
* rabbitmq.connection.name was falsely migrated to `rabbitmq.name`

Also adding tests to Filebeat and Metricbeat to ensure this does not happen again in the future.

Closes elastic#11271
@ruflin ruflin added bug module review Metricbeat Metricbeat needs_backport PR is waiting to be backported to other branches. Team:Integrations Label for the Integrations team labels Mar 18, 2019
@ruflin ruflin requested review from a team as code owners March 18, 2019 08:44
@ruflin ruflin self-assigned this Mar 18, 2019
@@ -20543,12 +20533,10 @@ connection



*`rabbitmq.connection.name`*::
*`rabbitmq.connection.connection.name`*::
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't this be rabbit.connection.name as before?

@ruflin
Copy link
Contributor Author

ruflin commented Mar 18, 2019

@jsoriano Thanks for catching, new version pushed.

@ruflin ruflin merged commit feaef40 into elastic:master Mar 18, 2019
@ruflin ruflin deleted the fix-rabbitmq-field-issue branch March 18, 2019 11:18
ruflin added a commit to ruflin/beats that referenced this pull request Mar 18, 2019
* rabbitmq.node.name was duplicated and alias pointed to itself
* rabbitmq.connection.name was falsely migrated to `rabbitmq.name`

Also adding tests to Filebeat and Metricbeat to ensure this does not happen again in the future.

Closes elastic#11271

(cherry picked from commit feaef40)
@ruflin ruflin added v7.0.0 and removed needs_backport PR is waiting to be backported to other branches. labels Mar 18, 2019
ruflin added a commit that referenced this pull request Mar 18, 2019
* rabbitmq.node.name was duplicated and alias pointed to itself
* rabbitmq.connection.name was falsely migrated to `rabbitmq.name`

Also adding tests to Filebeat and Metricbeat to ensure this does not happen again in the future.

Closes #11271

(cherry picked from commit feaef40)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants