Skip to content

Conversation

xendo
Copy link
Contributor

@xendo xendo commented Aug 14, 2025

Adding network egress and ingress monitoring to the EC2Monitoring.

This partially addresses: #515

I tested this by deploying to my infrastructure and eyeballing the dashboard and alarms from the console

monitoring.monitorEC2Instances({
        ...
        autoScalingGroup: autoscalingGroup,
        addNetworkOutTotalBytesExceedThresholdAlarm: {
          WARNING: {
            maxNetworkOutBytes: 1_000_000_000,
          },
        },
      });
ec2-network-monitoring Fixes #

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@xendo xendo changed the title Add EC2Monitoring for network ingress/egress feat: add EC2Monitoring for network ingress/egress Aug 14, 2025
@ghost ghost added the do-not-merge Label to signify Mergify to not auto-merge. Useful for grouping PR merges into a single release. label Aug 14, 2025
@xendo
Copy link
Contributor Author

xendo commented Aug 14, 2025

still need one more revision, I messed up one more thing

@ghost ghost merged commit f9b64e6 into cdklabs:main Aug 14, 2025
9 checks passed
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge Label to signify Mergify to not auto-merge. Useful for grouping PR merges into a single release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant