Skip to content

3175 add AWS smoke test #3317

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

Merged
merged 2 commits into from
Apr 10, 2025
Merged

3175 add AWS smoke test #3317

merged 2 commits into from
Apr 10, 2025

Conversation

jimmykarily
Copy link
Contributor

@jimmykarily jimmykarily commented Apr 7, 2025

Part of #3175

  • Fixes cleanup script that didn't properly split the versions
  • Treats beta,alpha,pre and others as non-stable (not just "rc")
  • Tests the aws AMI before copying it to other regions or making it public

@jimmykarily jimmykarily self-assigned this Apr 7, 2025
@jimmykarily jimmykarily moved this to In Progress 🏃 in 🧙Issue tracking board Apr 7, 2025
@jimmykarily jimmykarily force-pushed the 3175-add-smoke-tests branch from 35bdb36 to 681ee43 Compare April 7, 2025 14:39
@jimmykarily jimmykarily force-pushed the 3175-add-smoke-tests branch from 681ee43 to a9aa4d4 Compare April 8, 2025 07:01
@jimmykarily jimmykarily force-pushed the 3175-add-smoke-tests branch from 450a97c to ba3c4c0 Compare April 8, 2025 12:22
@jimmykarily jimmykarily force-pushed the 3175-add-smoke-tests branch from 39e94d1 to 14ba113 Compare April 9, 2025 05:54
@jimmykarily jimmykarily force-pushed the 3175-add-smoke-tests branch from 874a027 to 40674e7 Compare April 9, 2025 06:41
@jimmykarily jimmykarily force-pushed the 3175-add-smoke-tests branch from 3e297b6 to 41c3ab6 Compare April 9, 2025 07:58
@jimmykarily jimmykarily force-pushed the 3175-add-smoke-tests branch from 41c3ab6 to fb81dc8 Compare April 9, 2025 07:59
@jimmykarily jimmykarily changed the title 3175 add smoke tests 3175 add AWS smoke test Apr 9, 2025
@jimmykarily jimmykarily marked this pull request as ready for review April 9, 2025 08:00
@jimmykarily jimmykarily force-pushed the 3175-add-smoke-tests branch from fb81dc8 to 4948c55 Compare April 9, 2025 08:01
@jimmykarily jimmykarily force-pushed the 3175-add-smoke-tests branch from 32dc009 to 5a77bd2 Compare April 9, 2025 13:27
@jimmykarily jimmykarily force-pushed the 3175-add-smoke-tests branch from 5a77bd2 to 4fc2146 Compare April 9, 2025 14:01
@jimmykarily jimmykarily force-pushed the 3175-add-smoke-tests branch from 6f6517e to 17ab260 Compare April 10, 2025 05:55
@jimmykarily jimmykarily force-pushed the 3175-add-smoke-tests branch from 17ab260 to 8d7f5ac Compare April 10, 2025 06:07
- Fix cleanup script that didn't properly split the versions
- Treat beta,alpha,pre and others as non-stable (not just "rc")
- Test the aws AMI before copying it to other regions or making it
  public

Signed-off-by: Dimitris Karakasilis <[email protected]>
@jimmykarily jimmykarily force-pushed the 3175-add-smoke-tests branch from 8d7f5ac to c0d2c6e Compare April 10, 2025 07:15
Signed-off-by: Dimitris Karakasilis <[email protected]>
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Files not reviewed (5)
  • .github/public-cloud/cleanup-old-images-aws.sh: Language not supported
  • .github/public-cloud/cleanup-old-images-azure.sh: Language not supported
  • .github/public-cloud/cleanup-old-images-gcp.sh: Language not supported
  • .github/public-cloud/test-aws-image.sh: Language not supported
  • .github/public-cloud/upload-image-to-aws.sh: Language not supported

@jimmykarily jimmykarily merged commit fd906da into master Apr 10, 2025
42 of 72 checks passed
@jimmykarily jimmykarily deleted the 3175-add-smoke-tests branch April 10, 2025 09:19
@github-project-automation github-project-automation bot moved this from Under review 🔍 to Done ✅ in 🧙Issue tracking board Apr 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants