Skip to content

Commit 446da4e

Browse files
authored
Reduce verbosity of logging messages (#35)
1 parent 7b7f9e6 commit 446da4e

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

packages/k8s/src/hooks/run-script-step.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,6 @@ async function runScriptStepWithGRPC(
3535
args: RunScriptStepArgs,
3636
state
3737
): Promise<void> {
38-
const runnerDir = `/home/runner/_work/_temp/_runner_file_commands`
39-
const files = fs.readdirSync('/home/runner/_work/_temp/_runner_file_commands')
40-
4138
const { entryPoint, entryPointArgs, environmentVariables } = args
4239
const scriptContent = getEntryPointScriptContent(
4340
args.workingDirectory,
@@ -76,10 +73,11 @@ async function runScriptStepWithGRPC(
7673
break
7774
} catch (err) {
7875
const message = extractErrorMessageFromK8sError(err)
79-
core.warning(`ScriptExecutorError when trying to execute: ${message}`)
8076
if (message.includes('ECONNREFUSED') || message.includes('UNAVAILABLE')) {
8177
// Retry for 60s since the service may not be established.
82-
core.warning(`Retrying execution for ECONNREFUSED or UNAVAILABLE.`)
78+
core.warning(
79+
`Retrying execution for ECONNREFUSED or UNAVAILABLE ${message}.`
80+
)
8381
await backOffmanager.backOff()
8482
} else {
8583
throw new Error(

packages/k8s/src/k8s/utils.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -407,9 +407,7 @@ export async function runScriptByGrpc(
407407
if (exitCode === 0) {
408408
resolve()
409409
} else {
410-
reject(
411-
new Error(`Command ${command} failed with exit code ${exitCode}.`)
412-
)
410+
reject(new Error(`Job failed with exit code ${exitCode}.`))
413411
}
414412
})
415413

0 commit comments

Comments
 (0)