Skip to content

ci(rust): resolve aws-lc-sys versions, simplify duplicate aws-lc error message - #2366

Merged
lucasmcdonald3 merged 3 commits into
mainfrom
rust-dedupe-error-msg-simplify
Aug 28, 2026
Merged

ci(rust): resolve aws-lc-sys versions, simplify duplicate aws-lc error message#2366
lucasmcdonald3 merged 3 commits into
mainfrom
rust-dedupe-error-msg-simplify

Conversation

@lucasmcdonald3

@lucasmcdonald3 lucasmcdonald3 commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:

Replace verbose failure message slop with something useful.
ex. from actual DBESDK message log

Error: aws-lc-sys versions aren't consistent. Fix: pin it to 0.44.0 in DynamoDbEncryption/runtimes/rust/Cargo.toml.
In DynamoDbEncryption/runtimes/rust/Cargo.toml line 20, change
- aws-lc-sys = "0.43"
+ aws-lc-sys = "0.44.0"
(aws-lc-rs v1.18.0 resolves aws-lc-sys to v0.44.0; matching it avoids inconsistencies.)

Squash/merge commit message, if applicable:

ci(rust): resolve aws-lc-sys versions, simplify duplicate aws-lc error message

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Replace the verbose failure spiel with a concise, actionable diff that
prints the exact Cargo.toml line to change (before/after) and the
version aws-lc-rs resolves to.
@lucasmcdonald3
lucasmcdonald3 requested a review from a team as a code owner August 26, 2026 23:14
Refine the error message (repo-relative Cargo.toml path, exact line,
resolving aws-lc-rs version). Bump aws-lc-rs to 1.18.0 and pin
aws-lc-sys 0.44 / aws-lc-fips-sys 0.14.1 so only one AWS-LC compiles.
@lucasmcdonald3 lucasmcdonald3 changed the title ci(rust): simplify duplicate aws-lc error message ci(rust): resolve aws-lc-sys versions, simplify duplicate aws-lc error message Aug 27, 2026
Extend the duplicate-aws-lc gate to loop over both the DynamoDbEncryption
and TestVectors Rust runtimes (matching the ESDK setup), and bump the
TestVectors runtime to aws-lc-rs 1.18.0 / aws-lc-sys 0.44 /
aws-lc-fips-sys 0.14.1 so only one AWS-LC compiles.
@lucasmcdonald3
lucasmcdonald3 merged commit beb6a43 into main Aug 28, 2026
70 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants