Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(amazon): support ALBRequestCountPerTarget scaling policies #10160

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

chris-h-phillips
Copy link
Contributor

@chris-h-phillips chris-h-phillips commented Jan 1, 2025

The UI for TargetTracking scaling policies didn't support ALBRequestCountPerTarget. This was the only missing metric documented here: https://docs.aws.amazon.com/autoscaling/ec2/userguide/as-scaling-target-tracking.html

This requires setting the resourceLabel parameter which is a combination of part of both the ALB and target group ARNs. This metric also only works with the Sum statistic. I found that the chart component hard coded average so that had to be fixed up as well.

Here are some screenshots
Summary before and after:
image

image
UpsertModal before and after:
(before the ALBRequestCountPerTarget isn't available)
image

image

The UI for TargetTracking scaling policies didn't support ALBRequestCountPerTarget. This was the only missing metric documented here: https://docs.aws.amazon.com/autoscaling/ec2/userguide/as-scaling-target-tracking.html

This requires setting the resourceLabel parameter which is a combination of part of both the ALB and target group ARNs. This metric also only works with the Sum statistic. I found that the chart component hard coded average so that had to be fixed up as well.
@chris-h-phillips chris-h-phillips force-pushed the add-albrequestcountpertarget-support branch from 2254737 to 9a65f61 Compare January 1, 2025 09:02
@dbyron-sf
Copy link
Contributor

@mattgogerly can you take a look at this please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants