ignore errProcessInfoMissing when getting process RSS

[znull]

We have spikes of error getting RSS errors setting off the gitrpcd/dgit-update-status monitor. That shouldn't be failing 3pc.

Another query

[Copilot]

Pull request overview

This PR addresses error spikes in the gitrpcd/dgit-update monitor by ignoring errProcessInfoMissing errors when polling for process RSS (Resident Set Size) memory usage. The error occurs when cmd.Process is nil, which can happen legitimately during normal process lifecycle (e.g., during startup or shutdown). By ignoring this expected error condition, the PR prevents false alarms that were causing 3% failure rates.

Changes:

• Modified error handling in killAtLimit to ignore errProcessInfoMissing errors when getting RSS

Comments suppressed due to low confidence (1)

pipe/memorylimit.go:61

• The same errProcessInfoMissing error handling should also be applied in the logMaxRSS function at line 138. When GetRSSAnon returns errProcessInfoMissing, it shouldn't be counted as an error or trigger error logging in MemoryObserver either, for the same reason it's being ignored in killAtLimit. The process may legitimately not exist yet when polling starts.

				if err != nil && err != errProcessInfoMissing {

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.