Skip to content

[7.x](backport #27860) [Elastic Agent] Disable modules.d in metricbeat#27993

Merged
ruflin merged 1 commit into7.xfrom
mergify/bp/7.x/pr-27860
Sep 20, 2021
Merged

[7.x](backport #27860) [Elastic Agent] Disable modules.d in metricbeat#27993
ruflin merged 1 commit into7.xfrom
mergify/bp/7.x/pr-27860

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Sep 20, 2021

This is an automatic backport of pull request #27860 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.io/

Currently when Elastic Agent is started, on post-install of metricbeat the system.yml file in the modules.d directoy is renamed to system.yml.disabled to make sure it is not run. It turns out there might be some edge cases where this does not work. To not rely on renaming of files, instead the modules are fully disabled.

Closes #27857

(cherry picked from commit 41f17ab)
@mergify mergify bot added the backport label Sep 20, 2021
@mergify mergify bot assigned ruflin Sep 20, 2021
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Sep 20, 2021
@botelastic
Copy link
Copy Markdown

botelastic bot commented Sep 20, 2021

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

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-09-20T06:16:25.590+0000

  • Duration: 89 min 50 sec

  • Commit: 5da769f

Test stats 🧪

Test Results
Failed 0
Passed 6996
Skipped 24
Total 7020

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 6996
Skipped 24
Total 7020

@ruflin ruflin merged commit 51d7ee1 into 7.x Sep 20, 2021
@mergify mergify bot deleted the mergify/bp/7.x/pr-27860 branch September 20, 2021 11:25
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.

2 participants