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

Fix panic on metricbeat test modules #18797

Merged
merged 5 commits into from
May 29, 2020

Commits on May 27, 2020

  1. Fix panic on metricbeat test modules

    Since metricbeat light modules support processors (elastic#15923), module
    initialization requires a publisher in the beat so modules can attach
    their processors. `metricbeat test modules` is not initializing as
    normal metricbeat commands, and it is not initializing any output or
    publisher pipeline, so metricbeat panics when trying to initialize
    modules with the new method.
    
    This change adds a dummy publisher for this case, and fixes also a
    condition that was adding a `nil` module option, causing additional
    panics. A test that reproduced the issue is also added.
    jsoriano committed May 27, 2020
    Configuration menu
    Copy the full SHA
    2c61ce9 View commit details
    Browse the repository at this point in the history
  2. Fix changelog

    jsoriano committed May 27, 2020
    Configuration menu
    Copy the full SHA
    aaa9015 View commit details
    Browse the repository at this point in the history

Commits on May 28, 2020

  1. Configuration menu
    Copy the full SHA
    ba1fc7c View commit details
    Browse the repository at this point in the history

Commits on May 29, 2020

  1. Configuration menu
    Copy the full SHA
    8239828 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    873ceb8 View commit details
    Browse the repository at this point in the history