-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Observability Onboarding] Split Agent config into multiple files #191241
[Observability Onboarding] Split Agent config into multiple files #191241
Conversation
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
…nboarding-agent-config
@@ -1229,7 +1229,7 @@ | |||
"suricata-sid-db": "^1.0.2", | |||
"swr": "^2.2.5", | |||
"symbol-observable": "^1.2.0", | |||
"tar": "^6.2.1", | |||
"tar": "^7.4.3", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had to update this package to the latest version since v6 did not export the required lower level functions.
@@ -1609,7 +1609,6 @@ | |||
"@types/styled-components": "^5.1.0", | |||
"@types/supertest": "^6.0.2", | |||
"@types/tapable": "^1.0.6", | |||
"@types/tar": "^6.1.11", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
v7 ships with type definitions so was able to remove @types/tar
package.
Pinging @elastic/fleet (Team:Fleet) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally, everything works as expected ✨
/ci |
1 similar comment
/ci |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
💛 Build succeeded, but was flaky
Failed CI StepsMetrics [docs]
History
To update your PR or re-run it, just comment with: |
Resolves #191917
Summary
Splits up the Agent config generated during auto-detect based onboarding into multiple files.
This makes it easier for users to make changes following the initial onboarding.
The backup feature has also been updated to include files inside
inputs.d
directory.Example
Before this change the auto-detect script would write all settings to
elastic-agent.yml
file.After this change the script writes one separate config file for each integrations that was detected, for example:
elastic-agent.yml
- Contains global settingsinputs.d/system.yml
- Contains inputs config for System integrationinputs.d/docker.yml
- Contains inputs config for Docker integrationScreenshot