Karl Isenberg

> ResourceGroup is in the same namespace as the RepoSync. It would be deleted as well. I stand corrected. You're right, only the RootSync ResourceGroups are in the `config-management-system`. In...

I don't really understand why y'all are in a hurry to get this half-fix in with side effects, for an issue that's existed for years, right before a code freeze,...

/lgtm I suspect we'll eventually need to revert this later when we add a more general solution that works for deletion dependencies and RootSyncs, but for now it seems like...

/lgtm

Example memory use during TestStressMemoryUsageOCI on GKE Autopilot: 

All of that is of course true. But tracking object count is a lot easier than tracking memory and CPU usage. This is partly why I didn't make the steps...

Here's a shot of the usage and limits with this prototype while running `--test-features=reconciliation-2` e2e tests, which don't sync a lot of files. You can see that the 256Mi reconciler...

As another datapoint, here is TestStressManyDeployments on Autopilot, which uses ~300MiB memory to deploy 1,000 Deployments. 

Here's a new test called TestStressManyRootSyncs that deploys the same number of 1,000 Deployments, but uses 10 RootSyncs to do it, 100 each. You can see that with the autoscaling,...

Here's the same test with 2k deployments (10x200). Memory use went up, but not by 100%, more like 70% (180Mi -> 260Mi).