diff --git a/src/Runner.Listener/JobDispatcher.cs b/src/Runner.Listener/JobDispatcher.cs index 7913c9228..1cc37fec2 100644 --- a/src/Runner.Listener/JobDispatcher.cs +++ b/src/Runner.Listener/JobDispatcher.cs @@ -666,8 +666,15 @@ namespace GitHub.Runner.Listener { try { +#if USE_BROKER + if (!firstJobRequestRenewed.Task.IsCompleted) + { + // fire first renew succeed event. + firstJobRequestRenewed.TrySetResult(0); + } +#else request = await runnerServer.RenewAgentRequestAsync(poolId, requestId, lockToken, orchestrationId, token); - Trace.Info($"Successfully renew job request {requestId}, job is valid till {request.LockedUntil.Value}"); + Trace.Info($"Successfully renew job request {requestId}, job is valid till {request?.LockedUntil.Value}"); if (!firstJobRequestRenewed.Task.IsCompleted) { @@ -677,6 +684,7 @@ namespace GitHub.Runner.Listener // Update settings if the runner name has been changed server-side UpdateAgentNameIfNeeded(request.ReservedAgent?.Name); } +#endif if (encounteringError > 0) { diff --git a/src/Runner.Worker/ActionManager.cs b/src/Runner.Worker/ActionManager.cs index 954d9a691..54a49814d 100644 --- a/src/Runner.Worker/ActionManager.cs +++ b/src/Runner.Worker/ActionManager.cs @@ -817,6 +817,7 @@ namespace GitHub.Runner.Worker { // Something else bad happened, let's go to our retry logic response.EnsureSuccessStatusCode(); + throw new Exception("Unexpected response code: " + response.StatusCode); } } }