From 66800900843747f37591b077091dd2c8cf2c1796 Mon Sep 17 00:00:00 2001 From: Zach Renner <13670625+zarenner@users.noreply.github.com> Date: Thu, 12 Feb 2026 05:46:48 -0800 Subject: [PATCH] Remove unnecessary connection test during some registration flows (#4244) --- src/Runner.Listener/Configuration/ConfigurationManager.cs | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/Runner.Listener/Configuration/ConfigurationManager.cs b/src/Runner.Listener/Configuration/ConfigurationManager.cs index 2c18d8d60..360728a1c 100644 --- a/src/Runner.Listener/Configuration/ConfigurationManager.cs +++ b/src/Runner.Listener/Configuration/ConfigurationManager.cs @@ -178,8 +178,12 @@ namespace GitHub.Runner.Listener.Configuration } } - // Validate can connect. - await _runnerServer.ConnectAsync(new Uri(runnerSettings.ServerUrl), creds); + // Validate can connect using the obtained vss credentials. + // In Runner Admin flow there's nothing new to test connection to at this point as registerToken is already validated via GetTenantCredential. + if (!runnerSettings.UseRunnerAdminFlow) + { + await _runnerServer.ConnectAsync(new Uri(runnerSettings.ServerUrl), creds); + } _term.WriteLine(); _term.WriteSuccessMessage("Connected to GitHub");