WIP: feat(lambda): Use Lambda Runtime Emulator Interface v2#12736
WIP: feat(lambda): Use Lambda Runtime Emulator Interface v2#12736gregfurman wants to merge 23 commits intomainfrom
Conversation
Test Results - Preflight, Unit22 145 tests 20 407 ✅ 6m 42s ⏱️ Results for commit 4ea8ce2. ♻️ This comment has been updated with latest results. |
Test Results (MA/MR) - Preflight, Unit22 145 tests 20 407 ✅ 7m 0s ⏱️ Results for commit 4ea8ce2. ♻️ This comment has been updated with latest results. |
Test Results (amd64) - Acceptance0 tests 0 ✅ 0s ⏱️ Results for commit 4ea8ce2. ♻️ This comment has been updated with latest results. |
Test Results (amd64, MA/MR) - Acceptance0 tests 0 ✅ 0s ⏱️ Results for commit 4ea8ce2. ♻️ This comment has been updated with latest results. |
Test Results (MA/MR) - Alternative Providers987 tests 584 ✅ 23m 18s ⏱️ Results for commit 88b413e. ♻️ This comment has been updated with latest results. |
Test Results - Alternative Providers987 tests 584 ✅ 22m 18s ⏱️ Results for commit 88b413e. ♻️ This comment has been updated with latest results. |
Test Results (amd64, MA/MR) - Integration, Bootstrap24 tests 23 ✅ 11m 38s ⏱️ For more details on these failures, see this check. Results for commit 4ea8ce2. ♻️ This comment has been updated with latest results. |
Test Results (amd64) - Integration, Bootstrap24 tests 23 ✅ 11m 57s ⏱️ For more details on these failures, see this check. Results for commit 4ea8ce2. ♻️ This comment has been updated with latest results. |
LocalStack Community integration with Pro 2 files ± 0 2 suites ±0 40s ⏱️ - 1h 42m 14s For more details on these errors, see this check. Results for commit f79554a. ± Comparison against base commit 5783fa3. This pull request removes 4894 and adds 25 tests. Note that renamed tests count towards both.♻️ This comment has been updated with latest results. |
| "errorMessage": "[Errno 13] Permission denied: '/proc/1/environ'", | ||
| "errorType": "PermissionError", | ||
| "requestId": "<uuid:2>", | ||
| "stackTrace": [ | ||
| " File \"/var/task/handler.py\", line 3, in handler\n with open(f\"/proc/{pid}/environ\", mode=\"rt\") as f:\n" | ||
| ] |
There was a problem hiding this comment.
Can we parity-test/inspect this, or does the permission prohibit that?
If not, @dfangl might have a better guess. Maybe, try 600 🤷♂️
There was a problem hiding this comment.
I think the tests makes no longer sense then - it was mean to introspect the environment of the init process, if that is no longer possible in AWS, there is no real point (although we could try to hide it in our containers as well, I don't necessarily see this as a priority)
|
Currently, only patch changes are allowed on main. Your PR labels (aws:lambda, semver: minor) indicate that it cannot be merged into the main at this time. |
Motivation
Currently a WIP. This PR was created to trigger a CI/CD run with the new Lambda RIE v2 enabled by default.
Changes
Related
Part of DRG-1