Skip to content

Conversation

GarrettBeatty
Copy link
Contributor

@GarrettBeatty GarrettBeatty commented Jul 12, 2025

Issue #, if available: Update tests to use t3.micro free tier

it looks like there was a recent change where t3.micro is free tier now. so this updates the test to select "yes" for free tier so that t3.micro is used ( i couldnt find any news/docs on this change though so maybe im missing something)

below is query result i got when doing describe instances in us-west-2

InstanceType FreeTierEligible Architecture CPU Cores Memory (MiB)
------------ ---------------- ------------ --------- ------------
t2.2xlarge              False x86_64               8        32768
t3.xlarge               False x86_64               2        16384
t3.2xlarge              False x86_64               4        32768
t2.large                False x86_64               2         8192
t3.medium               False x86_64               1         4096
t3.large                False x86_64               1         8192
t2.micro                False i386, x86_64         1         1024
t3.micro                 True x86_64               1         1024
t3.nano                 False x86_64               1          512
t2.medium               False i386, x86_64         2         4096
t2.small                False i386, x86_64         1         2048
t3.small                 True x86_64               1         2048
t2.xlarge               False x86_64               4        16384
t2.nano                 False i386, x86_64         1          512

Description of changes:

  1. use t3 micro.
  • testing*
    im not able to test in my personal isengard account because my account still shows t2.nano as free tier (idk why)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@GarrettBeatty GarrettBeatty changed the title fix t2.micro usage fix t2.nano usage Jul 12, 2025
@GarrettBeatty GarrettBeatty marked this pull request as ready for review July 12, 2025 14:54
Copy link

codecov bot commented Jul 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 62.11%. Comparing base (520a18b) to head (37fa9d2).
Report is 5 commits behind head on dev.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #964   +/-   ##
=======================================
  Coverage   62.11%   62.11%           
=======================================
  Files         294      294           
  Lines       10813    10813           
  Branches     1616     1616           
=======================================
  Hits         6717     6717           
  Misses       3494     3494           
  Partials      602      602           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@GarrettBeatty GarrettBeatty requested a review from philasmar July 14, 2025 14:15
@GarrettBeatty GarrettBeatty added the Release Not Needed Add this label if a PR does not need to be released. label Jul 14, 2025
@GarrettBeatty GarrettBeatty merged commit a2ff878 into dev Jul 14, 2025
15 of 18 checks passed
@GarrettBeatty GarrettBeatty deleted the gcbeatty/freetier branch July 14, 2025 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release Not Needed Add this label if a PR does not need to be released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants