istio icon indicating copy to clipboard operation
istio copied to clipboard

More Ambient analyzers

Open Stevenjin8 opened this issue 1 year ago • 2 comments

This includes

  • #50249
  • #52602

Stevenjin8 avatar Aug 15 '24 15:08 Stevenjin8

With Ambient mode (and maybe even future multicluster analysis can be affected by this) there is a need to do things involving the Analyzer that are different than what we've done in the past. We will need to spend time thinking about and refactoring code to support these new expectations for this Istio feature. This will include refactoring the way we:

  • retrieve data for the Analyzer; might need to come from different sources (e.g. for Waypoints this will mean wds) as opposed to just expecting all data to come from k8s static resources. (examples of analyzers that need this and some related issues: #50249, #50849, #24345)
  • write status with the Analyzer; there are multiple ways so one consideration could be to refactor writing status to be more uniform and allow individual status writing controllers to have their own queues to enqueue status writing, with all status writing controllers leveraging a shared worker pool with a globally set limit to prevent overloading the system (as discussed here #52602)

MorrisLaw avatar Aug 15 '24 16:08 MorrisLaw

🚧 This issue or pull request has been closed due to not having had activity from an Istio team member since 2024-08-15. If you feel this issue or pull request deserves attention, please reopen the issue. Please see this wiki page for more information. Thank you for your contributions.

Created by the issue and PR lifecycle manager.

istio-policy-bot avatar Nov 29 '24 05:11 istio-policy-bot