Skip to content

[8.0](backport #28769) Change "filebeat.config.modules.enabled" to "true"#29111

Merged
dedemorton merged 1 commit into8.0from
mergify/bp/8.0/pr-28769
Nov 24, 2021
Merged

[8.0](backport #28769) Change "filebeat.config.modules.enabled" to "true"#29111
dedemorton merged 1 commit into8.0from
mergify/bp/8.0/pr-28769

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Nov 23, 2021

This is an automatic backport of pull request #28769 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

* Change filebeat.config.modules.enabled to true

As per https://www.elastic.co/guide/en/beats/filebeat/current/filebeat-configuration-reloading.html#load-module-config, the default value of filebeat.config.modules.enabled is `true`. Having it as `false` in filebeat.reference.yml would bring in confusion.

* Fix build error

Co-authored-by: dedemorton <dede.morton@elastic.co>
(cherry picked from commit 3e84736)
@mergify mergify bot added the backport label Nov 23, 2021
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Nov 23, 2021
@botelastic
Copy link
Copy Markdown

botelastic bot commented Nov 23, 2021

This pull request doesn't have a Team:<team> label.

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

  • /package : Generate the packages and run the E2E tests.

  • /beats-tester : Run the installation tests with beats-tester.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@dedemorton dedemorton merged commit b240b76 into 8.0 Nov 24, 2021
@dedemorton dedemorton deleted the mergify/bp/8.0/pr-28769 branch November 24, 2021 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport needs_team Indicates that the issue/PR needs a Team:* label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants