Skip to content
This repository has been archived by the owner on Apr 12, 2022. It is now read-only.

Added support for Beats modules #132

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

stoffus
Copy link

@stoffus stoffus commented Feb 25, 2021

Implemented support for managing Beats modules. For the case of default configuration, it assumes the corresponding module to be present inside the modules.d directory.

Basic usage

Enabling modules with custom configuration

Enable the nginx module with custom configuration:

beat_modules:
  nginx:
    config:
      access:
        enabled: true
        var.paths: ["/var/log/nginx/access.log", "/var/log/nginx/*/access.log"]
      error:
        enabled: true
        var.paths: ["/var/log/nginx/error.log", "/var/log/nginx/*/error.log"]

Enabling modules with default configuration

Enable the system and redis modules with default configuration:

beat_modules:
  system:
  redis:

Disabling modules

Disable the rabbitmq module:

beat_modules:
  rabbitmq:
    enabled: false

Signed-off-by: Christopher Svensson <stoffus@stoffus.com>
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@cla-checker-service
Copy link

cla-checker-service bot commented Feb 25, 2021

💚 CLA has been signed

@jmlrt
Copy link
Member

jmlrt commented Mar 3, 2021

jenkins test this please

@jmlrt
Copy link
Member

jmlrt commented Mar 3, 2021

Hi @stoffus,
Thanks for opening this PR.
We'll try to review it soon. Please be patient.

@K2Death
Copy link

K2Death commented Apr 7, 2021

Any news ?

@K2Death
Copy link

K2Death commented May 3, 2021

It's working in Production Environment. (test with my own fork)

Is it possible to review this change and add this feature ?

K2Death
K2Death previously approved these changes May 3, 2021
Copy link

@K2Death K2Death left a comment

Choose a reason for hiding this comment

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

Working in Prod env.

a03nikki
a03nikki previously approved these changes May 26, 2021
Copy link

@a03nikki a03nikki left a comment

Choose a reason for hiding this comment

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

LGTM but I don't feel comfortable signing off as the official approval for this pull request.

@a03nikki a03nikki dismissed their stale review May 26, 2021 14:19

I'm not an expert in Ansible module writing.

@K2Death
Copy link

K2Death commented Jun 11, 2021

Hi , any news for this PR ?
Regards

tasks/beats-config.yml Outdated Show resolved Hide resolved
tasks/beats-config.yml Outdated Show resolved Hide resolved
@botelastic
Copy link

botelastic bot commented Jan 11, 2022

This PR has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
To track this PR (even if closed), please open a corresponding issue if one does not already exist.

@devmills
Copy link

@ladislavmacoun please don't let this go stale. What needs to happen next?

@botelastic botelastic bot removed the triage/stale label Jan 13, 2022
@ladislavmacoun
Copy link

@ladislavmacoun please don't let this go stale. What needs to happen next?

Sorry @devmills, I can't help you with this since I am not a member of elastic team, also we have moved from the official beat role and not using this anymore.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants