Add run IDs to the list of runs in grpcrunlauncher used for test cleanup, even if the launch failed
This is important because we use this list for test cleanup to wait for any servers that launched runs (to avoid tempdir contention issues). So tests in which a launch failed might fail to wait for the server that launched the failed run to die before tearing down, causing flakiness (https://dev.azure.com/elementl/dagster/_build/results?buildId=9742&view=logs&j=97c9b631-cb7a-5c89-c2a5-acb43d4227bd&t=256f5766-8906-5a25-01de-b83c5e4e5216).
That said, we should really look into removing this cleanup logic from the run launcher and putting it somewhere better.
Test Plan: BK + Azure
Reviewers: alangenfeld, max, prha, schrockn
Reviewed By: max
Differential Revision: https://dagster.phacility.com/D4968