-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[FLEET][SECURITY_SOLUTION] Fix .fleet-artifacs index property names and add endpoint package policy migrations #94977
[FLEET][SECURITY_SOLUTION] Fix .fleet-artifacs index property names and add endpoint package policy migrations #94977
Conversation
…13-policy-migration # Conflicts: # x-pack/plugins/fleet/server/saved_objects/migrations/to_v7_13_0.ts
migrationContext | ||
) => { | ||
// Endpoint specific migrations | ||
// FIXME:PT remove `-OFF` from below once ready to be released |
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.
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.
Is the only place in which needs to fork based on that? Are there other options for detecting/inferring Fleet Server?
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 am okay to bring back the flag dependings on the timing of Fleet server for 7.13 we may have to hide some UI too, so as a temporary workaround it's probably our best solution
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.
@jfsiii I don't know the answers there.
For endpoint, I think this is the only branching we need in fleet - we already have security_solution specific flag in our code to handle one other branch.
@nchaulet thanks. I will add back in with another PR. I want this one to merge since it corrects the index property definition.
Pinging @elastic/fleet (Team:Fleet) |
Pinging @elastic/security-onboarding-and-lifecycle-mgt (Team:Onboarding and Lifecycle Mgt) |
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.
Only able to read on mobile at the moment. Left two small comments but otherwise 👍🏻
Will review more deeply on my computer later
💛 Build succeeded, but was flaky
Test FailuresKibana Pipeline / general / Closes and opens alerts.Closing alerts Closes and opens alertsStack Trace
Kibana Pipeline / general / "before all" hook for "should open a modal".Open timeline Open timeline modal "before all" hook for "should open a modal"Stack Trace
Metrics [docs]
History
To update your PR or re-run it, just comment with: |
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.
🚀
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.
👍
Summary
.fleet-artifacts
index so that property names are snake_cased instead of camleCased (change rippled through a few types, but contained internally to the artifacts code)Click here to see output of updated index definition
Checklist