-
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
[Research] Ability for alert / action types to define migrations #50216
Comments
Pinging @elastic/kibana-stack-services (Team:Stack Services) |
Some additional comments in issue #66608 |
Linking with #84907 as it could help us here. |
+1 here from the Security Detections side. It would be beneficial for us to be able to leverage dedicated SO migrations for Detection Rules (and not having to embed the migration within Alerting). A recent example here is in needing to find all Detection Rules that reference a set of Exception Lists, whose reference is currently being stored in the Rule's AlertParams and cannot be searched. We don't currently have a back reference on our Exception Lists, so one solution would be to move the Rule's Exception List reference from AlertParams to internal tags within an SO migration (or leverage the existing SO references array). |
Moving from |
@joshdover @ppisljar - This looks like another use case for Persistable State. This is an interesting one though because it appears that the registrar can define mappings on the saved object. I was hoping we could get away with just leveraging "migrateToLatest" functionality rather than the SO-required object mapping (of version to migrate function), but because of this use case, we may need to continue to support both. |
Closing, as things have changed SOOO much since this was created! In particular, our newest constraints surrounding rule/connector data shape changes is reflected in 2024-01 Requiring intermediate releases to change rule or connector schemas in serverless |
hahahaha - I just noticed Mike assigned this to JiaweiWu YESTERDAY hahahahaha ... so I'll re-open, it would be good to get some prose from @mikecote on what we're expecting here ... |
😁 We'll need to find a way using the new migration strategy to allow rule types to migrate their parameters so they can have the rule type running with the latest parameters at all times. It will simplify the codebase for teams as they don't have to migrate on the fly anywhere they access an object. Jiawei will create a PoC and assess our options 👍 |
In scenario they changed the parameters for their alert / action type.
In the scenario we added
name
to alert.This will be needed once we support mappings for config, params, etc
The text was updated successfully, but these errors were encountered: