Skip to content

Commit 22b280d

Browse files
committed
fix: remove server cleanup on failure, use project logger for ELO
1 parent f183bcc commit 22b280d

2 files changed

Lines changed: 6 additions & 11 deletions

File tree

src/dedicated-servers/dedicated-servers.service.ts

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -345,17 +345,11 @@ export class DedicatedServersService {
345345
await this.pingDedicatedServer(serverId);
346346
}, 10000);
347347
})
348-
.catch(async (error) => {
348+
.catch((error) => {
349349
this.logger.error(
350-
`[${serverId}] error waiting for pod to be ready, recovering server state`,
350+
`[${serverId}] error waiting for pod to be ready`,
351351
error,
352352
);
353-
await this.removeDedicatedServer(serverId).catch((removeError) => {
354-
this.logger.error(
355-
`[${serverId}] failed to clean up server after pod readiness failure`,
356-
removeError,
357-
);
358-
});
359353
});
360354

361355
return true;

src/matches/jobs/EloCalculation.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,10 @@ import { PostgresService } from "../../postgres/postgres.service";
77

88
@UseQueue("Matches", MatchQueues.EloCalculation)
99
export class EloCalculation extends WorkerHost {
10-
private readonly logger = new Logger(EloCalculation.name);
11-
12-
constructor(private readonly postgres: PostgresService) {
10+
constructor(
11+
private readonly logger: Logger,
12+
private readonly postgres: PostgresService,
13+
) {
1314
super();
1415
}
1516

0 commit comments

Comments
 (0)