-
Notifications
You must be signed in to change notification settings - Fork 49
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
Include t3a.micro in compute.yaml #304
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a sensible change:
t3a.micro (0.9¢/h)
t3.micro (~1¢/h)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The compute policy is too big. At this point, there's nothing obvious to trim down. I think I would recommend replacing all the delete permissions for autoscaling
in the AllowGlobalResourceRestrictedActionsWhichIncurNoFees
with autoscaling:Delete*
. The only new permission this adds is the ability to delete warm pools, and I think that's fine.
) SUMMARY Refer: https://issues.redhat.com/browse/ACA-1691 Requires mattclay/aws-terminator#304 for permissions. Some of the ec2 instance types used in the integration tests are switched to t3a.micro to avoid intermittent failures in the CI ISSUE TYPE Bugfix Pull Request Docs Pull Request Feature Pull Request New Module Pull Request COMPONENT NAME ADDITIONAL INFORMATION Reviewed-by: Alina Buzachis Reviewed-by: Mark Chappell
) SUMMARY Refer: https://issues.redhat.com/browse/ACA-1691 Requires mattclay/aws-terminator#304 for permissions. Some of the ec2 instance types used in the integration tests are switched to t3a.micro to avoid intermittent failures in the CI ISSUE TYPE Bugfix Pull Request Docs Pull Request Feature Pull Request New Module Pull Request COMPONENT NAME ADDITIONAL INFORMATION Reviewed-by: Alina Buzachis Reviewed-by: Mark Chappell (cherry picked from commit 2f1fcab)
) SUMMARY Refer: https://issues.redhat.com/browse/ACA-1691 Requires mattclay/aws-terminator#304 for permissions. Some of the ec2 instance types used in the integration tests are switched to t3a.micro to avoid intermittent failures in the CI ISSUE TYPE Bugfix Pull Request Docs Pull Request Feature Pull Request New Module Pull Request COMPONENT NAME ADDITIONAL INFORMATION Reviewed-by: Alina Buzachis Reviewed-by: Mark Chappell (cherry picked from commit 2f1fcab)
) (#2190) This is a backport of PR #2188 as merged into main (2f1fcab). SUMMARY Refer: https://issues.redhat.com/browse/ACA-1691 Requires mattclay/aws-terminator#304 for permissions. Some of the ec2 instance types used in the integration tests are switched to t3a.micro to avoid intermittent failures in the CI ISSUE TYPE Bugfix Pull Request Docs Pull Request Feature Pull Request New Module Pull Request COMPONENT NAME ADDITIONAL INFORMATION Reviewed-by: Mark Chappell
) (#2189) This is a backport of PR #2188 as merged into main (2f1fcab). SUMMARY Refer: https://issues.redhat.com/browse/ACA-1691 Requires mattclay/aws-terminator#304 for permissions. Some of the ec2 instance types used in the integration tests are switched to t3a.micro to avoid intermittent failures in the CI ISSUE TYPE Bugfix Pull Request Docs Pull Request Feature Pull Request New Module Pull Request COMPONENT NAME ADDITIONAL INFORMATION Reviewed-by: Mark Chappell
This PR adds t3a.micro instance type to aws/policy/compute.yaml
Refer: https://issues.redhat.com/browse/ACA-1691