Skip to content

Commit 61845e1

Browse files
authored
chore: add additional logging for worker status (#1020)
## Changes Adding additional logging for redis connection error and worker status for `closed` and `ready`. For debugging `failed to refresh slots cache` error.
1 parent 52663b1 commit 61845e1

File tree

4 files changed

+27
-0
lines changed

4 files changed

+27
-0
lines changed

packages/backend/src/config/redis.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
import ioRedis from 'ioredis'
22

3+
import logger from '@/helpers/logger'
4+
35
import appConfig from './app'
46

57
// Maximum of 16; be careful when adding!
@@ -13,6 +15,7 @@ export const REDIS_DB_INDEX = {
1315

1416
function reconnectOnError(err: Error) {
1517
const targetError = 'READONLY'
18+
logger.error('Redis connection error', err)
1619
if (err.message.includes(targetError)) {
1720
// Only reconnect when the error contains "READONLY"
1821
// during node failover, this is thrown: 149: -READONLY You can't write against a read only replica.

packages/backend/src/workers/flow.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,14 @@ worker.on('failed', (job, err) => {
6767
)
6868
})
6969

70+
worker.on('ready', () => {
71+
logger.info('Flow worker is ready!')
72+
})
73+
74+
worker.on('closed', () => {
75+
logger.info('Flow worker is closed!')
76+
})
77+
7078
worker.on('error', (err) => {
7179
if (!err) {
7280
logger.error('Worker undefined error')

packages/backend/src/workers/helpers/make-action-worker.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -278,6 +278,14 @@ export function makeActionWorker(
278278
}
279279
})
280280

281+
worker.on('ready', () => {
282+
logger.info(`[${queueName}] Worker is ready!`)
283+
})
284+
285+
worker.on('closed', () => {
286+
logger.info(`[${queueName}] Worker is closed!`)
287+
})
288+
281289
worker.on('error', (err) => {
282290
if (!err) {
283291
logger.error(`[${queueName}] Worker had undefined error`)

packages/backend/src/workers/trigger.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,14 @@ worker.on('failed', (job, err) => {
6666
)
6767
})
6868

69+
worker.on('ready', () => {
70+
logger.info('Trigger worker is ready!')
71+
})
72+
73+
worker.on('closed', () => {
74+
logger.info('Trigger worker is closed!')
75+
})
76+
6977
worker.on('error', (err) => {
7078
if (!err) {
7179
logger.error('Worker undefined error')

0 commit comments

Comments
 (0)