Skip to content

Commit

Permalink
Merge pull request #1055 from wish-oss/fix/console-error
Browse files Browse the repository at this point in the history
refactor: remove unnecessary error logging in GPU setup functions
  • Loading branch information
Siumauricio authored Jan 2, 2025
2 parents 41d4ff8 + 49a6b72 commit 65361d1
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions packages/server/src/utils/gpu-setup.ts
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@ export async function checkGPUStatus(serverId?: string): Promise<GPUInfo> {
...cudaInfo,
};
} catch (error) {
console.error("Error in checkGPUStatus:", error);
return {
driverInstalled: false,
driverVersion: undefined,
Expand Down Expand Up @@ -317,7 +316,6 @@ const setupLocalServer = async (daemonConfig: any) => {
try {
await execAsync(setupCommands);
} catch (error) {
console.error("Setup failed:", error);
throw new Error(
"Failed to configure GPU support. Please ensure you have sudo privileges and try again.",
);
Expand All @@ -344,11 +342,10 @@ const verifySetup = async (nodeId: string, serverId?: string) => {
"cat /etc/nvidia-container-runtime/config.toml",
].join(" && ");

const { stdout: diagnostics } = serverId
? await execAsyncRemote(serverId, diagnosticCommands)
: await execAsync(diagnosticCommands);
await (serverId
? execAsyncRemote(serverId, diagnosticCommands)
: execAsync(diagnosticCommands));

console.error("Diagnostic Information:", diagnostics);
throw new Error("GPU support not detected in swarm after setup");
}

Expand Down

0 comments on commit 65361d1

Please sign in to comment.