Skip to content

Commit

Permalink
Ordered the dashboard widgets to be orginized
Browse files Browse the repository at this point in the history
  • Loading branch information
Cameronsplaze committed Oct 13, 2024
1 parent 2f39443 commit bf03722
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -184,7 +184,7 @@ def __init__(
period=Duration.minutes(1),
statistic="Sum",
)
dashboard_widgets.append((0, widget_state_change_invocations))
dashboard_widgets.append((2, widget_state_change_invocations))

#####################
### cdk_nag stuff ###
Expand Down
2 changes: 1 addition & 1 deletion ContainerManager/leaf_stack/NestedStacks/Container.py
Original file line number Diff line number Diff line change
Expand Up @@ -86,4 +86,4 @@ def __init__(
"fields @message",
],
)
dashboard_widgets.append((0, container_logs_widget))
dashboard_widgets.append((9, container_logs_widget))
2 changes: 1 addition & 1 deletion ContainerManager/leaf_stack/NestedStacks/Dashboard.py
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ def __init__(
f"filter @message like / {route53_dns_sub_domain_name} /",
],
)
dashboard_widgets.append((0, dns_logs_widget))
dashboard_widgets.append((1, dns_logs_widget))

### Add the widgets to the dashboard:
widgets = [widget for _, widget in sorted(dashboard_widgets, key=lambda x: x[0])]
Expand Down
4 changes: 2 additions & 2 deletions ContainerManager/leaf_stack/NestedStacks/EcsAsg.py
Original file line number Diff line number Diff line change
Expand Up @@ -322,7 +322,7 @@ def __init__(
period=Duration.minutes(1),
statistic="Sum",
)
dashboard_widgets.append((0, traffic_widget))
dashboard_widgets.append((6, traffic_widget))

### ECS Info Widget:
# https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.aws_ecs.Ec2Service.html#metricwbrcpuwbrutilizationprops
Expand All @@ -343,7 +343,7 @@ def __init__(
period=Duration.minutes(1),
statistic="Maximum",
)
dashboard_widgets.append((0, ecs_container_graph))
dashboard_widgets.append((10, ecs_container_graph))

#####################
### cdk_nag stuff ###
Expand Down
12 changes: 6 additions & 6 deletions ContainerManager/leaf_stack/NestedStacks/Watchdog.py
Original file line number Diff line number Diff line change
Expand Up @@ -300,19 +300,19 @@ def __init__(
# have a different priority on the dashboard.
# (They have different numbers, so they stack on one-another.
# Otherwise they'd be on different rows).
(0, self.alarm_container_activity),
(0, self.alarm_watchdog_errors),
(0, self.alarm_asg_instance_left_up),
(5, self.alarm_container_activity),
(8, self.alarm_watchdog_errors),
(7, self.alarm_asg_instance_left_up),
]
## You can't append alarms to *this* widget after it's created, so I'm just having one per stack:
# https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.aws_cloudwatch.AlarmStatusWidget.html
alarm_status_widget = cloudwatch.AlarmStatusWidget(
title=f"Alarm Summary ({container_id})",
width=6,
width=3,
height=4,
alarms=[alarm[1] for alarm in graph_alarms],
)
dashboard_widgets.append((0, alarm_status_widget))
dashboard_widgets.append((4, alarm_status_widget))

for priority, alarm in graph_alarms:
alarm_widget = cloudwatch.AlarmWidget(
Expand All @@ -331,4 +331,4 @@ def __init__(
height=4,
metrics=[self.metric_asg_num_instances],
)
dashboard_widgets.append((0, num_instances_widget))
dashboard_widgets.append((3, num_instances_widget))

0 comments on commit bf03722

Please sign in to comment.