[Alerting] [7.x] Convert all savedObjectClient.get() calls to use savedObjectClient.resolve and handle the response appropriately

[chrisronline]

Summary

In #100067, we need to ensure that all rule and connector saved objects are shareable. Part of this effort involves needing to regenerate saved object IDs to guarantee uniqueness across all spaces (which is currently not guaranteed). This affects existing saved objects, as every saved object will receive a new ID as part of this effort and the new .resolve() saved objects API is designed to give insight into that process for a specific saved object.

This new API not only returns the saved object (it will attempt to find the actual saved object even if the consumer provides the outdated ID) but also information on the new and old saved object IDs and based on the response, plugin owners are expected to provide a quality UX and let the user know of any actions they need to take (such as updating bookmarks, etc). (#95958 is a good reference PR for the expected type of change)

As a result of the above, various plugin's server code needs to convert to using this new saved object .resolve API. We also need to ensure the UI handles showing the user an appropriate message regarding this situation. Luckily, the core/security team built out shareable React components to do that and a good example of how is here

For the sake of consistency, I recommend all .get() calls change to .resolve() (instead of attempting to only find the places where we need to change to .resolve()). It's unclear when/if the .resolve() API deprecates, but if/when that happens, we can change all .resolve() back to .get() in one move.

List of affected areas of code

As a result of the above, the alerting plugin's server code needs to convert to using this new saved object .resolve API. Our PoC PR revealed a few places where we need to do this:

• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/invalidate_pending_api_keys/task.ts#L200
• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/rules_client/rules_client.ts#L382
• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/rules_client/rules_client.ts#L615
• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/rules_client/rules_client.ts#L694
• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/rules_client/rules_client.ts#L705
• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/rules_client/rules_client.ts#L873
• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/rules_client/rules_client.ts#L992
• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/rules_client/rules_client.ts#L1096
• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/rules_client/rules_client.ts#L1110
• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/rules_client/rules_client.ts#L1183
• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/rules_client/rules_client.ts#L1245
• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/rules_client/rules_client.ts#L1307
• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/rules_client/rules_client.ts#L1375
• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/task_runner/task_runner.ts#L120

We also need to handle the situation where a stored task contains a reference to an outdated saved object id, specifically the alert id, such as:

• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/task_runner/task_runner.ts#L244
• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/task_runner/task_runner.ts#L430
• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/task_runner/task_runner.ts#L457
• https://github.com/elastic/kibana/blob/master/x-pack/plugins/alerting/server/task_runner/task_runner.ts#L495

There may be more places, and changes to the above listed files might mean additional changes to callers of the above code paths.

When and backporting

This work needs to be done before 8.0. Most of the work can be backported as well. Because #107084 will handle changing the namespaceType, this PR will just need to ensure the .resolve() changes do not break existing behavior.

[elasticmachine]

Pinging @elastic/kibana-alerting-services (Team:Alerting Services)

[chrisronline]

I'm going to close this out, as the original research is invalid and need to be reworked. If this item is still applicable, we still reopen this ticket.