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

Merged

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?

Copy link
Member

@mattgogerly mattgogerly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caveat that I'm not massively familiar with the ALB scaling policies, but lgtm.

Thanks for the screenshots!

@mattgogerly mattgogerly added the ready to merge Reviewed and ready for merge label Jan 22, 2025
@mergify mergify bot added the auto merged Merged automatically by a bot label Jan 22, 2025
@mergify mergify bot merged commit c6b3c0c into spinnaker:master Jan 22, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto merged Merged automatically by a bot ready to merge Reviewed and ready for merge target-release/1.37
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants