Skip to content

[Auditbeat] Cherry-pick #10018 to 6.6: Change module configuration from "metricsets" to "datasets"#10050

Merged
cwurm merged 1 commit intoelastic:6.6from
cwurm:backport_10018_6.6
Jan 14, 2019
Merged

[Auditbeat] Cherry-pick #10018 to 6.6: Change module configuration from "metricsets" to "datasets"#10050
cwurm merged 1 commit intoelastic:6.6from
cwurm:backport_10018_6.6

Conversation

@cwurm
Copy link
Copy Markdown
Contributor

@cwurm cwurm commented Jan 14, 2019

Cherry-pick of PR #10018 to 6.6 branch. Original message:

This changes the Auditbeat system module configuration to use "datasets" rather than "metricsets" to configure its sub-modules:

- module: system
  datasets:
    - host
    - process
    [...]

It simply overwrites the MetricSets field in the default Metricbeat configuration with the value of the datasets configuration.

@andrewkroh We talked about the difficulty of making changes to go-ucfg or to the Metricbeat code. Turns out there is no need to change anything under metricbeat/, so I think this might be ok? What do you think? I think in the long term we might still want to change to a fileset-like configuration, but for now this would avoid having a mix of "dataset" and "metricset" in anything user-facing, i.e. documentation or configuration.

Follow-up:

  1. Change documentation and other places where "metricset(s)" is used

…s" (elastic#10018)

Changes the Auditbeat system module configuration to use "datasets" rather than "metricsets" to configure its sub-modules:

```
- module: system
  datasets:
    - host
    - process
    [...]
```

(cherry picked from commit e42ccb4)
@cwurm cwurm changed the title Cherry-pick #10018 to 6.6: [Auditbeat] Change module configuration from "metricsets" to "datasets" [Auditbeat] Cherry-pick #10018 to 6.6: Change module configuration from "metricsets" to "datasets" Jan 14, 2019
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/secops

@cwurm cwurm requested review from andrewkroh and tsg January 14, 2019 15:11
@cwurm cwurm merged commit 7884b9c into elastic:6.6 Jan 14, 2019
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
…tion from "metricsets" to "datasets" (elastic#10050)

Cherry-pick of PR elastic#10018 to 6.6 branch. Original message: 

Changes the Auditbeat system module configuration to use "datasets" rather than "metricsets" to configure its sub-modules:

```
- module: system
  datasets:
    - host
    - process
    [...]
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants