-
Notifications
You must be signed in to change notification settings - Fork 19
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
update transforms to be unattended #353
Conversation
Does fleet respect this |
yes and yes. fleet passes the entire transform body to ES without any validation so don't need to add anything there. this won't change the transform functionally. this just tells ES to manage the transform's health and to always retry if in failing state: https://www.elastic.co/guide/en/elasticsearch/reference/current/put-transform.html (sorry no anchors on page, have to search for |
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.
looks good.
I bet the linters are complaining about the final newline in the file. Should pass if you remove it
c17da95
to
04d930b
Compare
## Summary Resolves #150343. Similar to #153612, mitigated by #152119 and elastic/endpoint-package#353. Flaky test runner x 200 ✅ https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/2041
Change Summary
Update current + united metadata transforms to be
settings.unattended: true
.Related: https://github.com/elastic/security-team/issues/6165
Release Target
8.8
For Transform changes: