Skip to content

Commit

Permalink
fix: change log level for termination watcher if no match (#3866)
Browse files Browse the repository at this point in the history
Changed log level to debug in case no match. Deploying the lambda
multiple times in a single aws account will create unrelated warning
logs otherwise.
  • Loading branch information
npalm authored Apr 25, 2024
1 parent 8e47807 commit 649ad35
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion lambdas/functions/termination-watcher/src/lambda.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ import { interruptionWarning } from './lambda';
import { SpotInterruptionWarning, SpotTerminationDetail } from './types';

jest.mock('./termination-warning');
//jest.mock('@terraform-aws-github-runner/aws-powertools-util');

process.env.POWERTOOLS_METRICS_NAMESPACE = 'test';
process.env.POWERTOOLS_TRACE_ENABLED = 'true';
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ async function handle(event: SpotInterruptionWarning<SpotTerminationDetail>, con
);
}
} else {
logger.warn(
logger.debug(
`Received spot termination notification warning for instance ${event.detail['instance-id']} but ` +
`details are not available or instance not matching the tag fileter (${config.tagFilters}).`,
);
Expand Down

0 comments on commit 649ad35

Please sign in to comment.