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: support customization metric auto scaling #40

Merged
merged 53 commits into from
Oct 24, 2023

Conversation

xshot9011
Copy link
Contributor

@xshot9011 xshot9011 commented Oct 10, 2023

Submit a pull request 🚀

Thank you for help us contribute! Please give us more information about this PR.


What 😗

Added

  • Support customization metric autoscaling
    • Local to make condition create resources: local.is_created_aws_appautoscaling_target local.is_target_tracking_scaling local.is_contain_predefined_metric
    • Related resources: aws_appautoscaling_policy.target_tracking_scaling_policies, aws_appautoscaling_policy.step_scaling_policies, module.step_alarm
    • Variables: step_scaling_configuration, target_tracking_configuration

Changed

  • Update resource tagging
    • local tags append with module's name created this resource
    • Resources: aws_secretsmanager_secret.this
  • Conditional to create resources: aws_appautoscaling_target.this, aws_appautoscaling_policy.target_tracking_scaling_policies, aws_appautoscaling_policy.step_scaling_policies

Removed

  • Variables: scaling_configuration

Why 🥺

  • Support multiple way to scale

xshot9011 and others added 19 commits February 22, 2023 03:51
* feat: add defualt kms with additional override kms

* fix: typo

* chore: update policy suffix

* chore: test cloudwatch log with kms

* chore: aws parameter confusion

* chore: update var naming

* fix: count depends on undetermined resource

* chore: ignore count on data source

* chore : turn var back to default null

* chore: test

* chore: test

* chore: test

* chore: test

* chore: test

* chore: update variable name

* chore: test new update from tf aws prov.

* fix: fix provider version to 4.65 dua to bug of 4.66

* chore: remove 4.66 feature

* chore: test <= sign
…:oozou/terraform-aws-ecs-fargate-service into feat/support-multiple-sidecard-container
@xshot9011 xshot9011 marked this pull request as ready for review October 20, 2023 11:30
@xshot9011 xshot9011 merged commit 040f05d into main Oct 24, 2023
1 check failed
@xshot9011 xshot9011 deleted the feat/support-customization-metric-auto-scaling branch October 24, 2023 03:46
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.

3 participants