Skip to content

Commit e94290d

Browse files
fix: handle llm handlers log setup gracefully (#668)
Co-authored-by: Maryam Khidir <[email protected]>
1 parent 75c5cad commit e94290d

File tree

2 files changed

+13
-9
lines changed

2 files changed

+13
-9
lines changed

aws-genai-llm-chatbot/modules/chatbot/deployspec.yaml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,11 @@ deploy:
66
- npx @aws-amplify/cli codegen add --yes
77
build:
88
commands:
9+
- mkdir -p /root/.cdk/cache
910
- env | grep NEXUS | sort | awk -F= '{printf "%-30s = %s\n", $1, $2}'
1011
- npm run tsc
1112
- npm run config -- --non-interactive --env-prefix NEXUS_PARAMETER_
12-
- npm run deploy -- --require-approval never --progress events --outputs-file ./cdk-exports.json -v
13+
- npm run deploy -- --require-approval never --progress events --outputs-file ./cdk-exports.json -v --no-notices
1314
- cat cdk-exports.json
1415
- seedfarmer metadata convert -f cdk-exports.json -jq .${NEXUS_PARAMETER_PREFIX}GenAIChatBotStack.metadata || true
1516
destroy:

lib/monitoring/index.ts

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -97,15 +97,18 @@ export class Monitoring extends Construct {
9797
])
9898
)
9999
);
100-
monitoring.addSegment(
101-
new SingleWidgetDashboardSegment(
102-
this.getLogsWidget(
103-
"LLM Request Handlers Logs:",
104-
props.llmRequestHandlersLogGroups,
105-
[]
100+
// Only add LLM Request Handlers Logs widget if there are log groups
101+
if (props.llmRequestHandlersLogGroups.length > 0) {
102+
monitoring.addSegment(
103+
new SingleWidgetDashboardSegment(
104+
this.getLogsWidget(
105+
"LLM Request Handlers Logs:",
106+
props.llmRequestHandlersLogGroups,
107+
[]
108+
)
106109
)
107-
)
108-
);
110+
);
111+
}
109112

110113
if (props.advancedMonitoring) {
111114
this.addMetricFilter(

0 commit comments

Comments
 (0)