CI: Do not use custom AMI for CBMC runners #614
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
rod-chapman
reviewed
Nov 5, 2025
mkannwischer
force-pushed
the
cbmc-r8g
branch
from
611bf04 to
50077ba
Compare
Contributor
Author
|
Proof times are increased 6 -> 10mins; 6->11mins; 8->12mins when switching from c7g.4xlarge -> r8g.xlarge. This is still a net saving in proof cost with current EC2 pricing: https://aws.amazon.com/ec2/pricing/on-demand/. Let me try r8g.2xlarge. |
mkannwischer
force-pushed
the
cbmc-r8g
branch
2 times, most recently
from
c43e68d to
e5b51f4
Compare
Currently we are using custom AMIs for the CBMC tests. The only reason for using custom AMI is that require a disk of larger than the default 8 GB. This commit switches to default AMIs and instead uses the ec2_volume_size argument of the ec2-github-runner action. Signed-off-by: Matthias J. Kannwischer <[email protected]>
mkannwischer
force-pushed
the
cbmc-r8g
branch
from
e5b51f4 to
e80ed63
Compare
mkannwischer
changed the title
Contributor
Author
Let's stick to c7g.4xlarge for now. If cost becomes an issue at some point, we can consider switching. |
hanno-becker
approved these changes
Nov 10, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR ports part of pq-code-package/mlkem-native#1250
Currently we are using custom AMIs for the CBMC tests. The only
reason for using custom AMI is that require a disk of larger than
the default 8 GB.
This commit switches to default AMIs and instead uses the ec2_volume_size
argument of the ec2-github-runner action.