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.

Sign up for GitHub

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

Jump to bottom

IPS-1284: Predictive scaling for ECS in forecast mode #357

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

IPS-1284: Predictive scaling for ECS in forecast mode #357

wants to merge 1 commit into from

Conversation

louiseli1
Copy link
Contributor

What changed

  • Added capacity mapping for ECS task counts
  • Removed condition on creating auto scaling policy so it applies to all envs
  • This adds a new Predicitive scaling policy which will scale the number of instances based on some AWS created algorithm using the previous traffic patterns as input. See this for more information:
    https://docs.aws.amazon.com/autoscaling/ec2/userguide/ec2-auto-scaling-predictive-scaling.html
  • The policy is in Forecast Mode only so we can understand its behaviour. The predictive scaling policy will not take any action

Why did it change

  • Cost saving
  • To be more Environmentally friendly
  • Reliability for users

Issue tracking

https://govukverify.atlassian.net/browse/IPS-1284

@louiseli1 louiseli1 requested a review from a team as a code owner January 14, 2025 16:56
@louiseli1 louiseli1 force-pushed the IPS-1284-turn-on-predictive-scaling-for-ecs-in-forecast-mode branch from 1212bc8 to ae14d28 Compare January 14, 2025 17:01
TobySaundersGDS
TobySaundersGDS previously approved these changes Jan 14, 2025
View reviewed changes
Copy link

@TobySaundersGDS TobySaundersGDS left a comment

Choose a reason for hiding this comment

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

Thanks - this will be interesting to see in few weeks 🖼️

@louiseli1 louiseli1 force-pushed the IPS-1284-turn-on-predictive-scaling-for-ecs-in-forecast-mode branch from ae14d28 to 6e8baf7 Compare January 17, 2025 11:12
@louiseli1 louiseli1 force-pushed the IPS-1284-turn-on-predictive-scaling-for-ecs-in-forecast-mode branch from 6e8baf7 to 02c3d18 Compare January 17, 2025 13:14
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ElliotMurphyGDS ElliotMurphyGDS ElliotMurphyGDS approved these changes

@TobySaundersGDS TobySaundersGDS TobySaundersGDS left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@louiseli1 @ElliotMurphyGDS @TobySaundersGDS