-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[8.x] [UII] Advanced agent monitoring options UI for HTTP endpoint and diagnostics (#193361) #193658
[8.x] [UII] Advanced agent monitoring options UI for HTTP endpoint and diagnostics (#193361) #193658
Conversation
…nostics (elastic#193361) ## Summary Resolves elastic#153950. This PR implements a UI to configure advanced Elastic Agent monitoring options under agent policy settings. These advanced options include enabling HTTP monitoring endpoints and various options for agent diagnostics. They are shown under an a toggle under the existing agent monitoring logs and metrics collection options: <img width="1326" alt="image" src="https://github.com/user-attachments/assets/ac8cbe00-d838-4c9a-8a35-3dbf31222dc9"> If the base HTTP monitoring endpoint is not enabled, the rest of the HTTP options are disabled: <img width="1328" alt="image" src="https://github.com/user-attachments/assets/2eac787c-3055-4862-b3eb-2566a39ee86c"> The following new fields are added to agent policy schema to support this: ``` monitoring_http monitoring_pprof_enabled monitoring_diagnostics ``` This work supersedes the previous `HTTP monitoring endpoint` options under `Advanced Settings` at the bottom of the page. Any previous configuration under an agent policy's `advanced_settings.agent_monitoring_http` saved object field are migrated over to the new `monitoring_http` field and the old field is deleted. See the migration fn `backfillAgentPolicyToV4`. These new options are compiled to agent yaml like this: ```yml agent: monitoring: enabled: true use_output: default logs: true metrics: true traces: true namespace: default pprof: enabled: true http: enabled: true host: localhost port: 6791 diagnostics: limit: interval: 1m burst: 1 uploader: max_retries: 10 init_dur: 1s max_dur: 10m ``` Summarize your PR. If it involves visual changes include a screenshot or gif. ### To-do - [x] API integration tests - [x] Full manual test of SO migration - [x] Full manual test with agent using these settings ### Checklist Delete any items that are not applicable to this PR. - [x] Any text added follows [EUI's writing guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses sentence case text and includes [i18n support](https://github.com/elastic/kibana/blob/main/packages/kbn-i18n/README.md) - [ ] [Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html) was added for features that require explanation or tutorials - elastic/ingest-docs#1333 - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios --------- Co-authored-by: kibanamachine <[email protected]> (cherry picked from commit 87cdc2d)
Pinging @elastic/fleet (Team:Fleet) |
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Module Count
Public APIs missing comments
Async chunks
Page load bundle
To update your PR or re-run it, just comment with: cc @jen-huang |
Backport
This will backport the following commits from
main
to8.x
:Questions ?
Please refer to the Backport tool documentation