Right now, on Windows, the new consoleusers.CurrentUids implementation added in #2422 is incorrectly identifying additional users that we should start desktop processes for. This means that the desktop runner is, every five seconds, trying to start a desktop process for these users, which means it calls consoleusers.ExplorerProcess and enumerates every single process. This is very noisy in the logs, and we suspect there's a performance impact. (This issue will, though, be invisible to the end user otherwise.)
For the next release:
Over the next two releases:
In the long term:
- Investigate subscribing to logon events so that we don't have to run this check quite so frequently (or maybe we can phase it out altogether)
Right now, on Windows, the new
consoleusers.CurrentUidsimplementation added in #2422 is incorrectly identifying additional users that we should start desktop processes for. This means that the desktop runner is, every five seconds, trying to start a desktop process for these users, which means it callsconsoleusers.ExplorerProcessand enumerates every single process. This is very noisy in the logs, and we suspect there's a performance impact. (This issue will, though, be invisible to the end user otherwise.)For the next release:
consoleusers.CurrentUids): Exclude some sessions from list of current UIDs #2465Over the next two releases:
DesktopUpdateInterval#2462, so that we can increase the interval on the cloud sideconsoleusers.ExplorerProcessthat doesn't require enumerating every single processconsoleusers.CurrentUidsimplementation on macOS and LinuxIn the long term: