Skip to content

Conversation

@bbrowning
Copy link
Contributor

We've had quite a few AWS scheduling issues with our e2e-medium tests lately. This ports the region fallback logic to those as well.

@bbrowning
Copy link
Contributor Author

See https://github.com/instructlab/sdg/actions/runs/15542864333/job/43757770391 where this ran and successfully scheduled an instance using the multi-region fallback logic.

@mergify mergify bot added CI/CD Affects CI/CD configuration ci-failure labels Jun 9, 2025
Signed-off-by: Ben Browning <bbrownin@redhat.com>
@bbrowning bbrowning force-pushed the e2e-medium-fallback branch from 18c07a0 to 430074c Compare June 9, 2025 19:41
@mergify mergify bot added ci-failure and removed ci-failure labels Jun 9, 2025
Copy link
Contributor

@booxter booxter left a comment

Choose a reason for hiding this comment

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

Looks fine. We should probably give the manual job run a chance to complete to prove that stop action works as intended too before merging.

@mergify mergify bot added the one-approval label Jun 9, 2025
@bbrowning
Copy link
Contributor Author

Oh, the stop action did not work but I just force-pushed the fix for that 😁

@bbrowning
Copy link
Contributor Author

For more context, by canceling the e2e-medium job it actually tests the stop action. And, that's how I found and fixed the bug in the stop action.

@bbrowning bbrowning merged commit 00d2454 into main Jun 9, 2025
8 of 10 checks passed
@bbrowning bbrowning deleted the e2e-medium-fallback branch June 9, 2025 19:48
@bbrowning
Copy link
Contributor Author

@Mergifyio backport release-v0.8

@mergify
Copy link
Contributor

mergify bot commented Jun 9, 2025

backport release-v0.8

✅ Backports have been created

Details

bbrowning added a commit that referenced this pull request Jun 9, 2025
Run the e2e-medium job with aws region fallback (backport #663)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Affects CI/CD configuration ci-failure one-approval

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants