Skip to content

Comments

fix: normalize runtime version format in integration test log comparison#729

Merged
lym953 merged 1 commit intomainfrom
yiming.luo/fix-integ-test
Feb 20, 2026
Merged

fix: normalize runtime version format in integration test log comparison#729
lym953 merged 1 commit intomainfrom
yiming.luo/fix-integ-test

Conversation

@lym953
Copy link
Contributor

@lym953 lym953 commented Feb 20, 2026

Summary

  • The Lambda runtime now emits versions like python:3.10.mainlinev2.v3 instead of the old python:3.10.v3 format
  • This caused integration test snapshot mismatches on the INIT_START Runtime Version log line
  • Broadened the sed normalization pattern in scripts/run_integration_tests.sh to match any alphanumeric version suffix, replacing it with vX

Error from a sample failed test run:

Fetching logs for invocations and comparing to snapshots
Failed: Mismatch found between new async-metrics_python logs (first) and snapshot (second):
911c911
< INIT_START Runtime Version: python:3.10.mainlinev2.v3	Runtime Version ARN: arn:aws:lambda:eu-west-1:XXXX:eu-west-1
---
> INIT_START Runtime Version: python:3.10.vX	Runtime Version ARN: arn:aws:lambda:eu-west-1:XXXX:eu-west-1

Test plan

  • Run integration tests and verify no snapshot mismatches on INIT_START Runtime Version lines

Notes

Thanks @ojproductions for discussion.

🤖 Generated with Claude Code

The Lambda runtime now emits versions like `python:3.10.mainlinev2.v3`
instead of the old `python:3.10.v3` format, causing snapshot mismatches.
Broaden the sed pattern to match any alphanumeric version suffix.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@lym953 lym953 marked this pull request as ready for review February 20, 2026 20:53
@lym953 lym953 requested review from a team as code owners February 20, 2026 20:53
@lym953 lym953 merged commit 1031435 into main Feb 20, 2026
105 checks passed
@lym953 lym953 deleted the yiming.luo/fix-integ-test branch February 20, 2026 22:09
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.

4 participants