Skip to content

Commit

Permalink
fix(lambda): add dashboard widgets properly (#576)
Browse files Browse the repository at this point in the history
Fixes regression from #574.

---

_By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license_
  • Loading branch information
echeung-amzn authored Sep 24, 2024
1 parent 5434499 commit 6b78967
Show file tree
Hide file tree
Showing 4 changed files with 240 additions and 11 deletions.
1 change: 1 addition & 0 deletions lib/monitoring/aws-lambda/LambdaFunctionMonitoring.ts
Original file line number Diff line number Diff line change
Expand Up @@ -612,6 +612,7 @@ export class LambdaFunctionMonitoring extends Monitoring {
),
);
}
widgets.push(secondRow);

if (this.lambdaInsightsEnabled) {
widgets.push(
Expand Down
38 changes: 37 additions & 1 deletion test/facade/__snapshots__/MonitoringAspect.test.ts.snap

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

14 changes: 13 additions & 1 deletion test/facade/__snapshots__/MonitoringFacade.test.ts.snap

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 6b78967

Please sign in to comment.