-
Notifications
You must be signed in to change notification settings - Fork 69
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
Add custom settings page #962
Conversation
A documentation preview will be available soon. Request a new doc build by commenting
If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here. |
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.
The content looks good, added a few suggestions.
Thanks @juliaElastic! Nice catches. I've addressed all comments in cfff910. |
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
* Add custom settings page * Fixup * Fix whitespace; replace policy ID with placeholder (cherry picked from commit 4106015)
* Add custom settings page * Fixup * Fix whitespace; replace policy ID with placeholder (cherry picked from commit 4106015)
* Add custom settings page * Fixup * Fix whitespace; replace policy ID with placeholder (cherry picked from commit 4106015) Co-authored-by: David Kilfoyle <[email protected]>
* Add custom settings page * Fixup * Fix whitespace; replace policy ID with placeholder (cherry picked from commit 4106015) Co-authored-by: David Kilfoyle <[email protected]>
As requested in #954, this adds examples of agent policy overrides. I've included the warning proposed by Luca here, that it's possible for a custom configuration to break things.
Please see docs preview page
Closes: #954